Activity
From 02/22/2013 to 03/23/2013
03/24/2013
- 10:11 PM Database Bug #2084 (Closed): Conversion misplaced parameter for can-find statement
- 10:10 PM Database Feature #2000 (Closed): add conversion support for table and table handle parameters
- 10:09 PM Database Bug #2094: OUTER-JOIN record phrase option not honored
- The following:...
- 09:49 PM Database Feature #2092 (Closed): add server-side database support for several builtin functions
- 09:46 PM Database Bug #2101 (Review): multi-buffer FOR loop with rowid and to-rowid functions in WHERE clause not c...
- Update ecf_upd20130323a.zip has passed conversion regression testing and is committed to bzr rev. 10325.
- 08:23 PM Database Bug #2101: multi-buffer FOR loop with rowid and to-rowid functions in WHERE clause not converting...
- Although I'm not the assignee, in having to think through the problem for the above post, I took a quick look at the ...
- 09:26 PM Database Feature #2080: add runtime support for database methods
- This indicates that the idiom of:...
- 08:40 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- This update fixes the regressions in Majic (although there is still 1 change that is expected but will need to be che...
03/23/2013
- 06:54 PM Database Feature #1668: add support for additional (non-dynamic) database attributes
- This update has passed conversion regression testing (CA 0323d used deleteRecord for the "quick_delete" loops, instea...
- 12:58 PM Database Feature #1668: add support for additional (non-dynamic) database attributes
- ca_upd20130323d.zip update looks good to me. If it passes conversion regression testing (expect some changes to Maji...
- 12:52 PM Database Feature #1668: add support for additional (non-dynamic) database attributes
- Replaces ca_upd2013023a.zip, and has the changes from note 105.
- 12:33 PM Database Feature #1668: add support for additional (non-dynamic) database attributes
- You are not missing anything, this is the first time I touch embedded sqls, so that's why the questions. Now I have a...
- 12:30 PM Database Feature #1668: add support for additional (non-dynamic) database attributes
- This refers to:...
- 12:23 PM Database Feature #1668: add support for additional (non-dynamic) database attributes
- For embedded SQL, I was referring to this rule in convert/database_access:...
- 12:19 PM Database Feature #1668: add support for additional (non-dynamic) database attributes
- Let's use:...
- 11:38 AM Database Feature #1668: add support for additional (non-dynamic) database attributes
- Eric: the ca_upd20130323a.zip has regressions. I see that the Buffer.delete API is used by other cases, like DELETE s...
- 10:23 AM Database Feature #1668: add support for additional (non-dynamic) database attributes
- This update changes the API for BUFFER-DELETE to deleteRecord, because else will collide with the resource's delete()...
- 06:53 PM Database Feature #2080: add runtime support for database methods
- Also I have noticed the difference in behavior of 4GL and converted code.
I am comparing a lot of tests in the form... - 06:50 PM Database Bug #2101: multi-buffer FOR loop with rowid and to-rowid functions in WHERE clause not converting...
- This issue was first reported in note 192 of #2068.
The following converts badly:... - 06:34 PM Database Bug #2101 (Closed): multi-buffer FOR loop with rowid and to-rowid functions in WHERE clause not c...
- 06:40 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- The changes in note 161 did cause some regressions in Majic. I have fixed all but one of them. As soon as that one ...
- 05:17 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- You are correct, the approach you mentioned from functions.rules matches the not-needed cases too.
> I can't really ... - 05:07 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Part of the issue with note 155 is the over-aggressive setting of ignorecast in annotations/functions.rules:...
- 04:44 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- The update looks OK, I can't find any obvious flaws in the logic.
- 04:26 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- About note 157: please see the handle_dyn_func6.p and handle_dyn_func5b.p (the substring issue at note 157 is from he...
- 04:18 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- The extra ';' might be related to the comments changes. For query-prepare (and dynamic-function as parameter in gener...
- 04:16 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- In regard to note 155, the same testcase is a bit different after my changes in note 161. Here is the output:
<pr... - 03:04 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Here is the proposed update to resolve notes 135 and 139. This resolves all known cases of these problems.
I will... - 03:18 PM Base Language Bug #2070: BlockManager Action disambiguation
- Converting the extent_params_proc.p test on windows fails with this:...
- 10:04 AM Conversion Tools Bug #2066: brew adds unnecessary comma in method parameter list after comment
- ca_upd20130322i.zip was committed to bzr revision 10320. MAJIC conversion regression testing has passed (with comment...
03/22/2013
- 05:38 PM Database Bug #2095: FIELDS/EXCEPT record phrase options not honored
- > I still think the include/exclude call should directly follow the associated addComponent call. I think it is easie...
- 05:31 PM Database Bug #2095: FIELDS/EXCEPT record phrase options not honored
- I guess there's one more option; chain the call, as in:...
- 05:04 PM Database Bug #2095: FIELDS/EXCEPT record phrase options not honored
- So if the full list of options is:
1. Add the list of fields to query c'tor / addComponent
2. include(String ...) -... - 04:34 PM Database Bug #2095: FIELDS/EXCEPT record phrase options not honored
- Stanislav Lomany wrote:
> For a multi-table query the buffer should be specified, but as an option we can have quali... - 04:28 PM Database Bug #2095: FIELDS/EXCEPT record phrase options not honored
- For a multi-table query the buffer should be specified, but as an option we can have qualified field names:...
- 03:32 PM Database Bug #2095: FIELDS/EXCEPT record phrase options not honored
- OK, but please emit this method immediately after the associated @addComponent@ call.
> ... - 03:07 PM Database Bug #2095: FIELDS/EXCEPT record phrase options not honored
- Unless someone has objections, for multi-component queries, I'll make include/exclude methods of the parent query:
<... - 11:42 AM Database Bug #2095: FIELDS/EXCEPT record phrase options not honored
- > Removes FIELDS/EXCEPT nodes.
Thanks, I'm starting server folder conversion, hope nothing new will appear. - 11:36 AM Database Bug #2095: FIELDS/EXCEPT record phrase options not honored
- Removes FIELDS/EXCEPT nodes.
- 10:38 AM Database Bug #2095: FIELDS/EXCEPT record phrase options not honored
- OK
- 10:14 AM Database Bug #2095: FIELDS/EXCEPT record phrase options not honored
- Stanislav: if it doesn't take you long to build some rules which just drop/hide the FIELDS/EXCEPT nodes (so they will...
- 04:25 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Ovidiu: please post here all the cases related to LENGTH function which are not converting properly (the cases when t...
- 02:43 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- OK, I think the case at note 155 here is related to 157. I'll wait for your changes before continuing.
- 01:39 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- > If think for such cases ECW should return null, if the function's return type and the parameter's type are not comp...
- 11:24 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- There is a problem in ECW/base_structure. For a case like:...
- 10:07 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- No, I haven't looked at this case.
- 09:59 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Greg, have you touched this case:...
- 09:33 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- ca_upd20130322d.zip committed to bzr revision 10312.
- 08:20 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- It is good. Check it in and distribute it if it passes testing.
- 07:23 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- I think I found the problem for the BEGINS operator problem @if h:buffer-value begins ch then message "blah".@. In op...
- 04:22 PM Conversion Tools Bug #2066: brew adds unnecessary comma in method parameter list after comment
- I agree. The approach you have taken is fine for now.
I also agree that we will need to make additional changes t... - 04:09 PM Conversion Tools Bug #2066: brew adds unnecessary comma in method parameter list after comment
- This is a quirk for the SWITCH statement comment case. At this time, I think convert/comments.rules might be incomple...
- 02:32 PM Database Feature #1668: add support for additional (non-dynamic) database attributes
- Costin Savin wrote:
> Well theoreticaly it is already there because both Buffer and TempTable extend it, but probabl... - 02:32 PM Database Feature #1668: add support for additional (non-dynamic) database attributes
- Well theoreticaly it is already there because both Buffer and TempTable extend it, but probably it's good to see it i...
- 02:12 PM Database Feature #1668: add support for additional (non-dynamic) database attributes
- Costin, regarding cs_upd20130322b.zip - you forgot to add the new interface to HandleCommon. Please put it right afte...
- 01:43 PM Database Feature #1668: add support for additional (non-dynamic) database attributes
- Commited as revision number 10317
- 01:07 PM Database Feature #1668: add support for additional (non-dynamic) database attributes
- cs_upd20130322b.zip has passed conversion regression testing. Please check it in and distribute.
- 10:54 AM Database Feature #1668: add support for additional (non-dynamic) database attributes
- cs_upd20130322b.zip looks good. I am putting it through conversion regression testing.
It does look like we still... - 08:21 AM Database Feature #1668: add support for additional (non-dynamic) database attributes
- Added proposed update for missing addFieldToIndex methods. Moved the defaultBufferHandle() method form the Buffer int...
- 05:06 AM Database Feature #1668: add support for additional (non-dynamic) database attributes
- This class evolved a little. The @defaultBufferHandle()@ (implementation of @DEFAULT-BUFFER-HANDLE@) was at the begin...
- 04:35 AM Database Feature #1668: add support for additional (non-dynamic) database attributes
- > ADMData was my mistake (should have not added it, as I don't see the ADM-DATA attribute defined for the temp-table ...
- 02:15 PM Database Feature #2080: add runtime support for database methods
- I have noticed from testing in 4GL that error message...
- 10:46 AM Database Feature #1947: add conversion support for database methods
- CA 0322c.zip was committed to bzr revision 10315 and CA 0322e.zip to 10316.
- 10:40 AM Database Feature #1947: add conversion support for database methods
- Both updates look good, please check them in.
- 09:30 AM Database Feature #1947: add conversion support for database methods
- Both 0322c.zip and 0322e.zip have passed conversion regression testing.
- 07:42 AM Database Feature #1947: add conversion support for database methods
- Adds missing @queryRows(integer)@ API.
- 06:49 AM Database Feature #1947: add conversion support for database methods
- Solved note 151 by always expanding array vars passed to the REPOSITION-TO-ROWID method. This avoids adding an @query...
- 06:06 AM Database Feature #1947: add conversion support for database methods
- I'm working on note 151.
- 05:23 AM Database Feature #1947: add conversion support for database methods
- See note 202 from #2068, which documents a server failure. Note that the @REPOSITION-TO-ROWID@ can accept mixed exten...
- 10:24 AM Database Feature #2000: add conversion support for table and table handle parameters
- ca_upd20130322g.zip committed to bzr revision 10214.
- 09:15 AM Database Feature #2000: add conversion support for table and table handle parameters
- The code looks fine. If it passes testing, check it in and distribute it.
- 09:11 AM Database Feature #2000: add conversion support for table and table handle parameters
- Fixes a regression in the TABLE-HANDLE parameter conversion, introduced by 10293.
- 08:55 AM Database Feature #2000: add conversion support for table and table handle parameters
- OK, I'll look at it.
- 08:15 AM Database Feature #2000: add conversion support for table and table handle parameters
- It has been regressed since revision 10279.
Correct output is:... - 07:38 AM Database Feature #2000: add conversion support for table and table handle parameters
- Stanislav: did this convert properly after you added TABLE-HANDLE parameter support?...
- 09:37 AM Base Language Feature #1920: implement persistent procedures
- Merged with latest bzr version, contains both CA 0322a.zip and CA 0322f.zip changes. committed to bzr revision 10313.
- 08:24 AM Base Language Feature #1920: implement persistent procedures
- The code is OK. It can be committed and distributed when it passes testing.
- 07:54 AM Base Language Feature #1920: implement persistent procedures
- Fixes the case when a method param has subscript:...
- 06:04 AM Base Language Feature #1920: implement persistent procedures
- The code is OK. If it passes conversion testing you can check it in and distribute it.
- 05:31 AM Base Language Feature #1920: implement persistent procedures
- Fixed conversion support for HANDLE attribute, when used with system handles:...
- 08:18 AM Database Feature #2092: add server-side database support for several builtin functions
- Go ahead and check this in immediately (and distribute it). It has no effect on conversion and it is already being r...
- 07:07 AM Database Feature #2092: add server-side database support for several builtin functions
- Removed the extra ' character from p2jpl.ddr at line 699
- 06:59 AM Base Language Feature #1608: implement full appserver support (from 4GL clients only)
- Checked in to bzr as revision 10310.
- 05:26 AM Base Language Feature #1608: implement full appserver support (from 4GL clients only)
- Conversion testing passed. Check it in and distribute it.
03/21/2013
- 04:10 PM Base Language Feature #1608: implement full appserver support (from 4GL clients only)
- Added merged update
- 01:51 PM Base Language Feature #1608: implement full appserver support (from 4GL clients only)
- Code Feedback:
1. The javadoc and formatting of ConnectableServer should be this:... - 01:13 PM Base Language Feature #1608: implement full appserver support (from 4GL clients only)
- Added proposed update which fixes connect for AppServer
- 10:37 AM Base Language Feature #1608: implement full appserver support (from 4GL clients only)
- You can force things to be wrapped in that case where there is more than 1 parameter.
- 10:27 AM Base Language Feature #1608: implement full appserver support (from 4GL clients only)
- > [javac] .../server/common/Apitest1.java:134: error: no suitable method found for connect(String,character,character...
- 03:54 PM Database Feature #1668: add support for additional (non-dynamic) database attributes
- From the TEMP-TABLE attributes the following are not implemented in any interface:
AFTER-TABLE, BEFORE-TABLE, DATA... - 01:54 PM Database Feature #1668: add support for additional (non-dynamic) database attributes
- Eric/Costin: there is one more thing we should do - identify all attributes which are already defined by interfaces (...
- 01:49 PM Database Feature #1668: add support for additional (non-dynamic) database attributes
- > So, Costin: please remove Buffer and ADMData from the TempTable interface and post here what method stubs are still...
- 01:42 PM Database Feature #1668: add support for additional (non-dynamic) database attributes
- So, Costin: please remove Buffer and ADMData from the TempTable interface and post here what method stubs are still ...
- 01:39 PM Database Feature #1668: add support for additional (non-dynamic) database attributes
- Constantin Asofiei wrote:
> Why is TempTable interface extending Buffer, ADMData and UniqueID interfaces?
Good qu... - 01:38 PM Database Feature #1668: add support for additional (non-dynamic) database attributes
- ADMData was my mistake (should have not added it, as I don't see the ADM-DATA attribute defined for the temp-table re...
- 01:31 PM Database Feature #1668: add support for additional (non-dynamic) database attributes
- Eric/Ovidiu/Costin: the interfaces extended by TempTable don't look OK. In the docs, the temp-table resource can acce...
- 01:30 PM Database Feature #1668: add support for additional (non-dynamic) database attributes
- Costin Savin wrote:
> > That's strange, all the TempTable methods should be implemented in TempTableBuilder, AFAIK. ... - 01:25 PM Database Feature #1668: add support for additional (non-dynamic) database attributes
- > That's strange, all the TempTable methods should be implemented in TempTableBuilder, AFAIK. I don't know why they a...
- 12:55 PM Database Feature #1668: add support for additional (non-dynamic) database attributes
- Vadim, your 20130321a update has passed conversion regression testing. Please check it in (and the test case) and di...
- 11:50 AM Database Feature #1668: add support for additional (non-dynamic) database attributes
- Ovidiu Maxiniuc wrote:
> > That's strange, all the TempTable methods should be implemented in TempTableBuilder, AFAI... - 11:36 AM Database Feature #1668: add support for additional (non-dynamic) database attributes
- Code review 20130321a:
Looks good, I'm putting it through conversion regression test now. - 10:59 AM Database Feature #1668: add support for additional (non-dynamic) database attributes
- Update vmn_upd20130321a.zip (vmn_upd20130321b.zip - conversion test) is replacement of vmn_upd20130320a.zip (vmn_upd2...
- 09:17 AM Database Feature #1668: add support for additional (non-dynamic) database attributes
- > That's strange, all the TempTable methods should be implemented in TempTableBuilder, AFAIK. I don't know why they ...
- 06:55 AM Database Feature #1668: add support for additional (non-dynamic) database attributes
- > Please post your test case.
I used the pers-addr table from p2j_test and created a temp table like pers-addr and... - 02:20 PM Database Bug #2095: FIELDS/EXCEPT record phrase options not honored
- Stanislav Lomany wrote:
> Do I need to ensure that there are no duplicates in the converted list (e.g. if we are con... - 02:01 PM Database Bug #2095: FIELDS/EXCEPT record phrase options not honored
- It is possible to reference an element of an extent field. The progress documentation says that "If field is an array...
- 11:21 AM Database Bug #2095 (WIP): FIELDS/EXCEPT record phrase options not honored
- 01:32 PM Base Language Bug #2099 (Closed): "ASSIGN widget." statement does not convert properly in some cases
- 12:53 PM Base Language Bug #2099 (Review): "ASSIGN widget." statement does not convert properly in some cases
- > Check it in and distribute it.
Committed to bzr revision 10308. - 12:46 PM Base Language Bug #2099: "ASSIGN widget." statement does not convert properly in some cases
- This looks good. Check it in and distribute it.
- 11:41 AM Base Language Bug #2099: "ASSIGN widget." statement does not convert properly in some cases
- This update has passed conversion regression testing. There are some changes in MAJIC, but they do not affect runtime...
- 09:45 AM Base Language Bug #2099: "ASSIGN widget." statement does not convert properly in some cases
- I'm still missing something, lots of invalid changes in MAJIC.
- 08:58 AM Base Language Bug #2099: "ASSIGN widget." statement does not convert properly in some cases
- This is a fix attempt for this issue. Looks like the solution was to explicitly set the frame name, in case the "fram...
- 01:30 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- While working on #2068 note 187, I encountered the following case:...
- 09:48 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Here is the final version of the indeterminate cases. I have done 2 equivalent versions: one using buffer-value and ...
- 11:23 AM Base Language Feature #2089 (Closed): INPUT function returning string value
- 10:37 AM Base Language Feature #2089: INPUT function returning string value
- Please do check in and distribute the change.
- 10:14 AM Base Language Feature #2089: INPUT function returning string value
- Yes, they all are OK.
- 09:51 AM Base Language Feature #2089: INPUT function returning string value
- Here are the results of the conversion testing:...
- 08:12 AM Base Language Feature #2089: INPUT function returning string value
- Code Review:
It looks good. 1 minor issue: the comment was corrupted in "handle_implicit_input" in the latest com... - 09:51 AM Database Feature #2000 (Review): add conversion support for table and table handle parameters
03/20/2013
- 06:55 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- OK, this testcase is close to being done. It shows the really insane variety of possibilities with the truly dynamic...
- 06:05 PM Database Feature #1668: add support for additional (non-dynamic) database attributes
- Code review 20130320a:
All looks good except QueryWrapper: the new @numBuffers@ method implementation should dele... - 05:38 PM Database Feature #1668: add support for additional (non-dynamic) database attributes
- Update vmn_upd20130320a.zip (vmn_upd20130320b.zip - conversion test) contains correction of NUM-BUFFERS()(KW_NUM_BUFF...
- 02:07 PM Database Feature #1668: add support for additional (non-dynamic) database attributes
- Costin Savin wrote:
> Tested this case, The result is a runtime error message: @"Third argument to ADD-INDEX-FIELD m... - 02:05 PM Database Feature #1668: add support for additional (non-dynamic) database attributes
- > Write a test case and see what happens if you pass in something other than "asc" or "desc" as the last parameter. D...
- 01:07 PM Database Feature #1668: add support for additional (non-dynamic) database attributes
- Costin Savin wrote:
> there are only these methods defined in the TempTable interface:
> public logical addFieldTo... - 12:37 PM Database Feature #1668: add support for additional (non-dynamic) database attributes
- For:...
- 05:54 PM Base Language Feature #1641: implement SAX XML support
- The update evl_upd20130320b.zip has been committed in bzr as 10303.
- 05:46 PM Base Language Feature #1641: implement SAX XML support
- > If it passes testing you can check it in and distribute it.
Conversion testing has been passed. I'm going to com... - 04:40 PM Base Language Feature #1641: implement SAX XML support
- If it passes testing you can check it in and distribute it.
- 03:11 PM Base Language Feature #1641: implement SAX XML support
- This update merged with bzr 10302. Starting the conversion testing.
- 02:46 PM Base Language Feature #1641: implement SAX XML support
- Code feedback: everything looks good.
The only issue is that you need to merge with the recent bzr revision. Ple... - 02:36 PM Base Language Feature #1641: implement SAX XML support
- The update includes:
1. Changing for constant types.
2. Adding history entries.
3. Fixes for misspelled words in S... - 02:38 PM Database Feature #2092: add server-side database support for several builtin functions
- Update cs_upd20130320a1.zip has passed conversion regression testing and is committed to bzr rev. 10301.
- 01:29 PM Database Feature #2092: add server-side database support for several builtin functions
- Code review 20130320a:
This one is mostly correct, except for the following:
* @Functions.minimum(Date var1, Date... - 11:55 AM Database Feature #2092: add server-side database support for several builtin functions
- Added proposed update, for the testcase it seems you can't define a field as rowid in 4gl so I modified that part.
W... - 05:35 AM Database Feature #2092: add server-side database support for several builtin functions
- Costin: the test you've uploaded doesn't work with the repository p2j_test.df schema definition. When modifying the p...
- 02:16 PM Base Language Feature #2089: INPUT function returning string value
- Yes, please do.
- 02:13 PM Base Language Feature #2089: INPUT function returning string value
- > When we search upward for KW_VALIDATE and enable implicit input testing, we must ensure that we are a descendant of...
- 02:00 PM Base Language Feature #2089: INPUT function returning string value
- When we search upward for KW_VALIDATE and enable implicit input testing, we must ensure that we are a descendant of t...
- 01:49 PM Base Language Feature #2089: INPUT function returning string value
- Note that...
- 12:26 PM Base Language Bug #2099: "ASSIGN widget." statement does not convert properly in some cases
- This case was exposed by server folder compilation, in #2068 note 172:...
- 12:08 PM Base Language Bug #2099 (Closed): "ASSIGN widget." statement does not convert properly in some cases
03/19/2013
- 07:46 PM Base Language Feature #2089: INPUT function returning string value
- 1 compile problem in Majic:...
- 03:13 PM Base Language Feature #2089: INPUT function returning string value
- The code looks good. I will put it into conversion testing here since it will be faster.
- 02:58 PM Base Language Feature #2089: INPUT function returning string value
- Not yet regression-tested.
- 09:42 AM Base Language Feature #2089: INPUT function returning string value
- > Stanislav: when will the update be ready?
I'm writing javadocs / merging it. - 09:17 AM Base Language Feature #2089: INPUT function returning string value
- Actually, this should be pretty much done at this point. Let's finish it now.
Stanislav: when will the update be ... - 07:51 AM Base Language Feature #2089: INPUT function returning string value
- Greg/Stanislav: we can make this a low priority, as after the latest schema VALEXP changes in #1620, the problematic ...
- 04:29 PM Database Feature #2092: add server-side database support for several builtin functions
- Code review 20130319a:
* common-progress.rules: changes look good, but see comment below about max/min.
* where-... - 02:29 PM Database Feature #2092: add server-side database support for several builtin functions
- Re-arranged the code , solved the reviewed problems and merged with latest updates.
Testcase used to check convers... - 01:29 AM Database Feature #2092: add server-side database support for several builtin functions
- Costin Savin wrote:
> Added merged update.
> Are there any other functions that need to be added? I saw DATETIME an... - 01:11 PM Database Feature #1668: add support for additional (non-dynamic) database attributes
- Constantin Asofiei wrote:
> For this, I think we should add a @P2JQuery.tableCount@ API which returns @integer@ and ... - 07:30 AM Database Feature #1668: add support for additional (non-dynamic) database attributes
- Eric,
> * num-buffers [KW_NUM_BUFF] (seems buggy) (I think this should map to P2JQuery.getTableCount, which already ... - 11:41 AM Base Language Feature #1641: implement SAX XML support
- > AFAIK any field declared in an interface is by default "static final" (the compiler takes care to put these modifie...
- 11:08 AM Base Language Feature #1641: implement SAX XML support
- > 1. Please make the constants all @public final static@ instead of just @public@ (which is editable, something we do...
- 09:45 AM Base Language Feature #1641: implement SAX XML support
- Code Feedback (Eugenie: you can make the final changes to the code Constantin posted):
1. Please make the constant... - 08:18 AM Base Language Feature #1641: implement SAX XML support
- Eugenie: attached you have the runtime support for this; please go ahead with the conversion support.
Greg: the co... - 08:13 AM Base Language Feature #1641: implement SAX XML support
- Eugenie, please implement the following:
These constants are recognized by the parser but we do not emit them as l... - 07:57 AM Database Bug #2093: VALEXP support in P2J
- The AST is slightly different with last revision from bzr:...
- 07:20 AM Database Bug #2093: VALEXP support in P2J
- Please check the failed case in the previous note with the latest #1620 fix, if you haven't done already so.
- 07:09 AM Database Bug #2093: VALEXP support in P2J
- Everything seems to work smooth, validation is only generated in @schema-validation@:...
- 04:03 AM Database Bug #2093: VALEXP support in P2J
- Ovidiu: with the #1620 changes, I can't see any more schema VALEXP problems in the logs (I think P2J was too aggressi...
- 07:03 AM Base Language Feature #1620: implement EXTENT parameter support
- Attached merged update, committed to bzr revision 10298.
- 07:02 AM Base Language Feature #1620: implement EXTENT parameter support
- > If it passes conversion testing AND if you deem the Majic changes to be correct, then go ahead and check in/distrib...
- 04:22 AM Conversion Tools Bug #2066: brew adds unnecessary comma in method parameter list after comment
- wrong task
- 04:12 AM Conversion Tools Bug #2066: brew adds unnecessary comma in method parameter list after comment
- wrong task
- 04:06 AM Conversion Tools Bug #2066: brew adds unnecessary comma in method parameter list after comment
- wrong task
03/18/2013
- 06:49 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- My changes do not work properly. Interestingly, even Majic has some pretty crazy combinations like this:
def var ... - 05:35 PM Base Language Feature #2089: INPUT function returning string value
- Understood.
- 05:27 PM Base Language Feature #2089: INPUT function returning string value
- > Also: this fuzzy matching may have bad side effects on other places we use expressionType(). For this reason, I th...
- 03:36 PM Base Language Feature #2089: INPUT function returning string value
- Yes, that is my point. getSignature(int, int, String[]) should be modified to return the default as the first one de...
- 03:13 PM Base Language Feature #2089: INPUT function returning string value
- > In regard to your idea about a default signature, I am OK with that approach. How about this: if the signature mat...
- 03:07 PM Base Language Feature #2089: INPUT function returning string value
- Greg Shah wrote:
> OK, you have to some some additional things here.
>
> 1. Before you set the implicit_input, yo... - 02:59 PM Base Language Feature #2089: INPUT function returning string value
- OK, you have to some some additional things here.
1. Before you set the implicit_input, you must first know the de... - 02:14 PM Base Language Feature #2089: INPUT function returning string value
- > Please post the code you are using to implement the implicit_input analysis....
- 01:50 PM Base Language Feature #2089: INPUT function returning string value
- In regard to your idea about a default signature, I am OK with that approach. How about this: if the signature match...
- 01:34 PM Base Language Feature #2089: INPUT function returning string value
- Please post the code you are using to implement the implicit_input analysis.
- 01:16 PM Base Language Feature #2089: INPUT function returning string value
- > What is the problem with the use of ParmType.BDT (BaseDataType) as the 1st parameter (the way it was originally cod...
- 12:19 PM Base Language Feature #2089: INPUT function returning string value
- What is the problem with the use of ParmType.BDT (BaseDataType) as the 1st parameter (the way it was originally coded)?
- 11:25 AM Base Language Feature #2089: INPUT function returning string value
- Signature of the @LENGTH@ function stored into @SignatureHelper.map_func@ map is @{ ParmType.BDT, ParmType.CHAR }@. A...
- 04:28 PM Conversion Tools Feature #1603: enhance name conversion to support internal procedure names that contain a '.' cha...
- The evl_upd20130318a.zip has been committed in bzr as 10296. As I note before it passed the conversion testing.
- 04:08 PM Conversion Tools Feature #1603: enhance name conversion to support internal procedure names that contain a '.' cha...
- Check it in and distribute it.
- 02:11 PM Conversion Tools Feature #1603: enhance name conversion to support internal procedure names that contain a '.' cha...
- The update 20130318a has passed the conversion testing, compared directories are /ddl and /src/aero/timco/majic.
- 12:13 PM Conversion Tools Feature #1603: enhance name conversion to support internal procedure names that contain a '.' cha...
- The new update with scoping support has been added for review. The case when we has separate handling for button, rec...
- 11:40 AM Conversion Tools Feature #1603: enhance name conversion to support internal procedure names that contain a '.' cha...
- Yes, frames are scoped too.
- 10:26 AM Conversion Tools Feature #1603: enhance name conversion to support internal procedure names that contain a '.' cha...
- I'm testing the implemented approach with dictionary usage. And I have a question. Do we need the same code to be imp...
- 03:39 PM Database Feature #2080: add runtime support for database methods
- Update vmn_upd20130318a.zip (vmn_upd20130318b.zip - tests) contains runtime support for BUFFER-COMPARE and some prepa...
- 12:30 PM Base Language Feature #1620: implement EXTENT parameter support
- This makes sense. I should have read the surrounding code more carefully and I would have seen this.
If it passes... - 11:58 AM Base Language Feature #1620: implement EXTENT parameter support
- > My only question is this: the schema_validation.rules includes support for the INSERT statement, but then the FORMA...
- 11:52 AM Base Language Feature #1620: implement EXTENT parameter support
- Code Review:
My only question is this: the schema_validation.rules includes support for the INSERT statement, but ... - 10:42 AM Base Language Feature #1620: implement EXTENT parameter support
- New version of the update in which:
# the NO-VALIDATE clause with a frame must affect only the schema validation exp... - 11:36 AM Database Bug #2097: FOR EACH with multiple, nested CAN-FINDs converts badly
- This issue stems from note 127 in #2068.
The following code:... - 11:16 AM Database Bug #2097 (Closed): FOR EACH with multiple, nested CAN-FINDs converts badly
- 08:23 AM Database Feature #2092: add server-side database support for several builtin functions
- Added merged update.
Are there any other functions that need to be added? I saw DATETIME and DATETIME-TZ which fit t... - 06:34 AM Database Feature #2092: add server-side database support for several builtin functions
- After some tests it seems that min and max can be applied also to character and date variables, not only numeric valu...
- 04:31 AM Database Feature #2092 (Review): add server-side database support for several builtin functions
- 04:31 AM Database Feature #2092 (WIP): add server-side database support for several builtin functions
- 12:12 AM Database Bug #2059 (Closed): CAN-FIND clause for a perm table in the RECORD_PHRASE of a temp table, with a...
03/17/2013
- 06:11 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- This set of changes resolves the problems in note 135 (literals.rules, ECW) and 139 (common-progress.rules). I also ...
- 04:07 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Update for note 142 is at #2068, note 133
- 04:34 PM Base Language Feature #1620: implement EXTENT parameter support
- I'm OK with the changes. If all of the Majic modifications appear to be correct, we will go ahead with runtime testin...
- 11:16 AM Base Language Feature #1620: implement EXTENT parameter support
- There are lots of changes in MAJIC converted code, here are some examples:
# for extent fields, all elements get the... - 10:03 AM Base Language Feature #1620: implement EXTENT parameter support
- Renamed test file.
- 10:01 AM Base Language Feature #1620: implement EXTENT parameter support
- When fixing conversion of schema validation expressions for extent fields, it hit me that the P2J approach using SCHE...
- 03:42 PM Database Bug #2077 (Closed): fix converted table and field name collisions
- 03:32 PM Database Bug #2084 (Review): Conversion misplaced parameter for can-find statement
- 03:31 PM Database Bug #2084: Conversion misplaced parameter for can-find statement
- The net effect of the cs_upd20130314a.zip update is that the correct Java output is produced, but the change still le...
03/16/2013
- 04:40 PM Conversion Tools Bug #2066: brew adds unnecessary comma in method parameter list after comment
- There is a problem when the comment is emitted after the peer:...
- 02:06 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- 16a.zip was committed to bzr revision 10292.
- 02:00 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Looks good. Check it in and distribute it.
- 01:40 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- This update changes the ECW (by moving the ATTR_HANDLE test out of the simpleExpressionType) and the javadocs (I touc...
- 01:20 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- I had the impression that this was fixed:...
- 10:31 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- I will take notes 135 and 139.
- 10:25 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Code Review (for all of the updates including 15c):
I am generally fine with the changes. There are some formatti... - 11:40 AM Conversion Tools Feature #1603: enhance name conversion to support internal procedure names that contain a '.' cha...
- This is the reason we use the dictionary. The example in note 33 has 2 scopes. It will result in a parameter for th...
- 10:24 AM Database Feature #2092: add server-side database support for several builtin functions
- Added proposed update,
For ROWID I didn't knew how a ROWID data type it's represented in the database so I assumed ...
03/15/2013
- 05:57 PM Conversion Tools Feature #1603: enhance name conversion to support internal procedure names that contain a '.' cha...
- ...
- 05:07 PM Conversion Tools Feature #1603: enhance name conversion to support internal procedure names that contain a '.' cha...
- Ok, I just lost a comment due to "redmine wants me to login again" so here is the short part. Your approach is too ag...
- 04:04 PM Conversion Tools Feature #1603: enhance name conversion to support internal procedure names that contain a '.' cha...
- > What troubles you? The fact that the procedure's parameter gets promoted and collides with another proc's parameter...
- 11:25 AM Conversion Tools Feature #1603: enhance name conversion to support internal procedure names that contain a '.' cha...
- > The question is do we need to ignore 4GL case sensitivity?
Please test your case in 4GL environment, and you will... - 11:19 AM Conversion Tools Feature #1603: enhance name conversion to support internal procedure names that contain a '.' cha...
- The remaining conversion issue with recent changes is related to the name case sensitivity issue. Consider the follow...
- 05:13 PM Database Feature #2080: add runtime support for database methods
- Constantin's advice has gotten you past the immediate handle problem, but please re-read note 9. The runtime impleme...
- 05:11 PM Database Feature #2080: add runtime support for database methods
- Yes, it works, thank you
- 04:53 PM Database Feature #2080: add runtime support for database methods
- Vadim, I don't think the runtime for the CREATE BUFFER statement is implemented. Try this instead:...
- 04:41 PM Database Feature #2080: add runtime support for database methods
- It seems, ...
- 12:09 PM Database Feature #2080: add runtime support for database methods
- update [vmn_upd20130314a.zip]
Refs #2080, checked in as bzr revision 10291.
Corrected runtime for BUFFER-COPY (KW_B... - 11:52 AM Database Feature #2080: add runtime support for database methods
- > Constantin: can you just pick up common-progress.rules from bzr and merge your quoter changes in?
OK, I'll wait f... - 11:49 AM Database Feature #2080: add runtime support for database methods
- Vadim: the 20130314a update looks good. Please go ahead and commit it to bzr.
Constantin: can you just pick up ... - 11:39 AM Database Feature #2080: add runtime support for database methods
- Constantin Asofiei wrote:
> Vadim, have you done the changes at notes #29? If yes, please upload just the common-pro... - 11:37 AM Database Feature #2080: add runtime support for database methods
- Vadim, have you done the changes at notes #29? If yes, please upload just the common-progress.rules file. If not, I c...
- 11:08 AM Database Feature #2080: add runtime support for database methods
- Yes, the pairs parameter is in use.
- 11:00 AM Database Feature #2080: add runtime support for database methods
- What about pairs in RecordBuffer.compare()? Should it be supported?
- 10:39 AM Database Feature #2080: add runtime support for database methods
- Vadim Nebogatov wrote:
> I am implementing now RecordBuffer.compare(Buffer buffer, character mode, character except,... - 08:46 AM Database Feature #2080: add runtime support for database methods
- I am implementing now RecordBuffer.compare(Buffer buffer, character mode, character except, character pairs) similar...
- 03:58 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Another case which does not convert properly:...
- 03:38 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Another update, built on top of 15b:
# fix a NPE during conversion
# fix the MINIMUM/MAXIMUM functions, when used w... - 12:25 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Attached update is on top of 15a.zip and adds:
# field assignment, when the field is date/datetime/datetimetz, to al... - 11:59 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Another case of character c'tor derail:...
- 10:54 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Another weird issue which needs to be handled by the ECW. Following code is valid 4GL:...
- 10:24 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- This update fixes:
# HANDLE attribute passed to method/function calls as a parameter (needs to be wrapped in a handl... - 02:40 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- > Also, please do check in your 14b fix if/when it is OK.
Committed to bzr revision 10290, as the server folders conv... - 08:13 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- I've been looking at the chaining problem. We have a model that is already pretty close. The OO features provide ch...
- 01:27 PM Database Bug #2095: FIELDS/EXCEPT record phrase options not honored
- BTW, since multi-table queries also implement P2JQuery indirectly, they should throw UnsupportedOperationException fo...
- 01:20 PM Database Bug #2095: FIELDS/EXCEPT record phrase options not honored
- The runtime API will look like:...
- 12:41 PM Database Bug #2095: FIELDS/EXCEPT record phrase options not honored
- The use cases we must support:...
- 12:16 PM Database Bug #2095: FIELDS/EXCEPT record phrase options not honored
- Paraphrased from #2068:
Looks like P2J doesn't like the FIELDS clause, because with it, the fields get emitted as ... - 12:12 PM Database Bug #2095 (Closed): FIELDS/EXCEPT record phrase options not honored
- 12:30 PM Database Bug #2084: Conversion misplaced parameter for can-find statement
- Eric: I've put this through conversion regression testing (which has passed) plus server folder conversion, and I don...
- 12:11 PM Database Bug #2094 (Closed): OUTER-JOIN record phrase option not honored
- 12:06 PM Database Bug #2093: VALEXP support in P2J
- This issue is dedicated to note 80 from Issue #2068.
> There is this validation expression for the @dvofrsts.dvofr... - 11:57 AM Database Bug #2093 (Closed): VALEXP support in P2J
- 02:06 AM Database Feature #2092: add server-side database support for several builtin functions
- The following builtin functions are used within where clauses in queries and thus need to be supported as database se...
- 01:14 AM Database Feature #2092 (Closed): add server-side database support for several builtin functions
03/14/2013
- 05:44 PM Database Feature #2080: add runtime support for database methods
- What methods are priority for now?
- 05:43 PM Database Feature #2080: add runtime support for database methods
- Attached update vmn_upd20130314a.zip (vmn_upd20130314b.zip - tests).
This update is replacement vmn_upd20130312a.z... - 03:12 PM Database Feature #2080: add runtime support for database methods
- for note 22:
> Constantin: you didn't already do this, right?
No, I have not changed the @Buffer.findXXXX@ APIs
for ... - 05:07 PM Base Language Feature #2089: INPUT function returning string value
- ...
- 09:55 PM Base Language Feature #2089: INPUT function returning string value
- I have an update posted in #2053. It is in testing, but hopefully you can use it to test your changes.
- 04:55 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Greg Shah wrote:
> In regard to note 116, I wonder where the extra wrapping is coming from. The h:label case is an A... - 04:39 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- OK. Test your change and check it in if it is OK.
The parser is the best solution for this case. Yes, COLON (and... - 04:31 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Greg Shah wrote:
> 14a causes the issue. I don't have the 14b applied. The testcase is checked in as well.
With 1... - 04:19 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- 14a causes the issue. I don't have the 14b applied. The testcase is checked in as well.
But if we are going to f... - 04:14 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Do you mean 14a or 14b? The taks is #1920.
- 04:13 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Your recent update also causes this issue:...
- 04:12 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- PS: sorry, the JAST is the wrong one, I'll update the previous note shortly.
- 04:10 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- The JAST at the time of the move_params is called, for the @h:buffer-field(ch):buffer-value@ part of the note 120's t...
- 04:03 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- I don't see a problem with this, but honestly, I don't really fully understand the code in move_params. If it works,...
- 03:59 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- This update fixes case at note 120. I'm putting this back into conversion regression testing and into server folder c...
- 03:57 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- In regard to note 116, I wonder where the extra wrapping is coming from. The h:label case is an ATTR_CHAR so there i...
- 03:48 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Urgh... I should have waited 5 more minutes. My ca_upd20130314a.zip update regresses this test, during conversion (LE...
- 03:40 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- > In regard to your update in note 115, check it in and distribute it.
Committed to bzr revision 10288. - 03:36 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- The problem is that it wraps all parameters passed to the @concat@ API call, as in concat(new character(*"<", handle....
- 03:34 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- In regard to your update in note 115, check it in and distribute it.
In regard to note 116, I don't see where the ... - 03:23 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Greg,
I've started per-folder conversion of the server project and I got this regression:... - 03:14 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- The attached update has passed conversion regression testing.
- 02:11 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- None at all. To make it generate instance method calls, only a few changes are needed in methods_attributes.rules (to...
- 01:52 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Remind me again: how much of the inversion problem is due to the fact that we use statics in handle for unwrapping in...
- 01:51 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- The ges_upd20130314d.zip passed conversion testing and checked in as bzr revision 10287.
- 01:49 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- With this in mind, note 93 looks like an effect of what we've discussed in the last 3 notes. I'll put the fix at note...
- 01:46 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- > And AFAIK createPeerAst sets the peerid for the current processed node (the this node), right?
Yes.
> I think... - 01:44 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Hmm... I see now what you mean. Note that in methods_attributes.rules, when we convert a METH_ node, the @this@ in th...
- 01:33 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- The peerid that is added at the STRING node is done after any wrapping would have to take place. I am more concerned...
- 01:24 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- LE: I see the problem, is related to the fact that when move_params is called by methods_attributes.rules in the seco...
- 01:08 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- About note 93. See these two ASTs:...
- 12:56 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- This version fixes both the regression in Majic (hopefully in all cases) as well as the super(?, ?, ?) case. It reso...
- 11:55 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- > > 2. in expressions.rules:308 - I'm not sure why you are thinking to remove the WHEN. Can you share the case you ha...
- 11:44 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Good feedback.
> 2. in expressions.rules:308 - I'm not sure why you are thinking to remove the WHEN. Can you share... - 11:05 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Review results:
1. This comment in base_structure.xml:379... - 11:03 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- > Unless I read the diff wrong, the new way is the way of eliminating the getScreenValue and using the widget getter....
- 11:00 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- > 1. The new way is more correct?
> 2. The old way may be working (or may not) depending on the format of the var?... - 10:53 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Questions:
1. The new way is more correct?
2. The old way may be working (or may not) depending on the format o... - 10:44 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- > 2. Detection of incompatible types in the comparison operators (which take BDT). A standalone testcase:
> def va... - 10:22 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Here is the final version, with some fixes necessary to convert Majic. Please see #2091 for a Majic change that must...
- 10:01 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- > Constantin: would you please do a code review of this update?
I'm looking at it now. - 07:57 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- I have fixed some issues (found in my manual testing) with the update.
The conversion testing didn't ever complete... - 09:54 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- This update (hopefully) resolves everything left on this task except for the issue mentioned in note 93. Here is a s...
- 04:38 PM Conversion Tools Feature #1603: enhance name conversion to support internal procedure names that contain a '.' cha...
- OK. I'll investigate the ScopedSymbolDictionary keeping approach. The next drop implements the Map usage and some min...
- 03:52 PM Conversion Tools Feature #1603: enhance name conversion to support internal procedure names that contain a '.' cha...
- > The conversion test is the comparison for MAJIC converted result before and after update. Is it correct?
Yes.
... - 03:08 PM Conversion Tools Feature #1603: enhance name conversion to support internal procedure names that contain a '.' cha...
- > There are many diffs in Majic. Please review the following and determine if these are correct or not.
I think be... - 12:41 PM Conversion Tools Feature #1603: enhance name conversion to support internal procedure names that contain a '.' cha...
- There are many diffs in Majic. Please review the following and determine if these are correct or not.
- 10:49 AM Conversion Tools Feature #1603: enhance name conversion to support internal procedure names that contain a '.' cha...
- > Do you want me to conversion test this on devsrv01?
Yes, please run this on devsrv01. - 10:36 AM Conversion Tools Feature #1603: enhance name conversion to support internal procedure names that contain a '.' cha...
- Do you want me to conversion test this on devsrv01? It is much faster than lightning and will not impact the conversi...
- 10:31 AM Conversion Tools Feature #1603: enhance name conversion to support internal procedure names that contain a '.' cha...
- > 1. In both files, please start with namesfx = 2. I think this is more natural for the 2nd instance of that var.
... - 08:22 AM Conversion Tools Feature #1603: enhance name conversion to support internal procedure names that contain a '.' cha...
- Code Review:
Overall, it is very good. Some minor things:
1. In both files, please start with namesfx = 2. I ... - 03:46 PM Database Bug #2084: Conversion misplaced parameter for can-find statement
- Went back and searched for another solution, since the last one started from the wrong premise that can-find BUFFER_S...
- 03:35 PM Database Bug #2084: Conversion misplaced parameter for can-find statement
- Costin, please provide a detailed update of what you have found/done on this today.
- 12:04 PM Base Language Bug #2065: ambiguity in date/decimal literals that is not handled properly by the ProgressLexer
- Committed to bzr revision 10285.
- 10:27 AM Base Language Bug #2065: ambiguity in date/decimal literals that is not handled properly by the ProgressLexer
- The update looks good. You can check it in and distribute it.
- 09:56 AM Base Language Bug #2065: ambiguity in date/decimal literals that is not handled properly by the ProgressLexer
- Correct, good letter, wrong date. Attached is the good one.
- 08:13 AM Base Language Bug #2065: ambiguity in date/decimal literals that is not handled properly by the ProgressLexer
- I think the wrong update is posted here.
03/13/2013
- 06:14 PM Base Language Feature #2089: INPUT function returning string value
- About note 8: no.
You can do the following:
1. Determine the type (wrapper classname: like "integer") of the va... - 06:00 PM Base Language Feature #2089: INPUT function returning string value
- Yes, you should work on item 2 and my changes will enable item 1. Please note that I don't want to transform all val...
- 04:42 PM Base Language Feature #2089: INPUT function returning string value
- In regards to #2: should I parent all occurrences of a validated value with FUNC_POLY node as a solution?
- 02:12 PM Base Language Feature #2089: INPUT function returning string value
- OK, I missed that. So your question remains.
- 02:11 PM Base Language Feature #2089: INPUT function returning string value
- Greg was saying that "I am currently working to extend that facility to be more robust, including detecting the case ...
- 01:53 PM Base Language Feature #2089: INPUT function returning string value
- Stanislav Lomany wrote:
> So, am I correct that except checking the runtime part we have two problems:
> 1. It is n... - 01:47 PM Base Language Feature #2089: INPUT function returning string value
- So, am I correct that except checking the runtime part we have two problems:
1. It is not properly recognized that a... - 06:06 PM Database Feature #2080: add runtime support for database methods
- Vadim Nebogatov wrote:
> Will be ast processing with rules also required with wrapping String to character?
Const... - 04:49 PM Database Feature #2080: add runtime support for database methods
- Eric Faulhaber wrote:
> With your next update, please replace the predicate data type of @String@ in all the @Buff... - 03:23 PM Database Feature #2080: add runtime support for database methods
- Vadim Nebogatov wrote:
> Eric Faulhaber wrote:
> > One question: why not just implement @AbstractQuery.queryReposi... - 03:18 PM Database Feature #2080: add runtime support for database methods
- Low priority: related to my last note, we are using the wrong data type for the NO-LOBS parameter in our bufferCopy/...
- 03:16 PM Database Feature #2080: add runtime support for database methods
- Eric Faulhaber wrote:
> One question: why not just implement @AbstractQuery.queryReposition(NumberType n)@ as
> [.... - 02:07 PM Database Feature #2080: add runtime support for database methods
- The current project never uses the NO-LOBS parameter of BUFFER-COPY and BUFFER-COMPARE, so the variants of @BufferImp...
- 01:46 PM Database Feature #2080: add runtime support for database methods
- Code review 20130312a:
Update looks fine.
Good catch on the @queryReposition@ method. One question: why not j... - 11:10 AM Database Feature #2080: add runtime support for database methods
- I suspect that much of the use we will see of this feature will involve expressions resolving to character values (si...
- 05:40 AM Database Feature #2080: add runtime support for database methods
- The FIND-FIRST/LAST/UNIQUE methods can receive a character for the first parameter. Currently, the Buffer.find* imple...
- 05:48 PM Conversion Tools Feature #1603: enhance name conversion to support internal procedure names that contain a '.' cha...
- The suggested fix has been uploaded. It properly converts the sample:...
- 02:06 PM Conversion Tools Feature #1603: enhance name conversion to support internal procedure names that contain a '.' cha...
- The places to modify:
for frames: rule file annotation/frame_scoping.rules, internal function "process_frame"... - 10:17 AM Conversion Tools Feature #1603: enhance name conversion to support internal procedure names that contain a '.' cha...
- The strange thing is that I thought we already did some of this kind of disambiguation.
We don't want this to be s... - 10:09 AM Conversion Tools Feature #1603: enhance name conversion to support internal procedure names that contain a '.' cha...
- Another word we need to decide what are the exact cases when we need special processing for "-" character. For now I ...
- 10:02 AM Conversion Tools Feature #1603: enhance name conversion to support internal procedure names that contain a '.' cha...
- Another name collision, when vars are used:...
- 07:55 AM Conversion Tools Feature #1603: enhance name conversion to support internal procedure names that contain a '.' cha...
- If we have a substitution that we want to be the default, then we would always want to encode it in P2J and not in th...
- 04:51 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Constantin: this case:...
- 01:34 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- The complete list is this (logged from where the ECW.expressionType would throw exception otherwise):...
- 01:30 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- > When do you think you will have this available?
The server conversion is in the Base Structure phase, with a few f... - 01:14 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- > What I can do is remove the last fallback checks (the _POLY-related checks ), adjust the ECW to log a message inste...
- 10:44 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- This is the good version, committed to bzr revision 10284.
LE: it passed conversion regression testing. - 10:34 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Yes, you understand correctly. The idea here was this: for the KW_DYN_FUNC node, we don't need to the ControlFlowOps....
- 10:32 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- I guess what is happening is that casttype is really meant to be used for the .class emit on a DYNAMIC-FUNCTION invoc...
- 10:24 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- I don't fully understand the purpose of the "ignorecast" annotation:...
- 10:01 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- OK, good idea.
- 10:00 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- How about this: if these cases are already known to return logical, then I will emit the logical.class and not wrap ...
- 09:55 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- > In regard to the removal of the castLogical(): currently we bypass the emit of the .class instance in this case (wh...
- 09:49 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- In regard to the removal of the castLogical(): currently we bypass the emit of the .class instance in this case (when...
- 09:25 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- > I think the only issue here is where the new character() wrapper is emitted.
I think you are right.
> If I unders... - 09:10 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Here is the code that is generated by my current implementation:...
- 08:52 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Had a problem, forgot to initialize the @ref@ var in base_structure.xml's @parent.type == prog.kw_run@ run (without t...
- 08:28 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Greg Shah wrote:
> I think there should be some code like this in the example:
The code should be @h = temp-table t... - 08:23 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- I think there should be some code like this in the example:...
- 07:39 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- And the update...
- 07:38 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- The attached update fixes this:
# wrapping of handle for a @RUN ... IN DYNAMIC-FUNCTION@ call
# varargs signature pro... - 06:17 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Greg, in the server project there are failures related to the fact that _POLY is used as an operand. The failed opera...
- 03:15 PM Base Language Bug #2065: ambiguity in date/decimal literals that is not handled properly by the ProgressLexer
- The update has passed conversion regression testing.
- 02:33 PM Base Language Bug #2065: ambiguity in date/decimal literals that is not handled properly by the ProgressLexer
- The only remaining problem was with char field initializers, which were not handled (it was parsers fault). The notes...
- 03:13 PM Database Bug #2084: Conversion misplaced parameter for can-find statement
- Tried to replicate the test which failed on update cs_upd20130312a.zip (-where- before I moved the logic from post-ru...
- 11:09 AM Database Bug #2084: Conversion misplaced parameter for can-find statement
- The testcase from note 11 passed, there was another file from conversion regression tests that failed (with the remap...
- 10:53 AM Database Bug #2084: Conversion misplaced parameter for can-find statement
- I haven't looked at this issue in detail nor tried to reproduce the problem yet, but the last update you sent looks w...
- 08:11 AM Database Bug #2084: Conversion misplaced parameter for can-find statement
- There seems to be a problem with some buffers being used in different db schemas,
after the change it seems the buff... - 05:25 AM Database Bug #2084: Conversion misplaced parameter for can-find statement
- There are changes in two files with this update, I think they are expected, but Costin is checking them now.
- 02:14 AM Database Bug #2090 (Closed): correct sorting of DMO properties during schema conversion
03/12/2013
- 06:43 PM Conversion Tools Feature #1603: enhance name conversion to support internal procedure names that contain a '.' cha...
- OK. I've found the answer for note #13. The matchlist.xml config file overrides the NameConverter.java internals. The...
- 06:17 PM Conversion Tools Feature #1603: enhance name conversion to support internal procedure names that contain a '.' cha...
- The question. Why adding replacement:...
- 05:20 PM Database Feature #2080: add runtime support for database methods
- Current status of methods from ref 1947
Implemented:
GET-CURRENT (KW_GET_CUR)
FIND-LAST (KW_FIND_LST)
FIND-FIRST ... - 05:19 PM Database Feature #2080: add runtime support for database methods
- Attached update vmn_upd20130312a.zip.
Corrected runtime for BUFFER-COPY (KW_BUF_COPY) and BUFFER-COMPARE (KW_BUF_CO... - 04:40 PM Database Feature #2080: add runtime support for database methods
- Yes, I handed my changes off to Constantin a few days ago, based on an older version, to integrate with some other ch...
- 04:26 PM Database Feature #2080: add runtime support for database methods
- Eric, I corrected BufferImpl.bufferCompare according #12, but noticed that you replaced function body with ... Unsupp...
- 06:22 AM Database Feature #2080: add runtime support for database methods
- Tests
uast/find-with-locks.p
uast/get-with-locks.p
committed revision 991.
- 05:01 AM Database Feature #2080: add runtime support for database methods
- vmn_upd20130311c.zip checked in as bzr revision 10278.
- 08:58 PM Database Feature #2080: add runtime support for database methods
- Update vmn_upd20130311c.zip has passed conversion regression testing. Please commit it to bzr and distribute. Note ...
- 04:05 PM Base Language Feature #2089: INPUT function returning string value
- As Constantin notes, the INPUT function in the 4GL can return either the actual data type of the widget OR it can ret...
- 03:46 PM Base Language Feature #2089: INPUT function returning string value
- Greg, how should correctly converted validation code look like for the following example?...
- 03:12 PM Base Language Feature #2089: INPUT function returning string value
- Copied from #2068:
*CA*
Another problem found in at least rtb/ folder:... - 03:09 PM Base Language Feature #2089 (Closed): INPUT function returning string value
- 03:37 PM Database Bug #2083: client-side where-clause leak in FindQuery
- This fix looks good to me. I have merged it with the latest bzr revision, re-run conversion regression testing with ...
- 01:36 PM Database Bug #2083: client-side where-clause leak in FindQuery
- I see you found another solution.
I also found one simple and straightforward solution by refining the condition f... - 12:19 PM Database Bug #2083: client-side where-clause leak in FindQuery
- Ovidiu, I was having many similar issues with client-side query substitution parameters and CAN-FIND while fixing #20...
- 02:33 PM Database Bug #2084: Conversion misplaced parameter for can-find statement
- Added proposed update
It seems the problem is in some situation the remap scope is deleted before we get into the <p... - 11:28 AM Database Bug #2084: Conversion misplaced parameter for can-find statement
- It seems there are still some weird cases where it fails. Working on reproducing and fixing it
- 04:54 AM Database Bug #2084: Conversion misplaced parameter for can-find statement
- I'm sorry about that, it seems I forgot to save after doing the clean-up, and created the archive with some unwanted ...
- 09:54 PM Database Bug #2084: Conversion misplaced parameter for can-find statement
- Code review 20130311a:
* Do you have a test case that illustrates the problem described by Constantin in notes 5 a... - 02:08 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Attached was committed to bzr revision 10282.
- 01:55 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- OK, please go ahead and check it in. You can leave the assignments.rules there. I will deal with the merge.
> Wh... - 01:49 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- PS: the conversion regression testing has passed. The only changes are related to:...
- 01:44 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- > The code looks fine. I have changes in the assignments.rules and I prefer if you don't include that file (it is ju...
- 01:28 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- The code looks fine. I have changes in the assignments.rules and I prefer if you don't include that file (it is just...
- 12:48 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- > That makes sense. Are you going to add that?
No, I don't want to add this now.
> In a related note: my changes... - 12:31 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- That makes sense. Are you going to add that?
In a related note: my changes are ready, but I want to test them wit... - 12:15 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- I don't recall seeing "attributes set to unknown"-related errors, but I think we should consider handling the attribu...
- 09:16 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- OK, I'll fold that into my changes. It will make the result much cleaner.
In regard to your changes, that is grea... - 08:39 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Yes, we can get rid of castLogical and use new logical() instead.
I have an update for which I still need to make ... - 02:06 PM User Interface Feature #2088: add HIDE STREAM statement support
- Committed to bzr revision 10281.
- 01:58 PM User Interface Feature #2088: add HIDE STREAM statement support
- This code is fine. It can be checked in without conversion regression testing.
- 01:17 PM Database Bug #2059: CAN-FIND clause for a perm table in the RECORD_PHRASE of a temp table, with a FOR EACH...
- Update has passed conversion regression testing and is committed to bzr rev. 10280.
- 12:14 PM Database Bug #2059: CAN-FIND clause for a perm table in the RECORD_PHRASE of a temp table, with a FOR EACH...
- Fix candidate is attached. I am conversion regression testing this now.
- 10:15 PM Database Feature #2000: add conversion support for table and table handle parameters
- Code looks good, AFAICT, but without test cases and being able to look at the ASTs, the changes were hard to follow, ...
03/11/2013
- 07:34 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Do we really need BDT.castLogical()? Or can we handle this using new logical(BDT)? If we can avoid the castLogical(...
- 12:06 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- > In these cases where ptypes[i] is null, I think it default to BaseDataType, right?
Are you asking what the type ... - 10:50 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- > 1. It seems to me that ParmType.typesMatch() should implement a wider range of matching behavior. For example, if ...
- 10:21 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Code Review:
1. It seems to me that ParmType.typesMatch() should implement a wider range of matching behavior. Fo... - 08:36 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- The case automatically assigns the lower 32-bits of the long to the int. Since oldtype can never be > 2GB, this is s...
- 08:05 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- > Yes, that was my fix too for Java 1.6. And I don't think there will be losses, as the token type doesn't need 32 bi...
- 06:28 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- This update contains some fixes in ECW.parameterTypeName and base_structure.xml, plus BUFFER-COPY/COMPARE (from Eric)...
- 05:25 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Yes, that was my fix too for Java 1.6. And I don't think there will be losses, as the token type doesn't need 32 bits...
- 05:22 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- > Eugenie: what Java are you using ? I haven't got any error on my system or devsrv01.
1.6.X
The long 64 bit ca... - 04:30 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Eugenie: what Java are you using ? I haven't got any error on my system or devsrv01.
- 06:30 PM Database Feature #2080: add runtime support for database methods
- Code review 20130311c:
The merge looks fine. The implementation of @BufferImpl.bufferCompare@ doesn't look right. ... - 05:00 PM Database Feature #2080: add runtime support for database methods
- Attached update
vmn_upd20130311c.zip
vmn_upd20130311d.zip (corresponding test cases)
is next replacement of vm... - 04:26 PM Database Feature #2080: add runtime support for database methods
- P2JQuery,
** 040 CA 20130311 Added bufferHandle(integer).
It seems wrong number - 03:08 PM Database Feature #2080: add runtime support for database methods
- Code review 20130311a:
The changes all look good.
I noticed on this second review that even the no-arg versions... - 10:01 AM Database Feature #2080: add runtime support for database methods
- Attached update
vmn_upd20130311a.zip
vmn_upd20130311b.zip (corresponding test cases)
is replacement of vmn_upd2013... - 06:20 PM Database Feature #2000: add conversion support for table and table handle parameters
- Regression conversion completed without errors.
- 03:42 PM Database Feature #2000: add conversion support for table and table handle parameters
- > Is this merged up to the latest bzr revision?
Yes, it is merged. - 02:59 PM Database Feature #2000: add conversion support for table and table handle parameters
- Is this merged up to the latest bzr revision?
If so, please do run conversion regression testing. Eric will do a ... - 02:32 PM Database Feature #2000: add conversion support for table and table handle parameters
- Update for TABLE-HANDLE FOR case. Should I regression-test it?
- 03:32 PM Database Bug #2083: client-side where-clause leak in FindQuery
- The @<action>sibref.setHidden(true)</action>@ from @where_clause_post.rules@ at line 438 from my previous update seem...
- 11:24 AM Database Bug #2083: client-side where-clause leak in FindQuery
- After some more tests, it seems like this update does not fix the issue, in fact it is too restrictive, it will cut m...
- 06:07 AM Database Bug #2083: client-side where-clause leak in FindQuery
- With help from Constantin, the missing @setHidden@ is now in place.
- 01:46 PM Database Bug #2084: Conversion misplaced parameter for can-find statement
- Added proposed update which checks if the javaname is marked for remap but not as a generated increment name.
- 11:09 AM Base Language Feature #1640: implement DOM XML support
- Added more missing APIs. Committed to bzr revision 10276
- 11:05 AM Base Language Feature #1884: add some of the v10 data types and core built-ins
- -Added more missing APIs. Committed to bzr revision 10276- was for task #1640
- 11:05 AM Database Feature #1947: add conversion support for database methods
- Added missing API bufferHandle(integer). Committed to bzr revision -10276- 10277.
03/10/2013
- 03:29 PM Database Feature #1668: add support for additional (non-dynamic) database attributes
- Update ecf_upd20130308a.zip has passed conversion regression testing and is committed as bzr rev. 10274.
- 02:36 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- > And the update for note 50.
Constantin, how you compiled the update? I put it on top of the current bzr code and... - 10:16 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- And the update for note 50.
- 10:16 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Attached update contains:
# ParmType - added needed APIs for the enum
# SignatureHelper - fixed the definition for ... - 07:30 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Eugenie: I think KW_SET_BUF should have a BDT_VARARGS instead of BDT_ARRAY, because as I see from the docs, it can re...
- 07:15 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Constantin Asofiei wrote:
> Greg Shah wrote:
> > Actually, after thinking about it some more, is there a need to ev...
03/09/2013
- 05:20 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- New drop uploaded. The function and methods are separated in two different maps.
- 04:52 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Yes, go with that approach.
- 04:30 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- > This also means we will have to pass the token type (FUNC_*/METH_*) in addition to the keyword.
Yes, we can impl... - 03:15 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- This also means we will have to pass the token type (FUNC_*/METH_*) in addition to the keyword.
- 02:54 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Eugenie Lyzenko wrote:
> > Eugenie: KW_FILL appears twice in the map.
>
> Yes, one for Built-in function, other f... - 02:34 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- > Eugenie: KW_FILL appears twice in the map.
Yes, one for Built-in function, other for method. They both have diff... - 02:27 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- > For KW_REPLACE: I see that you've specified two signatures for it, but according to 10.2B docs, it has 3 mandatory ...
- 12:01 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Greg Shah wrote:
> Actually, after thinking about it some more, is there a need to ever wrap the parameters to DYNAM... - 11:44 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Actually, after thinking about it some more, is there a need to ever wrap the parameters to DYNAMIC-FUNCTION? They w...
- 11:42 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- OK. But we could handle these cases too.
1. SUPER is especially easy to handle, because we can use the same techn... - 11:37 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- > Actually, I think the ECW already handles KW_SUPER fully. It searches up the tree to find the containing function ...
- 11:31 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Some thoughts:
> The fact that a function can have multiple signatures troubles us at conversion time, as if we do... - 11:02 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Another case of the KW_DYN_FUNC: the handle parameter is optional. I think this should also be explicitly treated by ...
- 09:40 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- There is another case which is not treated properly. When IF ternary operator has an ? on the else or then branch, as...
- 08:20 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- About KW_DATETIME:
* how can we disambiguate the signature, if the number of mandatory parameter is the same (1) in ... - 07:53 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Eugenie: KW_FILL appears twice in the map.
- 05:08 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- The fact that a function can have multiple signatures troubles us at conversion time, as if we don't offer a choice t...
- 03:57 PM Base Language Bug #2076: DMO setter fails when parameter is BaseDataType
- A simple change to ECW fixes this. I am making some other changes to the ECW as well. Then I will conversion test t...
- 11:25 AM Base Language Bug #2076: DMO setter fails when parameter is BaseDataType
- There is a regression related to this update:...
- 03:56 PM Base Language Feature #2050: ATTR_POLY/METH_POLY used as rvalue or in expressions
- Yes, I'm thinking about what is the right solution here. I wonder if I should make the previous solution (the extra ...
- 03:52 PM Base Language Feature #2050: ATTR_POLY/METH_POLY used as rvalue or in expressions
- Hmm... on a second thought, this is similar to the DMO setter case... it should wrap the _POLY/DB_NON_REF in a logica...
- 03:34 PM Base Language Feature #2050: ATTR_POLY/METH_POLY used as rvalue or in expressions
- I'm not sure the problem is in the ECW. Actually, there is never any attempt to call the ECW for these cases. If I ...
- 11:13 AM Base Language Feature #2050: ATTR_POLY/METH_POLY used as rvalue or in expressions
- The strange thing is that I already put in support for all of these cases. I guess it is not working properly. I wi...
- 07:28 AM Base Language Feature #2050: ATTR_POLY/METH_POLY used as rvalue or in expressions
- The ExpressionConversion worker needs to treat the cases when a _POLY is used in a logical test, as in:...
- 11:11 AM Base Language Feature #2087 (Closed): add support for extent vars/fields used with a PUT statement (stream-only...
- Checked in as bzr revision 10273.
- 04:44 AM Base Language Feature #2087: add support for extent vars/fields used with a PUT statement (stream-only feature)
- The ges_upd20130308g.zip update has passed conversion regression testing.
- 03:01 AM Base Language Feature #2087: add support for extent vars/fields used with a PUT statement (stream-only feature)
- Thanks, that did the trick. I'm putting it through conversion regression testing.
- 11:06 AM Base Language Bug #2065 (Closed): ambiguity in date/decimal literals that is not handled properly by the Progre...
- 11:06 AM Base Language Bug #2065: ambiguity in date/decimal literals that is not handled properly by the ProgressLexer
- Passed conversion testing and checked in as bzr revision 10272.
- 09:11 AM Database Bug #2083: client-side where-clause leak in FindQuery
- I couldn't get my mind of client_taint and I took a quick look where is used. In where_clause_post.rules, at line 413...
- 02:55 AM Base Language Feature #1640: implement DOM XML support
- Adds two missing APIs in XmlFactory, related to WIDGET-POOL clause. Committed to bzr revision 10271.
03/08/2013
- 05:25 PM Base Language Feature #2087: add support for extent vars/fields used with a PUT statement (stream-only feature)
- Take a look at this. I think this should do the same thing.
- 03:59 PM Base Language Feature #2087: add support for extent vars/fields used with a PUT statement (stream-only feature)
- The good news is that 0308h.zip has passed conversion regression testing - no changes.
- 03:47 PM Base Language Feature #2087: add support for extent vars/fields used with a PUT statement (stream-only feature)
- I had a feeling there was another file where this was done already for some other cases, but my brain chose to ignore...
- 03:41 PM Base Language Feature #2087: add support for extent vars/fields used with a PUT statement (stream-only feature)
- Code Review:
Could we have added this as an additional case in annotations/array_expansion.rules? - 02:58 PM Base Language Feature #2087: add support for extent vars/fields used with a PUT statement (stream-only feature)
- Had a regression (didn't handle the case when the PUT used a subscripted field). Back in regression testing.
- 01:31 PM Base Language Feature #2087: add support for extent vars/fields used with a PUT statement (stream-only feature)
- 4GL allows extent vars/fields to be used in a PUT statement:...
- 01:27 PM Base Language Feature #2087 (Closed): add support for extent vars/fields used with a PUT statement (stream-only...
- 05:07 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- The first complete drop to review has been uploaded.
PS: Forgot to note. The file SignatureHelper.java contains the ... - 04:06 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- > Undocumented functions?
Yes, these are undocumented.
It is our understanding that LOWER is the same thing as ... - 02:59 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Greg,
I'm preparing the final drop to upload and requesting for ability not to follow the 98 column restriction fo... - 01:38 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- The remaining questions:
The functions defined in progress.g but not in 4GL reference documentation:... - 11:18 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Add CLOB (which is really only the CLOB field case) and LARGE_CHAR_OBJECT which is either LONGCHAR or CLOB.
> SET-... - 10:39 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- > Can you provide an example? A CLOB can only be a field in the database, so I guess the method must take a reference...
- 08:35 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- > There are the methods we do not define in progress.g file:
These are from v11 which we don't support yet.
> D... - 10:04 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- The handle methods has been added. There are the methods we do not define in progress.g file:
ADD-PARENT-ID-RELATION... - 05:05 PM Base Language Bug #2065: ambiguity in date/decimal literals that is not handled properly by the ProgressLexer
- Fix for empty string used as a date var initializer. It is being conversion tested now.
- 04:30 PM Base Language Bug #2065: ambiguity in date/decimal literals that is not handled properly by the ProgressLexer
- Passed conversion testing and checked in as revision 10269.
I'll handle the date from an empty string next. - 03:38 PM Base Language Bug #2065: ambiguity in date/decimal literals that is not handled properly by the ProgressLexer
- Another issue related to dates:...
- 03:28 PM Base Language Bug #2065: ambiguity in date/decimal literals that is not handled properly by the ProgressLexer
- Final version going through conversion testing now. There was an issue with removal of zeros from the year portion o...
- 02:05 PM Base Language Bug #2065 (WIP): ambiguity in date/decimal literals that is not handled properly by the ProgressL...
- How many times am I going to be wrong today? Majic compiled fine with the c update but there are still many unwanted...
- 12:12 PM Base Language Bug #2065: ambiguity in date/decimal literals that is not handled properly by the ProgressLexer
- Another silly mistake, I didn't properly exclude num literals from the trailing zero removal. This updated version i...
- 10:56 AM Base Language Bug #2065: ambiguity in date/decimal literals that is not handled properly by the ProgressLexer
- Fixed a stupid indexing error. Going back into conversion testing.
- 10:48 AM Base Language Bug #2065: ambiguity in date/decimal literals that is not handled properly by the ProgressLexer
- Updated version of the testcase plus the prepared fixes which are going into conversion testing now.
The core solu... - 04:40 PM Database Feature #1668: add support for additional (non-dynamic) database attributes
- Fix candidate attached. I'll push this through conversion regression testing.
Vadim: please take note of this up... - 03:56 PM Database Feature #1668: add support for additional (non-dynamic) database attributes
- Eric, we've hit this on another occasion (? literal). For now, just handle both cases. The parser changes related to ...
- 03:43 PM Database Feature #1668: add support for additional (non-dynamic) database attributes
- Interesting...outside of the message statement, the NO_LOCK_LITERAL is the child of an intermediate EXPRESSION node, ...
- 03:24 PM Database Feature #1668: add support for additional (non-dynamic) database attributes
- The update doesn't handle this case:...
- 03:17 PM User Interface Feature #2088: add HIDE STREAM statement support
- The HIDE ALL statement can target a stream, as in:...
- 03:14 PM User Interface Feature #2088 (Closed): add HIDE STREAM statement support
- 02:19 PM Conversion Tools Feature #1603: enhance name conversion to support internal procedure names that contain a '.' cha...
- I'm putting this case here, because is a name collision:...
- 07:39 AM Conversion Tools Feature #1603: enhance name conversion to support internal procedure names that contain a '.' cha...
- Passed conversion regression testing, committed to bzr revision 10265.
- 04:40 AM Conversion Tools Feature #1603: enhance name conversion to support internal procedure names that contain a '.' cha...
- This converts dots in names to "dot" string. I will put this through conversion.
- 01:59 PM Database Bug #2084: Conversion misplaced parameter for can-find statement
- Sorry, I should have been more explicit. The failures are caused by the fact that, when you have a table named @prodc...
- 01:53 PM Database Bug #2084: Conversion misplaced parameter for can-find statement
- Constantin Asofiei wrote:
> Conversion regression testing has failed, as the tests above have passed because the jav... - 06:53 AM Database Bug #2084: Conversion misplaced parameter for can-find statement
- Conversion regression testing has failed, as the tests above have passed because the javaname in these cases is the s...
- 04:51 AM Database Bug #2084: Conversion misplaced parameter for can-find statement
- The changes looks good, also solves other compile errors where wrong buf name was used, in cases like:...
- 04:01 AM Database Bug #2084: Conversion misplaced parameter for can-find statement
- Added proposed update,
In previous comment I didn't understood exactly the process so I assumed there's an addition... - 12:20 PM Database Feature #2000: add conversion support for table and table handle parameters
- About the state of work: I changed the parser so TABLE-HANDLE FOR is recognized as a variable reference rather than a...
- 12:19 PM Database Bug #2083: client-side where-clause leak in FindQuery
- At this moment, I use the following simplified testcase:...
- 12:09 PM Database Bug #2083: client-side where-clause leak in FindQuery
- I'm getting closer. Here what I found:
The extra node (@f1 [FUNC_LOGICAL]@) should be made hidden so it will not b... - 09:09 AM Database Bug #2083: client-side where-clause leak in FindQuery
- In my previous note there the and operator from where clause contains a reference to a buffer other than the one that...
- 10:35 AM Database Feature #2086 (Closed): add support for the VALIDATE clause, for the temp-table's fiels defined u...
- 10:34 AM Database Feature #2086: add support for the VALIDATE clause, for the temp-table's fiels defined using LIKE...
- Committed to bzr 10266.
- 09:58 AM Database Feature #2086: add support for the VALIDATE clause, for the temp-table's fiels defined using LIKE...
- OK, this change looks fine. Commit and distribute it.
- 09:48 AM Database Feature #2086: add support for the VALIDATE clause, for the temp-table's fiels defined using LIKE...
- The update has passed conversion regression testing. There is one additional change in MAJIC, from a @DEFINE TEMP-TAB...
- 08:37 AM Database Feature #2086: add support for the VALIDATE clause, for the temp-table's fiels defined using LIKE...
- Sorry, when you say "VALIDATE clause", I thought you are referring to the option for a TEMP-TABLE (not for a specific...
- 07:27 AM Database Feature #2086: add support for the VALIDATE clause, for the temp-table's fiels defined using LIKE...
- New version of the update, which solves the VALIDATE when used with @DEFINE TEMP-TABLE tt1 LIKE source VALIDATE@. Als...
- 03:09 AM Database Feature #2086: add support for the VALIDATE clause, for the temp-table's fiels defined using LIKE...
- This fix removes the extent annotation from the source field reference in the VALEXP. What is still needed to do:
# ... - 01:41 AM Database Feature #2086: add support for the VALIDATE clause, for the temp-table's fiels defined using LIKE...
- Greg, the VALIDATE clause is used in the server project with a field definition. See file pmcon/cjcotdo0.w. There are...
- 10:35 AM Base Language Feature #1640: implement DOM XML support
- Committed to bzr revision 10267
- 10:02 AM Base Language Feature #1640: implement DOM XML support
- I'm fine with the change. Commit and distribute it.
- 09:45 AM Base Language Feature #1640: implement DOM XML support
- ca_upd20130308d.zip has passed conversion regression testing.
- 08:15 AM Base Language Feature #1640: implement DOM XML support
- Fixed conversion of CREATE statements related to XML support. The correct approach is to pass the handle instance as ...
03/07/2013
- 06:12 PM Database Feature #2086: add support for the VALIDATE clause, for the temp-table's fiels defined using LIKE...
- VALIDATE is NOT used in the server project (in a TEMP-TABLE or WORK-TABLE definition). Only NO-UNDO and RCODE-INFORM...
- 05:41 PM Database Feature #2086: add support for the VALIDATE clause, for the temp-table's fiels defined using LIKE...
- MAJIC has finished conversion and there are changes, expected: validation clauses are dropped for temp-table fields d...
- 04:55 PM Database Feature #2086: add support for the VALIDATE clause, for the temp-table's fiels defined using LIKE...
- The server project exposed a problem with the temp-table fields defined using the LIKE clause, when the target field ...
- 04:43 PM Database Feature #2086 (Closed): add support for the VALIDATE clause, for the temp-table's fiels defined u...
- 05:27 PM Conversion Tools Feature #1603: enhance name conversion to support internal procedure names that contain a '.' cha...
- Go ahead with this change.
> A question: are you sure it's safe to "eat" the "-", "_" and "/" character from the n... - 05:15 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- No, CONFIG-NAME is supposed to be an attribute. It is a bug in our parser. I have some other parser changes I'm mak...
- 05:01 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Found potential issue in progress.g:
According to Progress doc CONFIG-NAME is an attribute, not a method. But we def... - 10:44 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- > If parameter is widget we need to add something: WID mapping to GenerigWidget?
Yes, that is fine.
> Arrays ar... - 09:58 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Greg,
I'm working on the handle based methods adding. It's avout 300 lines. So I'm expecting to finish today-tomor... - 09:26 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- The update has been uploaded. All built-in functions are there except so called "fake" web-speed ones. No methods for...
- 05:00 PM Base Language Bug #2076 (Closed): DMO setter fails when parameter is BaseDataType
- 04:58 PM Base Language Bug #2076: DMO setter fails when parameter is BaseDataType
- Passes conversion testing and is checked into bzr as revision 10264.
- 04:46 PM Base Language Bug #2076: DMO setter fails when parameter is BaseDataType
- No, that is alright. The difference is not significant at runtime. Thanks.
I will check this in and distribute it. - 04:41 PM Base Language Bug #2076: DMO setter fails when parameter is BaseDataType
- > Constantin: what do you think?
The result will not affect runtime, is just some small overhead. I think this is be... - 04:32 PM Base Language Bug #2076: DMO setter fails when parameter is BaseDataType
- Conversion testing has finished. There is this one unexpected change:...
- 02:25 PM Base Language Bug #2076: DMO setter fails when parameter is BaseDataType
- This update is fine. I will conversion regression test it.
> Sorry I missed the "rule" directory , what I do when... - 01:14 PM Base Language Bug #2076: DMO setter fails when parameter is BaseDataType
- Added proposed update.
Sorry I missed the "rule" directory , what I do when adding an update: I manually create the... - 12:20 PM Base Language Bug #2076: DMO setter fails when parameter is BaseDataType
- Code Feedback:
1. Your zip still has the wrong directory name "rule" instead of "rules". How is that even getting... - 06:06 AM Base Language Bug #2076: DMO setter fails when parameter is BaseDataType
- Added proposed update (uploaded it first by mistake to #2070)
- 03:12 PM Database Feature #2080: add runtime support for database methods
- Code review 20130307a:
* There is no implementation for @QueryWrapper.getNext(LockType)@. The other @getXXX(LockT... - 11:10 AM Database Feature #2080: add runtime support for database methods
- Vadim Nebogatov wrote:
> Does P2J have common module responsible for 4GL errors processing? (I found actually ErrorM... - 11:01 AM Database Feature #2080: add runtime support for database methods
- Attached update vmn_upd20130307a.zip. Added runtime support for queryClose, queryForward, queryReposition, queryRepos...
- 05:11 AM Database Feature #2080: add runtime support for database methods
- Does P2J have common module responsible for 4GL errors processing? (I found actually ErrorManager.recordOrThrowError(...
- 02:10 PM Database Bug #2084: Conversion misplaced parameter for can-find statement
- The problem seems to be happening in the annotation phase in the following way:
The BUFFER_SCOPE with a bufname" va... - 07:46 AM Database Bug #2084: Conversion misplaced parameter for can-find statement
- Example:...
- 07:44 AM Database Bug #2084 (Closed): Conversion misplaced parameter for can-find statement
- 12:42 PM Base Language Feature #2082: add conversion support for 4GL's duplicate variable "feature"
- Greg Shah wrote:
> I'm not sure why you switched from a Set to a Map in fixups/variable_definitions.rules (did you p... - 12:29 PM Base Language Feature #2082: add conversion support for 4GL's duplicate variable "feature"
- I'm not sure why you switched from a Set to a Map in fixups/variable_definitions.rules (did you plan to use it and th...
- 12:13 PM Base Language Feature #2082: add conversion support for 4GL's duplicate variable "feature"
- The edited file is in task #2085.
This update has passed conversion regression testing (there are changes in two f... - 09:51 AM Base Language Feature #2082: add conversion support for 4GL's duplicate variable "feature"
- MAJIC has the case at note 14 in only one file. I've checked the history of this file and this change was introduced ...
- 08:59 AM Base Language Feature #2082: add conversion support for 4GL's duplicate variable "feature"
- Fixed a problem (I was using varDict.getSymbol instead of varDict.getSymbolAtScope(name, 0) to search only in the cur...
- 08:45 AM Base Language Feature #2082: add conversion support for 4GL's duplicate variable "feature"
- This has regressions, one of them relaed to this, as 4GL (v9-only I think) allowed this:...
- 06:27 AM Base Language Feature #2082: add conversion support for 4GL's duplicate variable "feature"
- So the problem was that the parser was adding tempidx to the dupped vars too. The sollution was to not register the d...
- 05:49 AM Base Language Feature #2082: add conversion support for 4GL's duplicate variable "feature"
- I need to rewrite this at the parser side, as the duped var gets ref'ed by the parser:...
- 05:13 AM Base Language Feature #2082: add conversion support for 4GL's duplicate variable "feature"
- Fixed multiple calls like:...
- 12:02 PM Base Language Bug #2069: Accumulator temp table field disambiguation
- Committed to bzr revision 10262.
- 11:56 AM Base Language Bug #2069: Accumulator temp table field disambiguation
- The change looks fine. Check it in and distribute it.
- 05:51 AM Base Language Bug #2069: Accumulator temp table field disambiguation
- This has passed conversion regression testing.
- 04:53 AM Base Language Bug #2069: Accumulator temp table field disambiguation
- I think the sollution was to mark the tables which are weak reference as preferred, when abbreviations are encountere...
- 03:51 AM Base Language Bug #2069: Accumulator temp table field disambiguation
- Another creature of the rabbit hole found at #1985, 120. This fails:...
- 09:57 AM Database Bug #2083: client-side where-clause leak in FindQuery
- I did some patch (removing the extra expression from where node after generating the anonymous inner class needed by ...
- 05:33 AM Database Bug #2083: client-side where-clause leak in FindQuery
- Another can-find leak:...
- 08:48 AM Database Feature #2000: add conversion support for table and table handle parameters
- > When will the solution be ready for review?
Hopefully will finish it today. - 08:46 AM Database Feature #2000: add conversion support for table and table handle parameters
- When will the solution be ready for review?
- 04:44 AM Database Feature #2000: add conversion support for table and table handle parameters
- About @DEF PARAMETER TABLE-HANDLE FOR th@ where @th@ is a regular variable:
1. OUTPUT case will produce compile-ti... - 05:28 AM Base Language Bug #2070: BlockManager Action disambiguation
- Added update to wrong task deleting it
03/06/2013
- 05:13 PM Database Feature #2080: add runtime support for database methods
- Please implement all the methods you can, which do not require dynamic database support. For example, FIND-FIRST wit...
- 04:38 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- OK.
The new datatype under questions:
1. If parameter is widget we need to add something: WID mapping to GenerigW... - 04:00 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- OK, get as much done tonight as possible. Then create an update zip and upload it here.
Constantin will be able t... - 03:34 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- > Eugenie: how much more time do you expect this to take?
It is > 100 items left in functions. I guess this is ~10... - 03:24 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- > Constantin: We can certainly go ahead with the infrastructure to use the facility. I planned this to be exposed th...
- 03:14 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Eugenie: how much more time do you expect this to take?
Constantin: We can certainly go ahead with the infrastruct... - 03:10 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Greg: I don't know the order you plan to have this working, but if documenting each signature will take more time, I ...
- 02:42 PM Conversion Tools Feature #1603: enhance name conversion to support internal procedure names that contain a '.' cha...
- OK, looks like this addition in NamepConverte's static c'tor:...
- 02:39 PM Conversion Tools Feature #1603: enhance name conversion to support internal procedure names that contain a '.' cha...
- This one did turn and bit us, because the server project has files named like "foo.bar.p", which get converted to "Fo...
- 02:02 PM Database Bug #2083: client-side where-clause leak in FindQuery
- Ovidiu Maxiniuc wrote:
> The parameter is in fact an echo of the content of the client-side WhereExpression: @f1(cus... - 01:59 PM Database Bug #2083: client-side where-clause leak in FindQuery
- Here are the converted code for two cases (which attempt to call two new constructors), with and without another join...
- 01:50 PM Database Bug #2083: client-side where-clause leak in FindQuery
- I tried to simplify the sample. Suppose we have a simple function:...
- 01:25 PM Database Bug #2083 (Closed): client-side where-clause leak in FindQuery
- 01:50 PM Base Language Feature #1884: add some of the v10 data types and core built-ins
- Committed to bzr revision 10259.
- 01:45 PM Base Language Feature #1884: add some of the v10 data types and core built-ins
- I agree. The code is fine. Check it in and distribute it.
- 01:30 PM Base Language Feature #1884: add some of the v10 data types and core built-ins
- Added missing TextOps.numEntries(String, String). I don't see a reason to put this through conversion regression test...
- 01:37 PM Database Feature #1947 (Closed): add conversion support for database methods
- Vadim committed this update to bzr as revision number 10256.
- 11:21 AM Database Feature #1947: add conversion support for database methods
- I made a minor change to the database_general.rules header. Please check in and distribute the attached version of y...
- 10:58 AM Database Feature #1947: add conversion support for database methods
- I have attached one more variant of the same update, but merged manually
- 10:29 AM Database Feature #1947: add conversion support for database methods
- Code review 20130306a:
Other than the literal.rules merge issue noted above (which I do think was my mistake), eve... - 09:35 AM Database Feature #1947: add conversion support for database methods
- I will merge everything manually and resubmit
- 09:25 AM Database Feature #1947: add conversion support for database methods
- As I see, update from Eric was not merged with revision you pointed. Maybe I am wrong with time calculation?
- 09:21 AM Database Feature #1947: add conversion support for database methods
- Update #134 from Eric was attached at 2013-03-05 11:38PM, so I supposed that it was merged with revision 10245 that w...
- 08:54 AM Database Feature #1947: add conversion support for database methods
- Vadim, literals.rules is not merged with 10245 (evl_upd20130305a.zip). I'll let Eric review the rest of the update.
- 08:49 AM Database Feature #1947: add conversion support for database methods
- Attached update
vmn_upd20130306a.zip / vmn_upd20130306a_test_cases.zip (start/final revisions 10245/10253)
The follow... - 11:46 PM Database Feature #1947: add conversion support for database methods
- Vadim, please note that Ovidiu soon will be committing to bzr some changes to files which will overlap with your upda...
- 01:25 PM Base Language Bug #2065: ambiguity in date/decimal literals that is not handled properly by the ProgressLexer
- The following testcase shows the limits of this ambiguity.
From the header:... - 01:20 PM Base Language Bug #2076: DMO setter fails when parameter is BaseDataType
- That crossed my mind too and started in that direction in the morning, but thought about the copy Ast with null paren...
- 01:09 PM Base Language Bug #2076: DMO setter fails when parameter is BaseDataType
- The best thing to do here is to move the current logic for ECW.expressionType(Aast) to a new public worker method ECW...
- 12:44 PM Base Language Bug #2076: DMO setter fails when parameter is BaseDataType
- The AST...
- 12:39 PM Base Language Bug #2076: DMO setter fails when parameter is BaseDataType
- Please post the AST here for the @assign test.int-field = h:BUFFER-FIELD("dsc"):BUFFER-VALUE(2).@.
- 12:16 PM Base Language Bug #2076: DMO setter fails when parameter is BaseDataType
- I'll use this example to explain....
- 11:45 AM Base Language Bug #2076: DMO setter fails when parameter is BaseDataType
- Code Review:
> I first tried to use ExpresionConversionWorker.expressionType with an ast copy without a parent (be... - 11:24 AM Base Language Bug #2076: DMO setter fails when parameter is BaseDataType
- Added proposed update
NOTE
I first tried to use ExpresionConversionWorker.expressionType with an ast copy without... - 12:54 PM Base Language Feature #1920: implement persistent procedures
- Added merged update, which was committed to bzr revision 10258.
- 12:44 PM Base Language Feature #1920: implement persistent procedures
- Passed conversion regression testing, no changes in generated sources.
I'll merge the file with latest version and... - 11:35 AM Base Language Feature #1920: implement persistent procedures
- Fixed attribute access when there are extent widgets, like:...
- 12:42 PM Database Bug #2077: fix converted table and field name collisions
- Committed to bzr revision 10257.
- 11:54 AM Database Bug #2077: fix converted table and field name collisions
- Check it in and distribute it.
- 11:40 AM Database Bug #2077: fix converted table and field name collisions
- Update has passed conversion regression testing - no changes in generated code.
- 10:46 AM Database Bug #2077: fix converted table and field name collisions
- This solves the collision cases for good. Also, I've added support for buffer names collision check. I'm putting this...
- 09:12 AM Database Bug #2077: fix converted table and field name collisions
- Yes, the fix is pretty simple. I'll provide an update today.
- 09:00 AM Database Bug #2077: fix converted table and field name collisions
- If this is easy enough to fix, please go ahead and put the fix in. That way we don't have to remember there is a pro...
- 07:38 AM Database Bug #2077: fix converted table and field name collisions
- Ovidiu, sorry I hijacked this from you last night. Anyway, this case I don't think is encountered in the server proje...
- 07:26 AM Database Bug #2077: fix converted table and field name collisions
- By chance I prepared a sample test-case for this issue which at this moment is not converted correctly by p2j:...
- 01:50 AM Database Bug #2077: fix converted table and field name collisions
- Committed to bzr revision 10246.
- 11:10 AM Base Language Feature #2082 (Closed): add conversion support for 4GL's duplicate variable "feature"
- 10:59 AM Base Language Feature #2082: add conversion support for 4GL's duplicate variable "feature"
- Committed to bzr revision 10255.
- 10:54 AM Base Language Feature #2082: add conversion support for 4GL's duplicate variable "feature"
- Check it in and distribute it.
- 10:33 AM Base Language Feature #2082: add conversion support for 4GL's duplicate variable "feature"
- The update has passed conversion regression testing. Note that even MAJIC had two cases of duplicated vars, in two fi...
- 09:10 AM Base Language Feature #2082: add conversion support for 4GL's duplicate variable "feature"
- Added comment and fixed the proc/function case. I'm putting it back into conversion regression testing.
- 09:05 AM Base Language Feature #2082: add conversion support for 4GL's duplicate variable "feature"
- Please put a comment into the variable_definitions.rules about the order of the definitions and that we deliberately ...
- 09:03 AM Base Language Feature #2082: add conversion support for 4GL's duplicate variable "feature"
- > 1. Is there any difference in behavior when the non-global define variable is inside a nested scope (e.g. an intern...
- 08:58 AM Base Language Feature #2082: add conversion support for 4GL's duplicate variable "feature"
- The code looks good. I only have 2 questions:
1. Is there any difference in behavior when the non-global define v... - 08:42 AM Base Language Feature #2082 (Review): add conversion support for 4GL's duplicate variable "feature"
- Attached update hides all duplicate var defs, only if the first one is DEF NEW GLOBAL SHARED. I'm putting this throug...
- 08:19 AM Base Language Feature #2082: add conversion support for 4GL's duplicate variable "feature"
- 4GL allows two vars with the same name, if this construct is encountered:...
- 08:18 AM Base Language Feature #2082 (Closed): add conversion support for 4GL's duplicate variable "feature"
- 08:57 AM Base Language Bug #2070: BlockManager Action disambiguation
- Commited to bzr as revision number 10254.
- 08:35 AM Base Language Bug #2070: BlockManager Action disambiguation
- The update has passed conversion regression testing, go ahead and distribute it.
- 07:36 AM Base Language Bug #2070: BlockManager Action disambiguation
- Added update
- 07:35 AM Base Language Bug #2070: BlockManager Action disambiguation
- Please remove the dangling text from the 421 and the other edits I mentioned, and re-upload. I'm putting it through c...
- 07:11 AM Base Language Bug #2070: BlockManager Action disambiguation
- Added fix update
- 06:50 AM Base Language Bug #2070: BlockManager Action disambiguation
- 0305d.zip doesn't fully solve the issue, as the package name is computed wrong. It uses the package from the nearest ...
- 08:47 AM Base Language Bug #2069 (Closed): Accumulator temp table field disambiguation
- 08:27 AM Base Language Feature #2079 (Closed): add missing conversion support for format string
- 08:26 AM Conversion Tools Bug #2066 (Closed): brew adds unnecessary comma in method parameter list after comment
- 07:13 AM Conversion Tools Bug #2066: brew adds unnecessary comma in method parameter list after comment
- Committed to bzr revision 10251.
- 05:18 AM Conversion Tools Bug #2066: brew adds unnecessary comma in method parameter list after comment
- The server folders had no errors related to comments.
- 08:08 AM Base Language Feature #2078 (Closed): provide support for malformed label names
- 01:52 AM Base Language Feature #2078: provide support for malformed label names
- Committed to bzr revision 10247.
- 07:14 AM Database Feature #1967: rework DMO class/interface hierarchy
- 0306b.zip was committed to bzr revision 10253.
- 04:46 AM Database Feature #1967: rework DMO class/interface hierarchy
- ca_upd20130306b.zip has passed conversion regression testing (no changes in generated sources).
- 02:25 AM Database Feature #1967: rework DMO class/interface hierarchy
- Attached update fixes the problem for my 0305d.zip. It brackets the deleteAll with a block only if it's parent is not...
- 11:36 PM Database Feature #1967: rework DMO class/interface hierarchy
- The om_upd20130304b.zip and ecf_upd20130305a.zip updates have gotten through conversion regression testing. However,...
- 09:50 PM Database Feature #1967: rework DMO class/interface hierarchy
- Code review -20130304a- 20130304b:
The code looks fine. The attached update corrects some cut-and-paste errors in t... - 07:14 AM Base Language Feature #1885: fix and add necessary format string support
- 0306a.zip was Committed to bzr revision 10252.
- 04:45 AM Base Language Feature #1885: fix and add necessary format string support
- 0306a.zip has passed conversion regression testing (no changes in generated sources).
- 02:32 AM Base Language Feature #1885: fix and add necessary format string support
- Correct version of progress.g
- 02:16 AM Base Language Feature #1885: fix and add necessary format string support
- This version emits the toStringMessage in case the format is a complex expression, for PUT field. Also, adds a TODO i...
- 03:03 AM Database Feature #1953: add conversion support for CREATE BUFFER statement
- Committed revision 10248.
- 09:07 PM Database Feature #1953: add conversion support for CREATE BUFFER statement
- Update 20130303a looks fine, except that the zip file is based at the com directory instead of src. Please fix this,...
- 11:41 PM Database Feature #1933: add conversion support for certain metadata constructs
- Conversion regression testing was successful. Please commit and distribute the 20130305b update.
- 10:15 PM Database Feature #1933: add conversion support for certain metadata constructs
- Code review 20130305a:
Changes look good.
Code review 20130305b:
The change is fine, but a slightly tighter ... - 11:39 PM Database Feature #1668: add support for additional (non-dynamic) database attributes
- Update 20130304c has passed conversion regression testing. Please correct the FieldReference header as noted above, ...
- 09:58 PM Database Feature #1668: add support for additional (non-dynamic) database attributes
- Code review 20130304c:
Code changes look fine, but please correct the file header numbering for the last 2 entries...
03/05/2013
- 06:32 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Generally, I like it. This will work for some of the cases, but not all cases. Still, it is good enough to go with ...
- 06:18 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- > How about this approach:
> ...
The approach is working but we need to modify the getter this way for example:
... - 05:22 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- > If we have multiple versions of the same function with different numbers of different kind of parameters how we dec...
- 05:00 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Several data type question for this time:
1. If we have multiple versions of the same function with different numbers... - 06:20 PM Database Bug #2077: fix converted table and field name collisions
- It looks fine. Check it in and distribute it.
- 05:16 PM Database Bug #2077: fix converted table and field name collisions
- Passed conversion regression testing.
- 04:18 PM Database Bug #2077: fix converted table and field name collisions
- I'm putting this through conversion regression testing.
- 02:39 PM Database Bug #2077: fix converted table and field name collisions
- This update solves collisions for dmo, table, property and column names. It adds a numeric suffix, similar to how tem...
- 02:05 PM Database Bug #2077: fix converted table and field name collisions
- In 4GL, it is possible to have two tables names named @p2j_test@ and @p2j-test@. Also, these field names can be in th...
- 02:02 PM Database Bug #2077 (Closed): fix converted table and field name collisions
- 06:02 PM Base Language Feature #2078: provide support for malformed label names
- I'm OK with the code. You can check it in and distribute.
- 05:15 PM Base Language Feature #2078: provide support for malformed label names
- Passed conversion regression testing.
- 04:18 PM Base Language Feature #2078: provide support for malformed label names
- I'm putting this through conversion regression testing.
- 04:09 PM Base Language Feature #2078: provide support for malformed label names
- In 4GL, it is possible to support malformed label names, as in:...
- 04:03 PM Base Language Feature #2078 (Hold): provide support for malformed label names
- 05:50 PM Database Feature #2080: add runtime support for database methods
- The following static methods should be removed from AbstractQuery:...
- 05:44 PM Database Feature #2080: add runtime support for database methods
- This issue is for adding runtime support for all the database methods added in issue #1947. For the most part, we on...
- 05:40 PM Database Feature #2080 (Closed): add runtime support for database methods
- 05:30 PM Database Feature #1947: add conversion support for database methods
- Update 20130305b hit a snag in conversion regression testing. It did not cause a regression per se, because the chan...
- 03:55 PM Database Feature #1947: add conversion support for database methods
- I tried to update code from bzr now (I did not make any changes since the time I sent update).
Please see result - ... - 03:49 PM Database Feature #1947: add conversion support for database methods
- I made fresh check out today morning and merged manually from updates with adding only my changes. Maybe I missed som...
- 03:44 PM Database Feature #1947: add conversion support for database methods
- Eric, sorry, but I updated code today from bzr minimum 5 times during the day with bzr update command. Last time imme...
- 03:38 PM Database Feature #1947: add conversion support for database methods
- Code review 20130305a:
The changes all look good, but rules/convert/literals.rules was out of date with bzr (confl... - 09:41 AM Database Feature #1947: add conversion support for database methods
- Updated test cases with expressions
- 09:31 AM Database Feature #1947: add conversion support for database methods
- Yes, it is true. Tests passed in this case too with the results like
query1.getNext(LockType.LT_EXCLUSIVE, plus(7,... - 07:43 AM Database Feature #1947: add conversion support for database methods
- Vadim,
> are even not compilable.
the correct syntax for you tests is this:... - 07:40 AM Database Feature #1947: add conversion support for database methods
- Attached update
vmn_upd20130305a.zip / vmn_upd20130305a_test_cases.zip
is next replacement of update vmn_upd20130... - 05:24 PM Base Language Feature #1885: fix and add necessary format string support
- Yes, using toStringMessage() is a great idea.
- 05:04 PM Base Language Feature #1885: fix and add necessary format string support
- This fixes the conversion problem, still need to expand PutField c'tors to accept character for the format... or, can...
- 04:38 PM Base Language Feature #1885: fix and add necessary format string support
- There is at least one format string problem in the server. Looks like PUT fields can have a dynamic format, and any e...
- 05:16 PM Database Feature #1967: rework DMO class/interface hierarchy
- I'm taking this back, wrong task: this has some regressions in terms that is too aggressive when bracketing the delet...
- 05:15 PM Database Feature #1967: rework DMO class/interface hierarchy
- -Passed conversion regression testing.-
- 04:18 PM Database Feature #1967: rework DMO class/interface hierarchy
- I'm putting this through conversion regression testing.
- 01:48 PM Database Feature #1967: rework DMO class/interface hierarchy
- Fix cases of a FOR EACH block converted to a tt.delete call, in case the FOR EACH is enclosed in a statement - in thi...
- 04:41 PM Base Language Feature #2079: add missing conversion support for format string
- This is for tracking the time for adding conversion support. All comments to be added in the parent #1885 task.
- 04:40 PM Base Language Feature #2079 (Closed): add missing conversion support for format string
- 04:35 PM Base Language Bug #2070 (Closed): BlockManager Action disambiguation
- Passed conversion testing and checked into bzr as revision 10244.
- 03:05 PM Base Language Bug #2070: BlockManager Action disambiguation
- I'm conversion testing this now.
- 02:56 PM Base Language Bug #2070: BlockManager Action disambiguation
- Added cs_upd20130305d.zip only with changes from previous update
- 02:55 PM Base Language Bug #2070: BlockManager Action disambiguation
- Commited cs_upd20130305c.zip to bzr as revision number 10243.
- 02:34 PM Base Language Bug #2070: BlockManager Action disambiguation
- In cs_upd20130305d.zip, please remove the unchanged files from cs_upd20130305c.zip and add a history entry to control...
- 02:32 PM Base Language Bug #2070: BlockManager Action disambiguation
- cs_upd20130305c.zip has passed conversion testing. Check it in and distribute it. I will look at cs_upd20130305d.zi...
- 02:05 PM Base Language Bug #2070: BlockManager Action disambiguation
- Added proposed update
- 01:11 PM Base Language Bug #2070: BlockManager Action disambiguation
- This is a problem indeed, the solution would be to explicitly import the DMO class.
- 12:23 PM Base Language Bug #2070: BlockManager Action disambiguation
- Costin, please double check this, because I think we need to disambiguate this in the opposite direction too, as I th...
- 11:41 AM Base Language Bug #2070: BlockManager Action disambiguation
- I'm conversion testing this now.
- 11:32 AM Base Language Bug #2070: BlockManager Action disambiguation
- Copied a wrong update deleted wrong file and uploaded the correct one
- 11:27 AM Base Language Bug #2070: BlockManager Action disambiguation
- merge
- 11:08 AM Base Language Bug #2070: BlockManager Action disambiguation
- The code looks good. Constantin will be checking in 1 more version of common-progress. Merge with that and we will ...
- 09:37 AM Base Language Bug #2070: BlockManager Action disambiguation
- merge
- 05:07 AM Base Language Bug #2070: BlockManager Action disambiguation
- Added proposed update
- 07:12 PM Base Language Bug #2070: BlockManager Action disambiguation
- Code Review:
1. Please merge up to the latest bzr level.
2. This method:... - 03:46 PM Conversion Tools Bug #2066: brew adds unnecessary comma in method parameter list after comment
- This update passes conversion regression testing with comments off (no changes in generated sources), and MAJIC does ...
- 03:34 PM Base Language Bug #2076: DMO setter fails when parameter is BaseDataType
- ExpressionConversionWorker never returns the int type (e.g. ATTR_POLY) but instead will return the wrapper class name...
- 02:36 PM Base Language Bug #2076: DMO setter fails when parameter is BaseDataType
- Trying to get the rules to create a new variable with the same field type to apply only for expression type ATTR_POLY...
- 11:12 AM Base Language Bug #2069: Accumulator temp table field disambiguation
- Greg Shah wrote:
> Check it in and distribute.
Committed to bzr revision 10242.
> Should we still test ca_upd20... - 11:02 AM Base Language Bug #2069: Accumulator temp table field disambiguation
- Check it in and distribute.
Should we still test ca_upd20130304b.zip by itself or should we include these other ch... - 10:03 AM Base Language Bug #2069: Accumulator temp table field disambiguation
- 0305b.zip has passed conversion regression testing.
- 09:17 AM Base Language Bug #2069: Accumulator temp table field disambiguation
- Another side-note for this task: we should identify all missing no-reference cases and code them in common-progress.r...
- 08:47 AM Base Language Bug #2069: Accumulator temp table field disambiguation
- I've fixed the history entry in common-progress.rules.
- 08:39 AM Base Language Bug #2069: Accumulator temp table field disambiguation
- Greg Shah wrote:
> Go ahead and check these in and distribute them. If Eric has concerns we will address them in a ... - 08:29 AM Base Language Bug #2069: Accumulator temp table field disambiguation
- Another case of no-reference is:...
- 08:18 AM Base Language Bug #2069: Accumulator temp table field disambiguation
- Go ahead and check these in and distribute them. If Eric has concerns we will address them in a separate distributio...
- 05:14 AM Base Language Bug #2069: Accumulator temp table field disambiguation
- In the end, I ended up testing (using 10237 as baseline, but this should make no difference):
# 05a (INPUT function ... - 11:41 PM Base Language Bug #2069: Accumulator temp table field disambiguation
- This makes the INPUT function to produce no-references. About 04c.zip: my test was without 04b.zip update, to better ...
- 10:48 PM Base Language Bug #2069: Accumulator temp table field disambiguation
- With these changes, pmcli/ converts fully. Also, MAJIC converts fully, but there is a CAN-FIND change in a single fil...
- 07:01 PM Base Language Bug #2069: Accumulator temp table field disambiguation
- Does this resolve #1985 note 64 or is this just more fixes on the way to the Mad Hatter's tea party?
- 10:56 AM Database Feature #1933: add conversion support for certain metadata constructs
- This update adds a message to standard console when a duplicate dmo name is detected.
- 10:11 AM Database Feature #1933: add conversion support for certain metadata constructs
- I only want you to add this feature if it is very fast to implement, since this is not in the critical path. Otherwi...
- 10:04 AM Database Feature #1933: add conversion support for certain metadata constructs
- Should I add the verification? What kind should it be? Just a @System.err/out.print()@ or persisted to logging file ?...
- 09:13 AM Database Feature #1933: add conversion support for certain metadata constructs
- Ovidiu Maxiniuc wrote:
> Shouldn't we use an automated postfixed name change if some name is already taken? Or at le... - 07:52 AM Database Feature #1933: add conversion support for certain metadata constructs
- Fixed java package for metadata tables.
Just a question. I know that the problem is solvable by using hints but if...
03/04/2013
- 06:46 PM Base Language Feature #2050: ATTR_POLY/METH_POLY used as rvalue or in expressions
- Passed conversion testing and checked in to bzr as revision 10239.
- 05:04 PM Base Language Feature #2050: ATTR_POLY/METH_POLY used as rvalue or in expressions
- New version of the update with another change to the parser that fixes a problem found in testing the server code. I...
- 03:32 PM Base Language Feature #2050: ATTR_POLY/METH_POLY used as rvalue or in expressions
- #1985 note 96 shows a new problem: DB_REF_NON_STATIC needs to be supported. I have added it to the ECW. It uses con...
- 04:38 PM Base Language Bug #2069: Accumulator temp table field disambiguation
- This fix attempt does the following:
# fixes #2059 's ecf_upd20130303b.zip, to include temp-tables too
# fixes a bu... - 03:59 PM Base Language Bug #2069: Accumulator temp table field disambiguation
- Constantin Asofiei wrote:
> This code tests only for BUFFER nodes, and excludes tables, temp-tables, work-tables. Wa... - 03:39 PM Base Language Bug #2069: Accumulator temp table field disambiguation
- Some news about the field name disambiguation issue. In SchemaDictionary.promoteEntry, there is this test:...
- 03:17 PM Base Language Bug #2069: Accumulator temp table field disambiguation
- Committed to bzr revision 10238.
These files have changes which are not limitted to just buffer scope:... - 03:01 PM Base Language Bug #2069: Accumulator temp table field disambiguation
- Check it it and distribute it. I will make sure runtime regression testing is run separately. What programs should ...
- 02:48 PM Base Language Bug #2069: Accumulator temp table field disambiguation
- Conversion regression testing has finished, the note about runtime regression testing still stands, with the addition...
- 01:52 PM Base Language Bug #2069: Accumulator temp table field disambiguation
- The code looks good.
It also fixes all remaining issues in #1985 notes 93, 104-109 and 111. These programs all co... - 01:27 PM Base Language Bug #2069: Accumulator temp table field disambiguation
- Looks like the matching in accumulators needs to be done using the bufname too, attached update does this. I'm puttin...
- 12:25 PM Base Language Bug #2069: Accumulator temp table field disambiguation
- > In my example, both fields have a schemaname of "tt.num" but the bufname will differ ("b1" vs "b2"). Actually, the...
- 12:18 PM Base Language Bug #2069: Accumulator temp table field disambiguation
- I don't think schemaname alone will work.
In my example, both fields have a schemaname of "tt.num" but the bufname... - 12:09 PM Base Language Bug #2069: Accumulator temp table field disambiguation
- The 0304a.zip passed conversion regression testing, but there are lots of buffer scope-related changes, as the accum ...
- 11:41 AM Base Language Bug #2069: Accumulator temp table field disambiguation
- If the parser gets the schemaname right, in case of unqualified field and compilable 4GL code, I don't think we will ...
- 11:15 AM Base Language Bug #2069: Accumulator temp table field disambiguation
- The changes look good, though we may have to work off something different than only schemaname....
- 10:06 AM Base Language Bug #2069: Accumulator temp table field disambiguation
- This update:
* makes the field in accum function to be a no-reference.
* fixes CAN-FIND function, when its WHERE cl... - 09:57 AM Base Language Bug #2069: Accumulator temp table field disambiguation
- About the ACCUM no_reference: after I let this be a no_reference, then the "refid" annotation will not be emitted for...
- 09:41 AM Base Language Bug #2069: Accumulator temp table field disambiguation
- Please prepare the ACCUM no_reference and CAN-FIND changes and upload the update ASAP. Both of these seem to be corr...
- 04:03 PM Database Bug #2059: CAN-FIND clause for a perm table in the RECORD_PHRASE of a temp table, with a FOR EACH...
- Good point. Actually, the best is probably just to get rid of the @schemaScope@ variable altogether and use the @glo...
- 03:53 PM Database Bug #2059: CAN-FIND clause for a perm table in the RECORD_PHRASE of a temp table, with a FOR EACH...
- LE: actually, I think the correct test is:...
- 03:45 PM Database Bug #2059: CAN-FIND clause for a perm table in the RECORD_PHRASE of a temp table, with a FOR EACH...
- Eric, your fix in SchemaDictionary I don't think it works for temp-tables, as the following code:...
- 03:12 PM Database Feature #1933: add conversion support for certain metadata constructs
- Ovidiu Maxiniuc wrote:
> You mean that we should use @P2OLookup@ instead of the current @SymbolResolver@ calls ?
... - 01:04 PM Database Feature #1933: add conversion support for certain metadata constructs
- > I think this is probably best done in P2OLookup (in the schema package), such that P2OLookup.javaPackage(String his...
- 11:07 AM Database Feature #1933: add conversion support for certain metadata constructs
- Ovidiu Maxiniuc wrote:
> 4GL has a meta set of tables associated with each database and from the user point of view ... - 02:35 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- > Didn't the static import statement work? I wasn't sure if it would work with enums or not. If it doesn't work, then...
- 02:31 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- > We need to create separate class file for ParmType?
Yes.
> In this case we need to declare ParmType.CHAR ins... - 01:35 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Two questions.
1. We need to create separate class file for ParmType? In this case we need to declare ParmType.CHAR ... - 10:51 AM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- This is a high priority for milestone 4 so that we can make sure to generate compilable code. Please implement this ...
- 01:19 PM Database Feature #1947: add conversion support for database methods
- Vadim Nebogatove wrote (in email):
> I tried to make tests today with literal constants. Possible I implemented chan... - 09:44 AM Database Feature #1947: add conversion support for database methods
- Vadim Nebogatov wrote:
> I have analyzed one more time. Solution you wrote is useful if both parameters are literals... - 04:05 AM Database Feature #1947: add conversion support for database methods
- I have analyzed one more time. Solution you wrote is useful if both parameters are literals. But many combined cases ...
- 03:04 AM Database Feature #1947: add conversion support for database methods
- I did not notice that Ubuntu Nautilus does not show backup files by default...
- 09:48 PM Database Feature #1947: add conversion support for database methods
- Code review 20130303a:
* As I noted earlier, please don't include backup files (e.g., @QueryWrapper.java~@) in you... - 12:38 PM Base Language Bug #2076: DMO setter fails when parameter is BaseDataType
- Example:...
- 12:34 PM Base Language Bug #2076 (Closed): DMO setter fails when parameter is BaseDataType
- 11:32 AM Base Language Bug #2070: BlockManager Action disambiguation
- Added the proposed update
- 10:54 AM Database Feature #1668: add support for additional (non-dynamic) database attributes
- Added @FieldInterface@/@FieldReference.value(NumberType)@. This is related to #2068, note 18.1.
This update must be... - 10:53 AM Base Language Bug #2075: unknown literals in nested expressions
- See #2053 where we are creating a parameter type lookup facility for builtin funcs and methods.
- 10:20 AM Base Language Bug #2075: unknown literals in nested expressions
- > One of the things I was interested in using the signature data for was during expression typing (we could know exac...
- 09:52 AM Base Language Bug #2075: unknown literals in nested expressions
- > Go ahead and commit the change and distribute it.
Committed to bzr revision 10237.
> If I understand properly, ... - 09:46 AM Base Language Bug #2075: unknown literals in nested expressions
- Go ahead and commit the change and distribute it.
If I understand properly, we are about to have compile issues wi... - 09:28 AM Base Language Bug #2075: unknown literals in nested expressions
- > The question: how much code do we have that is dependent upon this unintended result? I prefer to cleanup the pars...
- 09:07 AM Base Language Bug #2075: unknown literals in nested expressions
- Actually, I am worried that the problem is really in how the parser leaves the tree. A pretty important rule about t...
- 10:47 AM Database Feature #1967: rework DMO class/interface hierarchy
- Unfortunately, at some point we might have to rename the APIs in these interfaces too, which are extended by Buffer i...
- 10:33 AM Database Feature #1967: rework DMO class/interface hierarchy
- This update fixes eventual collisions in dmo interfaces because of @WrappedResource@ interface (@isValid()@ & @isUnkn...
- 09:37 AM Database Feature #2000: add conversion support for table and table handle parameters
- Got it. I was mis-reading the meaning of the syntax.
- 09:33 AM Database Feature #2000: add conversion support for table and table handle parameters
- > Is there any difference in behavior if you switch the order of the DEF SHARED VAR and DEF PARAMETER (if the DEF SHA...
- 09:31 AM Database Feature #2000: add conversion support for table and table handle parameters
- Is there any difference in behavior if you switch the order of the DEF SHARED VAR and DEF PARAMETER (if the DEF SHARE...
- 05:39 AM Database Feature #2000: add conversion support for table and table handle parameters
- I found that @DEF PARAMETER TABLE-HANDLE FOR th@ where @th@ is a [new] shared variable is handled differently that if...
- 05:15 AM Database Feature #1953: add conversion support for CREATE BUFFER statement
- Added missing create() overloads.
Also reformatted and sorted so that they are more visible/countable.
Fixes compil...
03/03/2013
- 03:40 PM Base Language Bug #2069: Accumulator temp table field disambiguation
- Some final notes for today:
# the @CREATE real@ statement produces a free reference, same as a @FIND FIRST real@. If... - 12:32 PM Base Language Bug #2069: Accumulator temp table field disambiguation
- Good information in note 3.
1. You have proven that the ACCUM() function parameters are not free references, they ... - 06:40 AM Base Language Bug #2069: Accumulator temp table field disambiguation
- First of all, looks like ACCUM function should not produce free-references, because if it did produce, then the code ...
- 03:28 PM Database Feature #1947: add conversion support for database methods
- Attached update vmn_upd20130303a.zip with tests vmn_upd20130303a_test_cases.zip is next replacement of update vmn_upd...
- 03:22 PM Database Bug #2059: CAN-FIND clause for a perm table in the RECORD_PHRASE of a temp table, with a FOR EACH...
- The attached update is an interim workaround to protect against the NPE so we can get further into conversion. Still...
- 11:46 AM Base Language Bug #2075: unknown literals in nested expressions
- The fix has passed conversion regression testing.
- 08:06 AM Base Language Bug #2075: unknown literals in nested expressions
- P2J fails to convert this case:...
- 07:22 AM Base Language Bug #2075 (WIP): unknown literals in nested expressions
03/02/2013
- 06:11 PM Base Language Feature #2074: add CAN-QUERY() and CAN-SET() support
- The ges_upd20130302b.zip passed conversion testing and checked in as bzr revision 10233.
- 05:40 PM Base Language Feature #2074: add CAN-QUERY() and CAN-SET() support
- This code provides the conversion support for these builtin functions. It also stubs out the methods in HandleOps. ...
- 05:38 PM Base Language Feature #2074 (Closed): add CAN-QUERY() and CAN-SET() support
- 06:11 PM Database Feature #1947: add conversion support for database methods
- The ges_upd20130302a.zip passed conversion testing and checked in as bzr revision 10232.
- 04:52 PM Database Feature #1947: add conversion support for database methods
- As found in #1985 (and documented in note 3 of #1655), the DELETE-RESULT-LIST-ENTRY() method should have been added p...
- 09:34 AM Database Feature #1947: add conversion support for database methods
- Vadim Nebogatov wrote:
> Eric Faulhaber wrote:
> > In convert/literals.rules, rewrite the rules regarding lock type... - 05:23 AM Database Feature #1947: add conversion support for database methods
- Eric Faulhaber wrote:
> In convert/literals.rules, rewrite the rules regarding lock type literals (added in the last... - 12:42 PM Database Feature #1933: add conversion support for certain metadata constructs
- For identifying the correct package to be imported I went back on the annotation paths for @pkgname@, @bufname@, @sch...
- 02:39 AM Database Bug #2071 (Closed): conversion fails to find attach point for QueryWrapper instance variable decl...
- 02:36 AM Database Bug #2071: conversion fails to find attach point for QueryWrapper instance variable declaration
- A non-promoted OPEN QUERY in an IF-THEN construct fails to find an attach point to which to graft its QueryWrapper de...
- 01:25 AM Database Bug #2071 (Closed): conversion fails to find attach point for QueryWrapper instance variable decl...
- 08:15 PM Base Language Feature #2050: ATTR_POLY/METH_POLY used as rvalue or in expressions
- Passed conversion testing and checked in as bzr revision 10230.
03/01/2013
- 05:17 PM Base Language Feature #2050: ATTR_POLY/METH_POLY used as rvalue or in expressions
- This passed testing, but in further processing of the customer code, another (virtually identical) NPE was encountere...
- 03:29 PM Base Language Feature #2050: ATTR_POLY/METH_POLY used as rvalue or in expressions
- Attached is the fix (and testcase updates) for this case. Based on testing in the 4GL, one can indeed have the retur...
- 04:05 PM Database Feature #1947: add conversion support for database methods
- Attached update vmn_upd20130301c.zip with tests vmn_upd20130301c_test_cases.zip is replacement of update vmn_upd20130...
- 12:15 PM Database Feature #1947: add conversion support for database methods
- Vadim Nebogatov wrote:
> Yes, I hope
If not, please post what you have at the end of the day so we have the optio... - 11:42 AM Database Feature #1947: add conversion support for database methods
- Yes, I hope
- 11:22 AM Database Feature #1947: add conversion support for database methods
- Vadim Nebogatov wrote:
> Changes in *.rules files. OK, will correct. My impression was that it will be needed for ru... - 10:55 AM Database Feature #1947: add conversion support for database methods
- Changes in *.rules files. OK, will correct. My impression was that it will be needed for runtime.
Static queryC... - 09:40 AM Database Feature #1947: add conversion support for database methods
- Code review 20130301a:
* The update is incomplete. I provided detailed instructions on the needed changes to rule... - 09:05 AM Database Feature #1947: add conversion support for database methods
- Attached update vmn_upd20130301b.zip with tests vmn_upd20130301b_test_cases.zip
containing fixes for # 1947/ #102: ... - 08:58 AM Database Feature #1947: add conversion support for database methods
- Sorry I have not specified, it was Ovidiu's update.
- 08:57 AM Database Feature #1947: add conversion support for database methods
- I think he means 104. I'm reviewing 103 now. It is not ready; I'll provide detail shortly.
- 08:50 AM Database Feature #1947: add conversion support for database methods
- Constantin Asofiei wrote:
> It passed conversion regression testing, please go ahead and checkin/ distribute.
Do ... - 08:49 AM Database Feature #1947: add conversion support for database methods
- It passed conversion regression testing, please go ahead and checkin/ distribute.
- 07:16 AM Database Feature #1947: add conversion support for database methods
- The code looks good, I'm putting this through conversion regression testing.
- 07:03 AM Database Feature #1947: add conversion support for database methods
- This update removes the unneeded @handle:@ token from the chaining like:...
- 03:46 AM Database Feature #1947: add conversion support for database methods
- Attached update vmn_upd20130301a.zip with tests vmn_upd20130301a_test_cases.zip containing
1) the conversion and runt... - 03:42 PM Conversion Tools Bug #2066: brew adds unnecessary comma in method parameter list after comment
- WOW! That is very cool. Great job!
- 03:40 PM Conversion Tools Bug #2066: brew adds unnecessary comma in method parameter list after comment
- MAJIC has compiled with comments enabled (and AFAIK this was not possible before this update).
- 02:48 PM Conversion Tools Bug #2066: brew adds unnecessary comma in method parameter list after comment
- Conversion has finished, compiled fully, but there are some regressions in terms of:
# new-lines no longer emitted i... - 01:40 PM Conversion Tools Bug #2066: brew adds unnecessary comma in method parameter list after comment
- OK, the server's compilable folders do not seem to report any comment-related errors. I'm putting this through conver...
- 11:53 AM Conversion Tools Bug #2066: brew adds unnecessary comma in method parameter list after comment
- Well done. It looks good. Painful, though. I have to think about this problem a bit to see if there is a way to he...
- 11:41 AM Conversion Tools Bug #2066: brew adds unnecessary comma in method parameter list after comment
- I'm testing this version with the server files which had comment errors, if everything is good I move to conversion r...
- 09:24 AM Conversion Tools Bug #2066: brew adds unnecessary comma in method parameter list after comment
- I'm stopping conversion, a server folder has failed with some other comment related issue. I need to get rewrite at l...
- 09:13 AM Conversion Tools Bug #2066: brew adds unnecessary comma in method parameter list after comment
- I'm running conversion now.
- 09:12 AM Conversion Tools Bug #2066: brew adds unnecessary comma in method parameter list after comment
- I am fine with the changes. If they pass conversion testing, let's go with it.
- 07:53 AM Conversion Tools Bug #2066: brew adds unnecessary comma in method parameter list after comment
- Replaced getNextSibling with ignore-star version.
- 06:57 AM Conversion Tools Bug #2066: brew adds unnecessary comma in method parameter list after comment
- Made next-child-rules to be ignorant of star-comment nodes, I'm putting this through conversion regression testing.
- 03:14 PM Database Feature #1652: add conversion and runtime support for dynamically prepared temp-tables
- Update has passed conversion regression test and is committed to bzr rev. 10228.
- 08:03 AM Database Feature #1652: add conversion and runtime support for dynamically prepared temp-tables
- @defaultBufferHandle@ is pushed up from @TempTable@ to @Buffer@ interface.
- 03:04 PM Database Feature #1949: add conversion support for database triggers
- This update looks good and has passed conversion regression testing. Please check it into bzr and distribute.
- 08:31 AM Database Feature #1949: add conversion support for database triggers
- Constantin's point is a good one. It looks to me like the parser is fine (of course I like saying that :)).
Can w... - 08:16 AM Database Feature #1949: add conversion support for database triggers
- 2.1 would be OK only if the ON WRITE statement would work only with table names, but looks like it works with buffer ...
- 07:59 AM Database Feature #1949: add conversion support for database triggers
- Option 2.1 seems most correct to me, because it makes the tree accurately reflect the meaning of the code from the be...
- 03:29 AM Database Feature #1949: add conversion support for database triggers
- ...
- 02:10 PM Base Language Bug #2070: BlockManager Action disambiguation
- Code Feedback:
In regard to the converted class names, there are several problems that have to be addressed:
1.... - 12:20 PM Base Language Bug #2070: BlockManager Action disambiguation
- I haven't reviewed your update, but yes, that approach is OK. I was responding to Constantin's idea about maintainin...
- 12:14 PM Base Language Bug #2070: BlockManager Action disambiguation
- I used isDMOInterface(String) in the defined function usedClassName (core_conversion.xml) from the proposed update to...
- 12:09 PM Base Language Bug #2070: BlockManager Action disambiguation
- BTW, if this is called more than once, we will want to optimize it, because it copies large sets of strings.
- 12:07 PM Base Language Bug #2070: BlockManager Action disambiguation
- ...and the attachment.
- 12:06 PM Base Language Bug #2070: BlockManager Action disambiguation
- Here, give this a try. Register P2OAccessWorker in TRPL and call @getAllDMOInterfaceNames()@ to return a set of all ...
- 11:58 AM Base Language Bug #2070: BlockManager Action disambiguation
- Added proposed update
- 11:31 AM Base Language Bug #2070: BlockManager Action disambiguation
- Greg, do we have a way of knowing if a certain name is for a converted class name (associated with the external progr...
- 11:26 AM Base Language Bug #2070: BlockManager Action disambiguation
- Please look at the com.goldencode.p2j.schema.P2OAccessWorker as a better solution than manually parsing the dmo_index...
- 11:22 AM Base Language Bug #2070: BlockManager Action disambiguation
- Please do the following and re-upload:
# rename dmoUsedNames to usedClassNames (idea is, to save all generated class... - 11:07 AM Base Language Bug #2070: BlockManager Action disambiguation
- Added proposed update. Solution was to parse the dmo_index.xml file and retrieve the existing dmo names in a global H...
- 10:13 AM Database Feature #1933: add conversion support for certain metadata constructs
- I will go on with this issue, as I am now able to see what's wrong using my project.
- 07:50 AM Database Feature #1933: add conversion support for certain metadata constructs
- Ovidiu Maxiniuc wrote:
> I exported a sample @_File@ structure (.df) and data (.d) from my test database ...
I ca... - 03:28 AM Database Feature #1933: add conversion support for certain metadata constructs
- Eric,
I exported a sample @_File@ structure (.df) and data (.d) from my test database but, for some unknown (to me)... - 01:01 AM Database Feature #1668: add support for additional (non-dynamic) database attributes
- FieldReference.value is missing the value(NumberType) version (subscript can be more than int literals).
PS: and Fie...
02/28/2013
- 06:48 PM Base Language Bug #2056 (Closed): null value for an annotation while persisting the AST
- 06:47 PM Base Language Bug #2056: null value for an annotation while persisting the AST
- Passed conversion testing and is checked into bzr as revision 10226.
- 05:31 PM Base Language Bug #2056: null value for an annotation while persisting the AST
- OK, some findings:
1. Generally, @@@ base-field can only be used in a DISPLAY stmt and in the column_spec of a DEF... - 06:10 PM Base Language Feature #2063: Support negative numbers defined with the "-" on the right side
- The lexer cannot easily change its behavior in response to feedback from the parser. This is a core limitation of AN...
- 04:30 PM Base Language Feature #2063: Support negative numbers defined with the "-" on the right side
- We have a case where the @999.99-@ literal is converted to @new date("999.99-")@ instead of decimal:...
- 04:41 PM Database Feature #1947: add conversion support for database methods
- I'm posting these here too, from #2068:
More errors related to database methods:... - 04:02 PM Database Feature #1947: add conversion support for database methods
- Vadim Nebogatov wrote:
> It works with 0 without errors too; - as far as I understood
>
> ... - 03:11 PM Database Feature #1947: add conversion support for database methods
- I tested in 4GL the following...
- 02:22 PM Database Feature #1947: add conversion support for database methods
- One correction to my last posting:
In the case where the first argument is NOT one of the lock literals, AND we ha... - 02:05 PM Database Feature #1947: add conversion support for database methods
- OK, then we must emulate this error handling at runtime, which means we need the following APIs:...
- 01:13 PM Database Feature #1947: add conversion support for database methods
- I have tested in 4GL the following...
- 11:59 AM Database Feature #1947: add conversion support for database methods
- Constantin Asofiei wrote:
> > Again, if the literals don't match the well-known compiler constants, Progress must de... - 11:36 AM Database Feature #1947: add conversion support for database methods
- > Again, if the literals don't match the well-known compiler constants, Progress must default to some lock behavior d...
- 11:34 AM Database Feature #1947: add conversion support for database methods
- Constantin Asofiei wrote:
> You forgot these:
> ... - 11:14 AM Database Feature #1947: add conversion support for database methods
- Eric Faulhaber wrote:
> Yes, please hold this update for the moment.
>
> I think we can cover all cases with:
> ... - 11:08 AM Database Feature #1947: add conversion support for database methods
- Yes, please hold this update for the moment.
I think we can cover all cases with:... - 10:45 AM Database Feature #1947: add conversion support for database methods
- Variant with 100+ methods (integer, int64, long, LockType) is ready
GET-NEXT (KW_GET_NEXT)
GET-PREV (KW_GET_PREV... - 10:41 AM Database Feature #1947: add conversion support for database methods
- > OK, but wouldn't the @i + 1@ expression be converted to @plus(i, 1)@ (which returns an integer wrapper instance), s...
- 10:36 AM Database Feature #1947: add conversion support for database methods
- Constantin Asofiei wrote:
> > Am I missing something?
> yes, you can pass vars/expressions to these parameters, not... - 10:22 AM Database Feature #1947: add conversion support for database methods
- > Am I missing something?
yes, you can pass vars/expressions to these parameters, not just literals:... - 10:19 AM Database Feature #1947: add conversion support for database methods
- Constantin Asofiei wrote:
> Vadim Nebogatov wrote:
> > If we want to reuse current(LockType lockType) we will need ... - 10:07 AM Database Feature #1947: add conversion support for database methods
- Eric Faulhaber wrote:
> I don't understand this. Why can't we use NumberType to reduce the explosion of APIs? We a... - 10:05 AM Database Feature #1947: add conversion support for database methods
- Constantin Asofiei wrote:
> where both parameters are optional, and nowait appears only if first is set. We can redu... - 08:51 AM Database Feature #1947: add conversion support for database methods
- Vadim Nebogatov wrote:
> If we want to reuse current(LockType lockType) we will need to create LockType from other ty... - 08:49 AM Database Feature #1947: add conversion support for database methods
- If we want to reuse current(LockType lockType) we will need to create LockType from other types including long
- 08:40 AM Database Feature #1947: add conversion support for database methods
- If you set the parameter to java native long, then all native int values will be accepted, so no casting is needed. I...
- 08:39 AM Database Feature #1947: add conversion support for database methods
- Will we cast long to int before using LockType.fromInt(int type)?
- 08:33 AM Database Feature #1947: add conversion support for database methods
- Vadim Nebogatov wrote:
> LockType aggregates lock and wait statuses, what is the sense of using it with other parame... - 08:26 AM Database Feature #1947: add conversion support for database methods
- LockType aggregates lock and wait statuses, what is the sense of using it with other parameters, especially with one ...
- 08:14 AM Database Feature #1947: add conversion support for database methods
- The syntax for this method would be (example for GET-NEXT):...
- 08:03 AM Database Feature #1947: add conversion support for database methods
- Constantin Asofiei wrote:
> Vadim Nebogatov wrote:
> > OK, good. What do you think about #68 (getNext(LockType lock... - 07:35 AM Database Feature #1947: add conversion support for database methods
- LE: when I say no work needed, I refer to the lock types. You will still need getNext(LockType) version to be added.
- 07:34 AM Database Feature #1947: add conversion support for database methods
- Vadim Nebogatov wrote:
> OK, good. What do you think about #68 (getNext(LockType lockType))?
Actually, NO-LOCK and ... - 06:38 AM Database Feature #1947: add conversion support for database methods
- OK, good. What do you think about #68 (getNext(LockType lockType))?
- 06:11 AM Database Feature #1947: add conversion support for database methods
- > According doc, lock is mandatory, no-wait is optional:
>
> GET-NEXT ( NO-LOCK| SHARE-LOCK [, NO-WAIT] | EXCLUSIV... - 06:00 AM Database Feature #1947: add conversion support for database methods
- Eric Faulhaber wrote:
> Ignore my last post; I mis-read the documentation. NO-LOCK is the default if not specified... - 03:51 AM Database Feature #1947: add conversion support for database methods
- I don't understand... stupid chrome or redmine ?
- 03:50 AM Database Feature #1947: add conversion support for database methods
- I think ...
- 03:47 AM Database Feature #1947: add conversion support for database methods
- The same question concerning getNext parameters. Do we really need method with signature using P2J type LockType?
... - 03:40 AM Database Feature #1947: add conversion support for database methods
- The question: BUFFER-COMPARE/ BUFFER-COPY support also should be corrected. May I do this in the same update?
BUF... - 04:15 PM Conversion Tools Bug #2066: brew adds unnecessary comma in method parameter list after comment
- The conclusion for today, after trying a scoping approach to solve some nested calls issues is this, in brew phase (a...
- 09:10 AM Conversion Tools Bug #2066: brew adds unnecessary comma in method parameter list after comment
- I still have some regressions in it, I'm testing and will re-upload.
- 09:09 AM Conversion Tools Bug #2066: brew adds unnecessary comma in method parameter list after comment
- The code looks fine. And that is some testcase! It made me laugh to read it. Well done.
- 08:41 AM Conversion Tools Bug #2066: brew adds unnecessary comma in method parameter list after comment
- This fixes these cases:...
- 05:12 AM Conversion Tools Bug #2066: brew adds unnecessary comma in method parameter list after comment
- I believe this was also addressed in the #1450. Unfortunately it was not finalized.
- 03:01 AM Conversion Tools Bug #2066: brew adds unnecessary comma in method parameter list after comment
- Fix attempt for star comments mixed with children of IF, method arguments, and others. Will test with MAJIC and some ...
- 01:44 AM Conversion Tools Bug #2066: brew adds unnecessary comma in method parameter list after comment
- I suggest to disable comments, run conversion in the good directories and compile again, to uncover non-comment probl...
- 01:21 AM Conversion Tools Bug #2066 (WIP): brew adds unnecessary comma in method parameter list after comment
- 03:33 PM Database Feature #1933: add conversion support for certain metadata constructs
- Correct, we need the import.
Ovidiu was looking at this at the end of the day today, I guess he will continue tomo... - 02:49 PM Database Feature #1933: add conversion support for certain metadata constructs
- Ah, good point. So, we need to emit an import statement for ...dmo._meta.*, right? Are you taking this, or do you w...
- 02:43 PM Database Feature #1933: add conversion support for certain metadata constructs
- The metadata tables are emitted to the dmo._meta package, but all buffers referring meta tables are still converted t...
- 03:31 PM Database Feature #1652: add conversion and runtime support for dynamically prepared temp-tables
- Note that the defaultBufferHandle API I think it needs to be added to the Buffer interface too, because there are com...
- 02:51 PM Base Language Bug #2070: BlockManager Action disambiguation
- Keep in mind that BUFFER_SCOPE is for DMOs beeing used by the current program, and not all the existing DMOs. I think...
- 02:35 PM Base Language Bug #2070: BlockManager Action disambiguation
- Can the "classname" annotation of BUFFER_SCOPE be used safely to get that checked?
- 02:12 PM Base Language Bug #2070: BlockManager Action disambiguation
- Costin, looks like we need another check: if there is a DMO class with the Action name, then qualify Action with Bloc...
- 11:47 AM Base Language Bug #2070: BlockManager Action disambiguation
- This is fine for now. Ultimately, we will need to do this on a more global scale for all names. Please get it conver...
- 11:39 AM Base Language Bug #2070: BlockManager Action disambiguation
- Added proposed update
- 10:43 AM Base Language Bug #2070: BlockManager Action disambiguation
- Do you think it's better to replace Action.<enum_val> with BlockManager.Action.<enum_val> allways or check file name ...
- 10:40 AM Base Language Bug #2070: BlockManager Action disambiguation
- When we have a progress file named action and it contains a block which will emit a reference to the Action enum of t...
- 10:35 AM Base Language Bug #2070 (Closed): BlockManager Action disambiguation
- 11:02 AM Base Language Bug #2069: Accumulator temp table field disambiguation
- After the note 75 in the 1985, looks like this is a true buffer disambiguation problem, which can be duplicated using...
- 06:09 AM Base Language Bug #2069: Accumulator temp table field disambiguation
- For the following testcase we define a temp table fit a field name that already exists in a permanent table and use i...
- 05:58 AM Base Language Bug #2069 (Closed): Accumulator temp table field disambiguation
- If we create a temp table with a field name that coincides with a field from an existent permanent table and use accu...
- 09:06 AM Database Feature #1949: add conversion support for database triggers
- > do you have any idea why this happens ? Is this intentional ?
No. No. I'll take it as my top priority. - 08:25 AM Database Feature #1949: add conversion support for database triggers
- During the investigations for #1985, note 69, resulted that a possible bug is in the parser.
The following code:
<p... - 01:48 AM Database Feature #1588: add conversion and runtime support for embedded SQL
- 20130227a update has passed regression testing and is committed to bzr rev. 10225.
- 12:40 AM Database Feature #1588: add conversion and runtime support for embedded SQL
- Attached update fixes a defect during annotations processing.
02/27/2013
- 06:13 PM Base Language Feature #2050: ATTR_POLY/METH_POLY used as rvalue or in expressions
- A new context analysis problem has been found in #1985 (see note 63). The 2nd operand of the PLUS is a FIELD_DEC typ...
- 05:43 PM Base Language Bug #2056: null value for an annotation while persisting the AST
- One other point: I think if we get the parser right, the downstream processing may be much simpler. Today, we have t...
- 05:41 PM Base Language Bug #2056: null value for an annotation while persisting the AST
- After looking at this in detail, it is clear that this is something that I need to work on. Both of you should work ...
- 10:32 AM Base Language Bug #2056: null value for an annotation while persisting the AST
- This code works in P2J:...
- 09:47 AM Base Language Bug #2056: null value for an annotation while persisting the AST
- I was trying to force the rules to see the var definition hidden in the format_phrase node, and emit widget + var for...
- 09:36 AM Base Language Bug #2056: null value for an annotation while persisting the AST
- I can help with this.
Constantin: do you have any other findings to report?
We do support @-base fields alrea... - 05:17 PM Database Feature #2000: add conversion support for table and table handle parameters
- I uncommited my last commit.
- 05:13 PM Database Feature #2000: add conversion support for table and table handle parameters
- > Please distribute the Config change as a separate update zip file BUT at the SAME TIME as the svl_upd20130226a.zip....
- 05:04 PM Database Feature #2000: add conversion support for table and table handle parameters
- Please distribute the Config change as a separate update zip file BUT at the SAME TIME as the svl_upd20130226a.zip. ...
- 04:37 PM Database Feature #2000: add conversion support for table and table handle parameters
- Actually, I think we're OK here. All the failures are in the Config class, which is hand-written and will need to be...
- 03:47 PM Database Feature #2000: add conversion support for table and table handle parameters
- Majic did not compile after this update. Please see #2067 for the results.
- 03:08 PM Database Feature #2000: add conversion support for table and table handle parameters
- > I'll do it here. Do you expect any differences? If I find anything unusual, I'll ask.
TABLE parameters should us... - 02:29 PM Database Feature #2000: add conversion support for table and table handle parameters
- I'll do it here. Do you expect any differences? If I find anything unusual, I'll ask.
- 02:17 PM Database Feature #2000: add conversion support for table and table handle parameters
- > Have you run conversion on Majic with this to check for unexpected changes?
No. I can run, but it will take 2.5 h. - 02:10 PM Database Feature #2000: add conversion support for table and table handle parameters
- Have you run conversion on Majic with this to check for unexpected changes?
- 02:07 PM Database Feature #2000: add conversion support for table and table handle parameters
- Everything except SHARED variables for TABLE-HANDLE FOR. I recommend to distribute the remaining part later as a sepa...
- 12:11 AM Database Feature #2000: add conversion support for table and table handle parameters
- Stanislav Lomany wrote:
> Heh, it look like BUFFER parameter is misconverted, but not because of me:
>
> [...]
... - 07:23 PM Database Feature #2000: add conversion support for table and table handle parameters
- Fix for #2.
- 03:53 PM Database Feature #1947: add conversion support for database methods
- om_upd20130227c.zip has passed conversion testing and is checked into bzr as revision 10223.
- 02:17 PM Database Feature #1947: add conversion support for database methods
- The om_upd20130227b.zip is close, but I have made a couple of changes to the code. The post-rules section is moved t...
- 01:29 PM Database Feature #1947: add conversion support for database methods
- Ignore my last post; I mis-read the documentation. NO-LOCK is the default if not specified. So, it seems we will n...
- 01:25 PM Database Feature #1947: add conversion support for database methods
- Correction: I don't think we'll need @P2JQuery.getNext()@ (only @P2JQuery.getNext(LockType lockType)@); seems the 4...
- 01:14 PM Database Feature #1947: add conversion support for database methods
- Vadim, please fix the query handle GET-* return type problem, as described in #1985, note 61 (repeated below for conv...
- 12:37 PM Database Feature #1947: add conversion support for database methods
- This update fixes generation of @LockType@ literals into converted java source.
- 12:04 PM Database Feature #1947 (WIP): add conversion support for database methods
- Needed to re-open this for some conversion corrections to query methods GET-NEXT, GET-FIRST, etc. Please see note 61...
- 02:06 PM Conversion Tools Bug #2066: brew adds unnecessary comma in method parameter list after comment
- On a whim last night, I tried to compile the partially converted customer app project, and quickly hit an "illegal st...
- 01:59 PM Conversion Tools Bug #2066 (Closed): brew adds unnecessary comma in method parameter list after comment
- 01:48 PM Database Feature #1655: add conversion and runtime support for certain dynamic database attributes and met...
- Passed runtime regression testing and is checked into bzr as revision 10222.
- 01:45 PM Database Feature #1949: add conversion support for database triggers
- Passed conversion testing and is checked in as bzr revision 10221.
- 12:16 PM Database Feature #1949: add conversion support for database triggers
- Conversion testing passed. Please check in the update and distribute it.
- 09:45 AM Database Feature #1949: add conversion support for database triggers
- I have started conversion testing.
- 09:41 AM Database Feature #1949: add conversion support for database triggers
- The code looks OK. We need to conversion test it.
- 09:40 AM Database Feature #1949: add conversion support for database triggers
- Indeed, the annotations are not set unless some variables are first declared. And, because in our sample the block is...
- 09:19 AM Database Feature #1949: add conversion support for database triggers
- Looks like for trigger blocks, the undoable counter are not initialized (written as annotation).
The attached propos... - 05:24 AM Database Feature #1949: add conversion support for database triggers
- Problem with DB triggers in server project:...
- 12:14 PM Base Language Bug #2065: ambiguity in date/decimal literals that is not handled properly by the ProgressLexer
- For details, please see notes 6 and 7 of #2063.
- 12:13 PM Base Language Bug #2065 (Closed): ambiguity in date/decimal literals that is not handled properly by the Progre...
- 12:12 PM Base Language Feature #2063 (Closed): Support negative numbers defined with the "-" on the right side
- 11:39 AM Base Language Feature #2063: Support negative numbers defined with the "-" on the right side
- Commited to bzr as revision number 10220.
- 11:17 AM Base Language Feature #2063: Support negative numbers defined with the "-" on the right side
- Has passed conversion regression testing (no changes in generated sources), please check in and distribute.
- 09:30 AM Base Language Feature #2063: Support negative numbers defined with the "-" on the right side
- I'm OK with it. Put it through conversion testing.
- 09:28 AM Base Language Feature #2063: Support negative numbers defined with the "-" on the right side
- Added proposed update
- 09:08 AM Base Language Feature #2063: Support negative numbers defined with the "-" on the right side
- The coding changes look OK. Moving the sign to the left was what I was going to propose. Make sure you remove the e...
- 09:06 AM Base Language Feature #2063: Support negative numbers defined with the "-" on the right side
- We must be very careful here....
- 08:18 AM Base Language Feature #2063: Support negative numbers defined with the "-" on the right side
- Please test with these cases too:...
- 08:02 AM Base Language Feature #2063: Support negative numbers defined with the "-" on the right side
- Added proposed update which corrects the assignment.
- 06:44 AM Base Language Feature #2063: Support negative numbers defined with the "-" on the right side
- Update looks good, this is going through conversion regression testing now.
- 06:13 AM Base Language Feature #2063: Support negative numbers defined with the "-" on the right side
- This update fixes conversion side but there is still work to do to have the correct value set.
- 04:37 AM Base Language Feature #2063: Support negative numbers defined with the "-" on the right side
- Statements like: ...
- 04:31 AM Base Language Feature #2063 (Closed): Support negative numbers defined with the "-" on the right side
- 11:53 AM Database Feature #1668: add support for additional (non-dynamic) database attributes
- Discussion moved from emails.
---
Eric,
As far as I understood, Ovidiu is already busy with BUFFER-VALUE r... - 10:35 AM Base Language Bug #2061: unreachable code is too aggressive when shared vars are used
- Greg Shah wrote:
> Everything looks good here, except the copyright dates in the new ruleset. You can check in a fi... - 09:09 AM Base Language Bug #2061 (Closed): unreachable code is too aggressive when shared vars are used
- Everything looks good here, except the copyright dates in the new ruleset. You can check in a fix without any furthe...
- 08:00 AM Base Language Bug #2061 (Review): unreachable code is too aggressive when shared vars are used
- This has passed conversion regression testing on MAJIC, no resurfaced dead-code.
Committed to bzr revision 10218. - 06:28 AM Base Language Bug #2061: unreachable code is too aggressive when shared vars are used
- I'm reposting cause I had a left behind printfln.
- 06:11 AM Base Language Bug #2061: unreachable code is too aggressive when shared vars are used
- My reachable processing was too aggressive, I'm putting this into conversion regression testing.
- 08:09 PM Base Language Bug #2061: unreachable code is too aggressive when shared vars are used
- I've checked converted sources, there is some resurrected dead-code in MAJIC that should not affect runtime; MAJIC di...
- 07:02 PM Base Language Bug #2061: unreachable code is too aggressive when shared vars are used
- renamed to not conflict...
- 07:00 PM Base Language Bug #2061: unreachable code is too aggressive when shared vars are used
- This code fixes this testcase, I will test the server file and regression conversion testing.
- 12:07 AM Database Bug #2062 (Closed): buffer parameter names doubled up, not included in RecordBuffer.openScope() call
- The following test:...
- 12:01 AM Database Bug #2062 (Closed): buffer parameter names doubled up, not included in RecordBuffer.openScope() call
- 08:00 PM Base Language Bug #2060 (Closed): forward function definitions that include a parameter name can "fixup" missin...
- 08:00 PM Base Language Bug #2060: forward function definitions that include a parameter name can "fixup" missing paramet...
- Passed conversion and is checked into bzr as revision 10216.
02/26/2013
- 06:53 PM Database Feature #2000: add conversion support for table and table handle parameters
- > In regards to issue number 4, can you explain what the code in th-proc.p is supposed to do?
> This code seems like... - 06:08 PM Database Feature #2000: add conversion support for table and table handle parameters
- Heh, it look like BUFFER parameter is misconverted, but not because of me:...
- 05:33 PM Database Feature #2000: add conversion support for table and table handle parameters
- > Hm...actually, a BUFFER parameter on a temp-table won't cause a call to TemporaryBuffer.associate. Is there still ...
- 05:28 PM Database Feature #2000: add conversion support for table and table handle parameters
- Stanislav Lomany wrote:
> > What about BUFFER parameters on temp-tables?
>
> Yes, I missed this type of parameter... - 04:58 PM Database Feature #2000: add conversion support for table and table handle parameters
- > What about BUFFER parameters on temp-tables?
Yes, I missed this type of parameters when I was analyzing the impa... - 04:56 PM Database Feature #2000: add conversion support for table and table handle parameters
- Stanislav Lomany wrote:
> >In record_scoping.rules, you changed the bufclassname annotation to TableParameter from T... - 04:47 PM Database Feature #2000: add conversion support for table and table handle parameters
- In regards to issue number 4, can you explain what the code in th-proc.p is supposed to do?...
- 04:43 PM Database Feature #2000: add conversion support for table and table handle parameters
- >In record_scoping.rules, you changed the bufclassname annotation to TableParameter from Temporary. Isn't that going ...
- 04:37 PM Database Feature #2000: add conversion support for table and table handle parameters
- In record_scoping.rules, you changed the bufclassname annotation to TableParameter from Temporary. Isn't that going ...
- 04:12 PM Database Feature #2000: add conversion support for table and table handle parameters
- > Did you intend to remove this from the define_stmt rule in progress.g?
Yes. I don't know why it was there, proba... - 04:09 PM Database Feature #2000: add conversion support for table and table handle parameters
- Did you intend to remove this from the define_stmt rule in progress.g?...
- 04:08 PM Database Feature #2000: add conversion support for table and table handle parameters
- #1 - yes, working on it
#2 - emit TableParameter when calling a function, working on it
#3 - yes, it works fine
#4... - 04:00 PM Database Feature #2000: add conversion support for table and table handle parameters
- Stanislav Lomany wrote:
> What remains is:
> 1. procedure TABLE-HANDLE calls using TableParameters, calling side - ... - 03:44 PM Database Feature #2000: add conversion support for table and table handle parameters
- I'll add javadocs later.
- 01:19 PM Database Feature #2000: add conversion support for table and table handle parameters
- Attached is the parser change to accept the APPEND syntax on a function call.
- 06:37 PM Base Language Bug #2061: unreachable code is too aggressive when shared vars are used
- I think the solution is to not remove vars/buffers/etc marked as unreachable if there are reachable nodes with a refi...
- 06:00 PM Base Language Bug #2061: unreachable code is too aggressive when shared vars are used
- unreachable code is removed and with it defined vars. following conversion fails:...
- 05:58 PM Base Language Bug #2061 (Closed): unreachable code is too aggressive when shared vars are used
- 05:56 PM User Interface Feature #2058 (Closed): COLOR DISPLAY .. FIELD statement fails
- 04:20 PM User Interface Feature #2058: COLOR DISPLAY .. FIELD statement fails
- Committed to bzr revision 10214.
- 03:58 PM User Interface Feature #2058: COLOR DISPLAY .. FIELD statement fails
- There is a buffer scope change in so/serv53-r.p for buffer co-bnchmrk, but this is expected, as an UNDERLINE statemen...
- 01:57 PM User Interface Feature #2058: COLOR DISPLAY .. FIELD statement fails
- The changes look good, I'm putting this into conversion regression testing.
- 01:23 PM User Interface Feature #2058: COLOR DISPLAY .. FIELD statement fails
- Added proposed update
- 12:57 PM User Interface Feature #2058: COLOR DISPLAY .. FIELD statement fails
- The following statements work the same as COLOR DISPLAY statement (no buffer is used):...
- 11:59 AM User Interface Feature #2058: COLOR DISPLAY .. FIELD statement fails
- Statements like :
@COLOR DISPLAY red book.price.@
This fails with:... - 11:57 AM User Interface Feature #2058 (Closed): COLOR DISPLAY .. FIELD statement fails
- 05:55 PM Database Bug #2057 (Closed): buffer unique name disambiguation (separator problems)
- 02:16 PM Database Bug #2057: buffer unique name disambiguation (separator problems)
- Passed conversion regression testing, committed to bzr revision 10213.
- 02:13 PM Database Bug #2057: buffer unique name disambiguation (separator problems)
- attached correct update, previous one was for another task (will delete it)
- 02:08 PM Database Bug #2057: buffer unique name disambiguation (separator problems)
- The conversion regression testing has passed, the only diffs were related to instance field order (RecordBuffer.defin...
- 12:15 PM Database Bug #2057: buffer unique name disambiguation (separator problems)
- Constantin Asofiei wrote:
> Eric, one more question: if you can, please explain why BufferScopeWorker.getBuffer atte... - 11:45 AM Database Bug #2057: buffer unique name disambiguation (separator problems)
- Update: with the BuffescScopeWorker.getBuffer changed I mentioned earlier, conversion of app folder goes through all ...
- 11:20 AM Database Bug #2057: buffer unique name disambiguation (separator problems)
- Eric, one more question: if you can, please explain why BufferScopeWorker.getBuffer attempts to match the buffer usin...
- 11:00 AM Database Bug #2057: buffer unique name disambiguation (separator problems)
- Eric, the code converts because our test db is named "p2j_test", with an underscore... I had to change the DB name to...
- 10:59 AM Database Bug #2057: buffer unique name disambiguation (separator problems)
- I realize you're the assignee for this, but it looked like it might be related to what I fixed last night, so I conve...
- 10:10 AM Database Bug #2057: buffer unique name disambiguation (separator problems)
- We have a case where an existing buffer defined for a table, like:...
- 10:00 AM Database Bug #2057 (Closed): buffer unique name disambiguation (separator problems)
- 05:54 PM Database Feature #2055 (Closed): RECID AND ROWID IN WHERE CLAUSE
- 12:37 PM Database Feature #2055: RECID AND ROWID IN WHERE CLAUSE
- Commited to bzr as revision 10212.
- 12:26 PM Database Feature #2055: RECID AND ROWID IN WHERE CLAUSE
- Conversion regression testing has passed, please check in and distribute update.
- 11:08 AM Database Feature #2055: RECID AND ROWID IN WHERE CLAUSE
- Eliminated the problems from previous update
- 10:17 AM Database Feature #2055: RECID AND ROWID IN WHERE CLAUSE
- Beside the header in RecidExpression (where you reference RowidExpression), I don't see any problems. I'm putting thi...
- 10:14 AM Database Feature #2055: RECID AND ROWID IN WHERE CLAUSE
- I'm reviewing this now.
- 09:59 AM Database Feature #2055: RECID AND ROWID IN WHERE CLAUSE
- Derived a case from that for recid
Added proposed update. - 09:11 AM Database Feature #2055: RECID AND ROWID IN WHERE CLAUSE
- Costin, use this test:...
- 08:53 AM Database Feature #2055: RECID AND ROWID IN WHERE CLAUSE
- Is there any case for RECID I can use?
For ROWID I testedn the changes with something like @FIND FIRST book WHERE TO... - 05:01 AM Database Feature #2055: RECID AND ROWID IN WHERE CLAUSE
- Next error is in .../vcpl/daqueau0.p:...
- 04:59 AM Database Feature #2055 (Closed): RECID AND ROWID IN WHERE CLAUSE
- 05:53 PM Database Bug #2052 (Closed): conversion error resolving master temp-table buffer
- 07:48 PM Database Bug #2052: conversion error resolving master temp-table buffer
- This update has passed conversion regression testing and is checked into bzr revision 10205.
- 05:01 PM Base Language Feature #1884: add some of the v10 data types and core built-ins
- Passed conversion regression testing, committed to bzr revision 10215.
- 03:25 PM Base Language Feature #1884: add some of the v10 data types and core built-ins
- The server project has a case of:...
- 04:15 PM Base Language Bug #2060: forward function definitions that include a parameter name can "fixup" missing paramet...
- Attached is the update that resolves this issue. It will duplicate the non-forward function definition's buffer para...
- 02:58 PM Base Language Bug #2060: forward function definitions that include a parameter name can "fixup" missing paramet...
- I tested various combinations of other parameter types:
- the regular AS "primitive" types (CHAR, INT, LOGICAL...)... - 12:55 PM Base Language Bug #2060: forward function definitions that include a parameter name can "fixup" missing paramet...
- In #1985 (see notes 29, 36 - 43 in that task) we found that forward function definitions that include a parameter nam...
- 12:52 PM Base Language Bug #2060 (Closed): forward function definitions that include a parameter name can "fixup" missin...
- 02:36 PM Base Language Bug #2056: null value for an annotation while persisting the AST
- The NPE is thrown because the function "copy_as_widget" attempts to duplicate an incomplete (missing "getter" annotat...
- 10:13 AM Base Language Bug #2056: null value for an annotation while persisting the AST
- Ovidiu, for now try and fix the "display c @ b as datatype" case, leave that for later.
- 10:12 AM Base Language Bug #2056: null value for an annotation while persisting the AST
- I looked into manuals about the @base-field specification, but they only describe how data is displayed into the resp...
- 08:25 AM Base Language Bug #2056: null value for an annotation while persisting the AST
- Note that the following case is converted correctly by P2J:...
- 08:17 AM Base Language Bug #2056: null value for an annotation while persisting the AST
- This is a task that tracks the issues reported by Eric in #1985, note 19.
I tried to eliminate the issue by pinpoint... - 08:05 AM Base Language Bug #2056 (Closed): null value for an annotation while persisting the AST
- 12:45 PM Database Bug #2059 (WIP): CAN-FIND clause for a perm table in the RECORD_PHRASE of a temp table, with a FO...
- The following code fails at conversion time:...
- 12:43 PM Database Bug #2059 (Closed): CAN-FIND clause for a perm table in the RECORD_PHRASE of a temp table, with a...
- 12:14 PM Database Feature #1655: add conversion and runtime support for certain dynamic database attributes and met...
- I have updated the p2jpl.ddr and have applied it to staging. It has no conversion impact, only runtime. Runtime regr...
- 12:13 PM Base Language Feature #2050: ATTR_POLY/METH_POLY used as rvalue or in expressions
- Passed conversion testing and is checked in as bzr revision 10211. Runtime recursion testing is deferred.
- 10:57 AM Base Language Feature #2050: ATTR_POLY/METH_POLY used as rvalue or in expressions
- The issue is caused by the over-simplistic recursion protection in our context analysis. In addition, even in the ca...
- 10:53 AM Base Language Feature #2050: ATTR_POLY/METH_POLY used as rvalue or in expressions
- From #1985:
ra/ folder fails with:... - 03:42 AM Base Language Feature #2050: ATTR_POLY/METH_POLY used as rvalue or in expressions
- > Somehow, the improved expression typing changes the INPUT FUNC_POLY processing. As far as I know, these are equiva...
- 08:52 PM Base Language Feature #2050: ATTR_POLY/METH_POLY used as rvalue or in expressions
- I believe this is passing conversion testing.
The following diffs are present:... - 07:50 PM Base Language Feature #2050: ATTR_POLY/METH_POLY used as rvalue or in expressions
- Yes, it does seem to resolve the problem in raam/. Now we get a different error (see #1985).
- 07:48 PM Base Language Feature #2050: ATTR_POLY/METH_POLY used as rvalue or in expressions
- I have implemented major changes to ExpressionConversionWorker.expressionType() to do the following:
1. Add suppor... - 10:40 AM Base Language Feature #1920: implement persistent procedures
- Conversion regression testing has passed, committed to bzr revision 10210.
- 09:05 AM Base Language Feature #1920: implement persistent procedures
- It looks good.
- 08:58 AM Base Language Feature #1920: implement persistent procedures
- The removeQuotes change, I'm running regression conversion testing again.
- 08:55 AM Base Language Feature #1920: implement persistent procedures
- OK, so I'm dropping the functions.rules change and I'll change CommonAstSupport.removeQuotes. Note that removeQuotes ...
- 08:44 AM Base Language Feature #1920: implement persistent procedures
- OK. You can use the same approach as progressToJavaString() for dropping the quotes (and any following options):
<... - 08:41 AM Base Language Feature #1920: implement persistent procedures
- About issue 1: I don't think is OK to convert the string to Java, as currently I need to match with a Progress legacy...
- 08:35 AM Base Language Feature #1920: implement persistent procedures
- Code Feedback:
1. Strings can be qualified with more than just the @:U@. Your code assumes there can only be 1 qu... - 07:33 AM Base Language Feature #1920: implement persistent procedures
- No changes in generated MAJIC sources. Greg, if the update looks good to you, I will release it.
- 05:55 AM Base Language Feature #1920: implement persistent procedures
- Handle the case of single quotes used with function names and others (removeQuotes was improved to remove enclosing s...
- 05:44 AM Base Language Feature #1920: implement persistent procedures
- This update fixes a bug in dynamic-function's func name resolution, for cases like:...
- 08:21 AM Base Language Feature #2048 (Closed): BUFFER parameters without name in forward function definitions
- 06:17 AM Base Language Feature #2048: BUFFER parameters without name in forward function definitions
- Commited to bazaar as revision number 10209.
- 05:44 AM Base Language Feature #2048: BUFFER parameters without name in forward function definitions
- Update has passed conversion regression testing, please check in and distribute.
- 04:46 AM Base Language Feature #2048: BUFFER parameters without name in forward function definitions
- The update looks OK, is going through conversion regression testing now.
- 04:44 AM Base Language Feature #2048: BUFFER parameters without name in forward function definitions
- Added proposed update
- 04:05 AM Database Feature #1668: add support for additional (non-dynamic) database attributes
- Added conversion support for EMPTY-TEMP-TABLE and BUFFER-DELETE (see ecf_upd20130226a.zip at #1985). This update pas...
- 02:34 AM Database Bug #2054 (Closed): conversion errors with complex buffer parameters and definitions
- The fix has passed conversion regression testing. It is checked into bzr revision 10207.
- 02:08 AM Database Bug #2054: conversion errors with complex buffer parameters and definitions
- The following testcases produce buffers and fields with the incorrect schemaname annotation, such that downstream pro...
- 01:59 AM Database Bug #2054 (Closed): conversion errors with complex buffer parameters and definitions
- 07:58 PM Base Language Feature #2051 (Closed): conversion support for ATTR_POLY/METH_POLY used as rvalue or in expression
02/25/2013
- 04:51 PM Database Bug #2052: conversion error resolving master temp-table buffer
- Turns out the previous update fixed the symptom rather than the root cause. Turns out we were not handling empty int...
- 03:40 PM Database Bug #2052: conversion error resolving master temp-table buffer
- Constantin Asofiei wrote:
> This testcase is related to this issue:
> [...]
>
> The buffers get reordered, but n... - 03:08 PM Database Bug #2052: conversion error resolving master temp-table buffer
- This testcase is related to this issue:...
- 02:59 PM Database Bug #2052: conversion error resolving master temp-table buffer
- OK, I'll try postponing the DEFINE BUFFER lookups until all the DEFINE TEMP-TABLE statements in a file have been visi...
- 02:56 PM Database Bug #2052: conversion error resolving master temp-table buffer
- I think this is related to annotations/record_scoping_post.rules H008.
- 02:49 PM Database Bug #2052: conversion error resolving master temp-table buffer
- The error occurs because there is an expectation of a DEFINE TEMP-TABLE xyz statement appearing earlier in the AST th...
- 02:18 PM Database Bug #2052 (Closed): conversion error resolving master temp-table buffer
- 03:37 PM Database Feature #2047 (Closed): field references in procedures without explicit buffer scope
- 03:15 PM Database Feature #2047: field references in procedures without explicit buffer scope
- Committed to bzr revision 10204.
- 03:02 PM Database Feature #2047: field references in procedures without explicit buffer scope
- Check it in and distribute it.
- 02:42 PM Database Feature #2047: field references in procedures without explicit buffer scope
- The conversion has finished and the generated sources look good to me (only some reordering in openScope and buffer i...
- 01:32 PM Database Feature #2047: field references in procedures without explicit buffer scope
- I had a copy-paste error in SymbolResolver, I saved bufname instead of dbname in the "dbname" annotation. I'm running...
- 11:40 AM Database Feature #2047: field references in procedures without explicit buffer scope
- I'm a little nervous about the change BUT I think it should be OK. I'm going to put it into conversion testing now.
- 11:01 AM Database Feature #2047: field references in procedures without explicit buffer scope
- Fix attempt for scope computation for fields in procedures/functions without explicit buffer scope created by other s...
- 09:39 AM Database Feature #2047: field references in procedures without explicit buffer scope
- This one is related to #1706. Looking at the perm buffer scoping from some compile listings from 4GL v9 and new ones ...
- 08:53 AM Database Feature #2047: field references in procedures without explicit buffer scope
- Looks like 4GL allows a buffer reference in a procedure, without having its scope set, as it inherits the buffer scop...
- 08:51 AM Database Feature #2047 (Closed): field references in procedures without explicit buffer scope
- 03:34 PM Conversion Tools Feature #2053: create a parameter type lookup facility for built-in functions and handle based me...
- Consider these classes:...
- 03:19 PM Conversion Tools Feature #2053 (Closed): create a parameter type lookup facility for built-in functions and handle...
- 02:36 PM Base Language Feature #1647: implement an option for SETUSERID and USERID to be backed by a replacement for the...
- Committed in bzr as 10203.
- 02:27 PM Base Language Feature #1647: implement an option for SETUSERID and USERID to be backed by a replacement for the...
- Conversion testing passed. Please check this in and distribute it.
- 12:49 PM Base Language Feature #1647: implement an option for SETUSERID and USERID to be backed by a replacement for the...
- This looks good. I will put it into conversion testing now.
- 12:41 PM Base Language Feature #1647: implement an option for SETUSERID and USERID to be backed by a replacement for the...
- The suggested update has been uploaded. Also I have appended simple test and conversion result. The code drop merged ...
- 09:49 AM Base Language Feature #1647: implement an option for SETUSERID and USERID to be backed by a replacement for the...
- I should have provided more information previously, sorry.
For now, all that needs to be done is to add support fo... - 09:42 AM Base Language Feature #1647: implement an option for SETUSERID and USERID to be backed by a replacement for the...
- Let me clarify some points.
1. Is the _USER table already implemented in P2J?
2. My work is to add conversion suppo... - 02:27 PM Base Language Bug #2049 (Closed): SET statement and NO-ERROR clause
- 02:25 PM Base Language Bug #2049: SET statement and NO-ERROR clause
- Conversion testing has passed. I have checked it in as bzr revision 10102.
- 12:11 PM Base Language Bug #2049: SET statement and NO-ERROR clause
- But since I haven't tested that assumption (that no-error always will appear at the end), I think it is best to go wi...
- 12:09 PM Base Language Bug #2049: SET statement and NO-ERROR clause
- Added the new archive, I will remove the older, unneeded one.
I believe that @isWidget@ is used as a flag when sca... - 11:37 AM Base Language Bug #2049: SET statement and NO-ERROR clause
- It is not clear to me whether we should have a section like this:...
- 11:15 AM Base Language Bug #2049 (Review): SET statement and NO-ERROR clause
- Looks like the @N0-ERROR@ clause was treated as an widget in the list of parameters of the statements (@SET@, @ASSIGN...
- 11:04 AM Base Language Bug #2049 (WIP): SET statement and NO-ERROR clause
- 08:58 AM Base Language Bug #2049: SET statement and NO-ERROR clause
- Conversion fails in a case like...
- 08:57 AM Base Language Bug #2049 (Closed): SET statement and NO-ERROR clause
- 02:20 PM Base Language Feature #2048: BUFFER parameters without name in forward function definitions
- Keep in mind that the FORWARD declaration should not be changing buffer scoping at all. My guess is that somewhere w...
- 02:19 PM Base Language Feature #2048: BUFFER parameters without name in forward function definitions
- Actually, both results look wrong. doubler2() gets an openScope() with the same buffer name passed twice. That may ...
- 01:59 PM Base Language Feature #2048: BUFFER parameters without name in forward function definitions
- The conversion fails because of code in record_scoping_prep.rules which assumes the buffer annotation is in the copy....
- 08:56 AM Base Language Feature #2048: BUFFER parameters without name in forward function definitions
- Conversion fails in this case:...
- 08:53 AM Base Language Feature #2048 (Closed): BUFFER parameters without name in forward function definitions
- 01:01 PM Database Feature #1992 (Closed): add support for double-colon (::) syntax
- 12:35 PM Database Feature #1992: add support for double-colon (::) syntax
- I have chosen not to alter in any way the update that have passed the testing (om_upd20130225a.zip) and committed it ...
- 11:38 AM Database Feature #1992: add support for double-colon (::) syntax
- Your update (including my change to progress.g) has passed conversion regression testing. I am attaching my updated ...
- 10:55 AM Database Feature #1992: add support for double-colon (::) syntax
- > There was one change to progress.g that seemed unnecessary: after discussion with Greg, I removed the actions in t...
- 10:43 AM Database Feature #1992: add support for double-colon (::) syntax
- Code review 20130225a:
Nice work! I am running it through conversion regression testing now.
There was one cha... - 06:19 AM Database Feature #1992: add support for double-colon (::) syntax
- Latest :: conversion merged with bzr revision 10199 and updated testcase.
- 11:47 AM Database Feature #1655: add conversion and runtime support for certain dynamic database attributes and met...
- Greg Shah wrote:
> Final conversion testing passed. This is checked into bzr as 10200.
This will fail runtime reg... - 10:21 AM Database Feature #1655: add conversion and runtime support for certain dynamic database attributes and met...
- Final conversion testing passed. This is checked into bzr as 10200.
- 09:15 AM Database Feature #1655: add conversion and runtime support for certain dynamic database attributes and met...
- Initial conversion testing passed. Attached is the update merged up to bzr 10199. This is going back into conversio...
- 11:21 AM Base Language Feature #2050: ATTR_POLY/METH_POLY used as rvalue or in expressions
- The issue only occurs for those ATTR_POLY/FUNC_POLY/METH_POLY cases where we cannot detect the type based on the chil...
- 10:20 AM Base Language Feature #2050: ATTR_POLY/METH_POLY used as rvalue or in expressions
- I wonder if ExpressionConversionWorker should ignore these _POLY cases ? And determine the expression type based on t...
- 10:16 AM Base Language Feature #2050: ATTR_POLY/METH_POLY used as rvalue or in expressions
- The problem I was working (in #1655) is the same problem. I "fixed" it by eliminating the use of expressionType() in...
- 10:02 AM Base Language Feature #2050: ATTR_POLY/METH_POLY used as rvalue or in expressions
- Conversion for ATTR_POLY used in complex expressions fails, as the ExpressionConversionWorker can't determine the typ...
- 09:56 AM Base Language Feature #2050 (Closed): ATTR_POLY/METH_POLY used as rvalue or in expressions
- 09:57 AM Base Language Feature #2051: conversion support for ATTR_POLY/METH_POLY used as rvalue or in expression
- This is for milestone 4 time tracking only. All comments must be added in parent task #2050
- 09:57 AM Base Language Feature #2051 (Closed): conversion support for ATTR_POLY/METH_POLY used as rvalue or in expression
- 02:43 AM Database Feature #1975: add runtime support for (non-dynamic) database attributes
- Log all history in parent task.
- 02:37 AM Database Feature #2042 (Closed): add conversion support for some embedded SQL features
- 02:34 AM Database Feature #1588: add conversion and runtime support for embedded SQL
- Added support for the simple, single table (and column) constructs listed above. These convert to methods in Buffer....
02/24/2013
- 05:29 PM Database Feature #1655: add conversion and runtime support for certain dynamic database attributes and met...
- A problem has occurred in the use of the ATTR_POLY BUFFER-VALUE (KW_BUF_VAL) as a parameter to an INTEGER() built-in ...
- 05:08 PM Database Feature #2045 (Closed): conversion support for a DELETE statement with a validation expression
- 03:18 PM Database Feature #2045: conversion support for a DELETE statement with a validation expression
- This is for time tracking and milestone association only. All history should be in the parent task #2044.
- 03:17 PM Database Feature #2045 (Closed): conversion support for a DELETE statement with a validation expression
- 05:08 PM Database Feature #2044: implement support for a DELETE statement with a validation expression
- It passed conversion testing and is committed to bzr as revision 10198.
- 04:08 PM Database Feature #2044: implement support for a DELETE statement with a validation expression
- And now the update...
- 04:08 PM Database Feature #2044: implement support for a DELETE statement with a validation expression
- I fixed the header and have re-attached the update. This is going into conversion regression testing now.
- 04:04 PM Database Feature #2044: implement support for a DELETE statement with a validation expression
- Code review 20130224a:
Changes look OK to me. Note that database_access.rules needs a header entry for your changes. - 03:31 PM Database Feature #2044: implement support for a DELETE statement with a validation expression
- Add support to the conversion and runtime for a DELETE statement with a validation expression. We already support DE...
- 03:17 PM Database Feature #2044 (Closed): implement support for a DELETE statement with a validation expression
- 03:52 PM Base Language Feature #1627 (Closed): support for empty EXPORT statements
- 03:43 PM Base Language Feature #1627: support for empty EXPORT statements
- The evl_upd20130224a.zip has been committed in bzr as 10197.
- 03:32 PM Base Language Feature #1627: support for empty EXPORT statements
- This has passed conversion testing. Go ahead and check it in and distribute it.
- 02:33 PM Base Language Feature #1627: support for empty EXPORT statements
- The code looks good. I will run conversion testing now.
- 01:43 PM Base Language Feature #1627: support for empty EXPORT statements
- The modified post_parse_fixups.xml and new rule have been uploaded to eliminate empty export. Also the converted code...
- 03:19 PM Database Feature #2046: runtime support for DELETE statement with a validation expression
- This is for time tracking and milestone association only. All history should be in the parent task #2044.
- 03:19 PM Database Feature #2046 (Closed): runtime support for DELETE statement with a validation expression
- 10:31 AM User Interface Feature #2016: implement CURRENT-WINDOW, ACTIVE-WINDOW and DEFAULT-WINDOW system handles
- Checked in as bzr revision 10196.
- 08:54 AM User Interface Feature #2016: implement CURRENT-WINDOW, ACTIVE-WINDOW and DEFAULT-WINDOW system handles
- Attached update makes the window system handles to convert as true handles; also, I've removed the THIS-PROCEDURE hac...
- 07:19 AM User Interface Feature #2016: implement CURRENT-WINDOW, ACTIVE-WINDOW and DEFAULT-WINDOW system handles
- I was thinking for the implementation of the window system-handles to allow them to convert to an API which returns t...
- 07:06 PM User Interface Feature #2016: implement CURRENT-WINDOW, ACTIVE-WINDOW and DEFAULT-WINDOW system handles
- There are major changes in Majic. And those changes don't compile. This is an example change:...
- 10:27 AM User Interface Feature #2036 (Closed): conversion support for some frame options
- 10:27 AM User Interface Feature #1801: add some frame options
- Passed conversion testing and checked into bzr as 10195.
- 09:24 AM User Interface Feature #1801: add some frame options
- I wish you were not right but clearly you are. Luckily for us, you are right often. :) But being right this time me...
- 06:50 AM User Interface Feature #1801: add some frame options
- Looks like my instinct was correct. Using this test:...
- 09:29 AM Conversion Tools Feature #1603: enhance name conversion to support internal procedure names that contain a '.' cha...
- OK, for now this is done. I am going to move this to another milestone. The only remaining work is to remove the ha...
- 08:00 AM Conversion Tools Feature #1603: enhance name conversion to support internal procedure names that contain a '.' cha...
- The only real problem is when converting the 4GL proc name, to make the name compatbile with Java syntax. As we don't...
- 07:18 PM Conversion Tools Feature #1603: enhance name conversion to support internal procedure names that contain a '.' cha...
- Interesting. I wonder what is going on there. I can duplicate your result as well.
But when I used the testcases... - 06:16 AM Base Language Feature #2013: implement the SELF system handle
- Committed to bzr revision 10194, passed runtime regression testing.
02/23/2013
- 05:53 PM User Interface Feature #1801: add some frame options
- In regard to the COLOR option, all of the cases are "custom" COLOR names (the protermcap-attribute). VALUE() is not ...
- 03:01 PM User Interface Feature #1801: add some frame options
- This update adds support for:
# STREAM-IO clause
# FONT clause
# IN WIDGET clause
Note that only the static ver... - 02:22 PM User Interface Feature #1801: add some frame options
- > # IN WINDOW <handle-expression>
> This should convert to a @frame.[update|display|set|etc](elementList#, <handle-e... - 01:16 PM User Interface Feature #1801: add some frame options
- # COLOR
syntax:... - 05:43 PM Base Language Feature #2013: implement the SELF system handle
- The change looks fine. This is being conversion tested now.
- 01:30 PM Base Language Feature #2013: implement the SELF system handle
- Fixed initialization of context-local data in SelfManager.
- 10:14 PM Base Language Feature #2013: implement the SELF system handle
- Passed conversion testing. Checked into bzr as 10189.
- 05:41 PM User Interface Feature #2016: implement CURRENT-WINDOW, ACTIVE-WINDOW and DEFAULT-WINDOW system handles
- The change looks fine. This is being conversion tested now.
- 12:29 PM User Interface Feature #2016: implement CURRENT-WINDOW, ACTIVE-WINDOW and DEFAULT-WINDOW system handles
- Fixed the issues at comment 12.
- 12:24 PM User Interface Feature #2016: implement CURRENT-WINDOW, ACTIVE-WINDOW and DEFAULT-WINDOW system handles
- OK, please do fix it.
- 12:23 PM User Interface Feature #2016: implement CURRENT-WINDOW, ACTIVE-WINDOW and DEFAULT-WINDOW system handles
- There is a problem when CURRENT/DEFAULT/ACTIVE-WINDOW are not used in an assignment, as in:...
- 10:23 AM User Interface Feature #2016: implement CURRENT-WINDOW, ACTIVE-WINDOW and DEFAULT-WINDOW system handles
- Committed to bzr revision 10191.
- 10:17 AM User Interface Feature #2016: implement CURRENT-WINDOW, ACTIVE-WINDOW and DEFAULT-WINDOW system handles
- Conversion testing has passed. Please check it in to bzr and distribute via email.
- 09:20 AM User Interface Feature #2016: implement CURRENT-WINDOW, ACTIVE-WINDOW and DEFAULT-WINDOW system handles
- The code looks good. It is in conversion testing now.
- 08:59 AM User Interface Feature #2016: implement CURRENT-WINDOW, ACTIVE-WINDOW and DEFAULT-WINDOW system handles
- Yes, is merged and ready for testing.
- 08:58 AM User Interface Feature #2016: implement CURRENT-WINDOW, ACTIVE-WINDOW and DEFAULT-WINDOW system handles
- I assume this update is merged up to bzr 10190? It is ready for testing?
- 05:39 AM User Interface Feature #2016: implement CURRENT-WINDOW, ACTIVE-WINDOW and DEFAULT-WINDOW system handles
- Conversion regression testing passed on my system (no changes in generated code).
- 02:30 AM User Interface Feature #2016: implement CURRENT-WINDOW, ACTIVE-WINDOW and DEFAULT-WINDOW system handles
- > Is the assignment of DEFAULT-WINDOW and ACTIVE-WINDOW (and SESSION) silently tolerated (and ignored) at runtime in ...
- 05:27 PM Base Language Feature #2021 (Closed): conversion support for web services
- 05:26 PM Base Language Feature #1645: implement web services support
- With this change, testing passed. Checked into bzr as 10193.
- 04:19 PM Base Language Feature #1645: implement web services support
- This update causes a problem with many (all?) RUN statements. Here is a common example diff (this is broken througho...
- 12:21 PM Base Language Feature #1645: implement web services support
- This is in testing now.
- 11:54 AM Base Language Feature #1645: implement web services support
- It looks good. This will go into conversion testing next.
- 11:46 AM Base Language Feature #1645: implement web services support
- Attached update fixes the refAsync issue and refSet.move (which should move to copy not refAsync)
- 11:39 AM Base Language Feature #1645: implement web services support
- Everything else looks OK with the code.
- 11:38 AM Base Language Feature #1645: implement web services support
- In fixups/control_flow.rules refAsync is never assigned, but it is relied upon in the ASYNCH ON case and in the SET h...
- 11:26 AM Base Language Feature #1645: implement web services support
- Built on top of #1782 23d.zip update. Fixed support for @RUN ... SET hPortType ON hWebServ@ statement. Added support ...
- 08:39 AM Base Language Feature #1645: implement web services support
- More info about RUN ... SET ... ON SERVER. Something like this works:...
- 05:28 AM Base Language Feature #1645: implement web services support
- Another later edit: it is possible to use the @SET hPortType@ specification even with the @TRANSACTION DISTINCT@ and ...
- 04:51 AM Base Language Feature #1645: implement web services support
- > fix the @RUN SET ... ON SERVER@ statement
Look like the ASYNC clause is supported (at least at compile-time) for th... - 04:28 AM Base Language Feature #1645: implement web services support
- For the Server object handle, all documented attributes/methods are supported except these:
# REQUEST-INFO, RESPONSE... - 04:39 PM Database Feature #2042: add conversion support for some embedded SQL features
- Log history in parent issue.
- 03:27 PM Database Feature #2042 (Closed): add conversion support for some embedded SQL features
- 04:39 PM Database Feature #2043: add runtime support for some embedded SQL features
- Log history in parent issue.
- 04:38 PM Database Feature #2043 (Closed): add runtime support for some embedded SQL features
- 03:24 PM Conversion Tools Feature #1603: enhance name conversion to support internal procedure names that contain a '.' cha...
- I'm not sure what is needed here. The following converts nicely in P2J:...
- 12:20 PM Conversion Tools Feature #2001: reporting v2.5
- This improvement to the reports is designed to massively improve performance. This was needed because on large proje...
- 12:14 PM User Interface Feature #2019 (Closed): conversion support for LAST-EVENT system handle and attributes
- 12:13 PM User Interface Feature #1782: implement LAST-EVENT system handle and associated attributes
- Committe to bzr revision 10192.
- 12:06 PM User Interface Feature #1782: implement LAST-EVENT system handle and associated attributes
- This passed testing. Please check it in and distribute it.
- 11:06 AM User Interface Feature #1782: implement LAST-EVENT system handle and associated attributes
- Looks good. It is going into conversion testing now.
- 10:51 AM User Interface Feature #1782: implement LAST-EVENT system handle and associated attributes
- Beside LABEL, the WIDTH/HEIGHT-CHARS/PIXELS attrs had to be duped in the main section too.
- 10:41 AM User Interface Feature #1782: implement LAST-EVENT system handle and associated attributes
- Correct, any attribute/method used with a widget should be in the main section too. I'm going through all the attrs/m...
- 10:29 AM User Interface Feature #1782: implement LAST-EVENT system handle and associated attributes
- Since the getLabel/setLabel is moved inside the check for LAST-EVENT, doesn't that disable using LABEL on a widget (o...
- 10:26 AM User Interface Feature #1782: implement LAST-EVENT system handle and associated attributes
- Merged with revision 10191.
- 06:29 AM User Interface Feature #1782: implement LAST-EVENT system handle and associated attributes
- LAST-EVENT handle is supported in P2J by using KeyReader.asHandle (when we have a @h = LAST-EVENT@ case). For explici...
- 11:51 AM Base Language Feature #2041 (Closed): implement runtime support for "shared" attributes that are just stubbed
- 11:50 AM Base Language Feature #2002 (Closed): refactor some "shared" attributes
- 11:15 AM User Interface Feature #2036 (WIP): conversion support for some frame options
- 11:11 AM Database Feature #1992: add support for double-colon (::) syntax
- OK, please add a suitable task to the Base Language project and target it to milestone 7. Please put all the relevan...
- 10:55 AM Database Feature #1992: add support for double-colon (::) syntax
- For milestone 4, I don't think there is anything we need to do about the *_POLY rvalues, all -will- can be in the run...
- 10:48 AM Database Feature #1992: add support for double-colon (::) syntax
- > Idea is, if we have the ATTR/METH_POLY in an expression or as a rvalue, then 4GL does some automatic conversion of ...
- 10:25 AM User Interface Feature #2017 (Closed): conversion sypport for CURRENT-WINDOW, ACTIVE-WINDOW and DEFAULT-WINDOW s...
- 10:16 AM User Interface Feature #2029 (Closed): conversion support for the SIZE phrase
- 10:16 AM User Interface Feature #1800: add some missing widget options
- For milestone 4, that is fine.
- 06:19 AM User Interface Feature #1800: add some missing widget options
- The full syntax of the size phrase is:...
- 03:49 AM Database Feature #2000: add conversion support for table and table handle parameters
- I'm sending the update in its current state: INPUT/OUTPUT parameters and APPEND option are not handled, createDynamic...
- 03:00 AM Database Feature #2000: add conversion support for table and table handle parameters
- Stanislav, OK now I understand what you mean. APPEND clause used with OUTPUT/INPUT-OUTPUT table at the caller affects...
- 10:47 PM Database Feature #2000: add conversion support for table and table handle parameters
- Stas: please provide an update for all the other functionality discussed above ASAP, even if you don't yet have an a...
- 01:42 AM Database Feature #1660: add conversion and runtime support for OF keyword use in a CAN-FIND
- Vadim Nebogatov wrote:
> ... but conversion of
> ... - 12:38 AM Database Feature #1660: add conversion and runtime support for OF keyword use in a CAN-FIND
- I have created environment for tests and made some first conversion tests. Will try to make more tests on Monday.
... - 10:14 PM Base Language Feature #2014 (Closed): conversion support for SELF system handle
- 10:13 PM User Interface Feature #2025 (Closed): conversion support for the CREATE_WIDGET statement
- 10:13 PM User Interface Feature #1791: implement dynamic UI support
- Passed conversion testing. Checked into bzr as 10190.
02/22/2013
- 04:45 PM User Interface Feature #2016: implement CURRENT-WINDOW, ACTIVE-WINDOW and DEFAULT-WINDOW system handles
- Is the assignment of DEFAULT-WINDOW and ACTIVE-WINDOW (and SESSION) silently tolerated (and ignored) at runtime in th...
- 02:49 PM User Interface Feature #2016: implement CURRENT-WINDOW, ACTIVE-WINDOW and DEFAULT-WINDOW system handles
- Added intermediate update merged to latest revision, this should cover the cases showed here minus the case for which...
- 07:35 AM User Interface Feature #2016: implement CURRENT-WINDOW, ACTIVE-WINDOW and DEFAULT-WINDOW system handles
- What will be the approach for system handles besides CURRENT-WINDOW that are assigned as left value.
It seems that ... - 04:37 PM User Interface Feature #1791: implement dynamic UI support
- Merged up to bzr 10188. This version will be going into testing next.
- 02:18 PM User Interface Feature #1791: implement dynamic UI support
- Merged with #2013, fixed conversion of @CREATE VALUE(type)@ when the type is a known widget type.
- 01:47 PM User Interface Feature #1791: implement dynamic UI support
- Yes, please do merge it with the #2013 update. We will put them both in together.
- 01:42 PM User Interface Feature #1791: implement dynamic UI support
- > > I chose the approach of having dedicated APIs because IMO is cleaner, but I can switch to only one API DynamicWid...
- 01:15 PM User Interface Feature #1791: implement dynamic UI support
- > I chose the approach of having dedicated APIs because IMO is cleaner, but I can switch to only one API DynamicWidge...
- 01:07 PM User Interface Feature #1791: implement dynamic UI support
- > 1. Why not do everything using an enum: WidgetTypes.WINDOW, WidgetTypes.FILLIN...
In the end, we will still have t... - 12:52 PM User Interface Feature #1791: implement dynamic UI support
- No, leave the trigger phrase out for now.
2 questions:
1. Why not do everything using an enum: WidgetTypes.WIND... - 10:35 AM User Interface Feature #1791: implement dynamic UI support
- The trigger phrase was not touched - should I work on this too ?
- 10:11 AM User Interface Feature #1791: implement dynamic UI support
- Attached update adds CREATE WIDGET support (explicit and VALUE versions). Converted code looks like:...
- 09:15 AM User Interface Feature #1791: implement dynamic UI support
- Yes, I like the approach. The assignments for a CREATE WIDGET can be easily dealt with by implicitly using the handl...
- 05:06 AM User Interface Feature #1791: implement dynamic UI support
- The conversion rules to emit this were pretty simple (I have code for all except CREATE BROWSE), although I'm not sur...
- 04:31 PM Base Language Feature #2013: implement the SELF system handle
- Here is the right version.
- 04:27 PM Base Language Feature #2013: implement the SELF system handle
- This is the latest version merged up to bzr 10188. It will go into testing next.
- 02:17 PM Base Language Feature #2013: implement the SELF system handle
- > I think it might be more correct to say that the rule-set only handles the resource reference for handle types. In...
- 01:46 PM Base Language Feature #2013: implement the SELF system handle
- The code looks good....
- 01:33 PM Base Language Feature #2013: implement the SELF system handle
- > 1. refName is a dead variable and can be removed from methods_attributes.rules.
OK, removed
> 2. How do we emit t... - 12:34 PM Base Language Feature #2013: implement the SELF system handle
- Code Feedback:
1. refName is a dead variable and can be removed from methods_attributes.rules.
2. How do we emi... - 12:14 PM Base Language Feature #2013: implement the SELF system handle
- Conversion regression testing has passed - compilation was OK, all changes are expected.
- 06:39 AM Base Language Feature #2013: implement the SELF system handle
- Conversion has finished, there was a APPLY ... TO SELF problem, fixed in attached update. All other changes look good...
- 03:22 AM Base Language Feature #2013: implement the SELF system handle
- Added testcases (previous were for session handle)
- 03:20 AM Base Language Feature #2013: implement the SELF system handle
- -Added testcases.- they are for session handle, I've deleted it from here.
- 03:18 AM Base Language Feature #2013: implement the SELF system handle
- Fixed a conversion error, I'm running conversion again.
- 04:21 PM User Interface Feature #2005 (Closed): add conversion support for methods
- 04:20 PM User Interface Feature #2007 (Closed): conversion support for attributes
- 04:12 PM User Interface Feature #1804: add missing UI attributes
- The update evl_upd20130222c.zip has been committed in bzr as 10188.
- 04:04 PM User Interface Feature #1804: add missing UI attributes
- Testing passed. Please check it in and distribute the update. ASAP.
- 02:46 PM User Interface Feature #1804: add missing UI attributes
- Looks great. It is going into testing now.
- 02:43 PM User Interface Feature #1804: add missing UI attributes
- The merged version combined with #2005 changes has been uploaded.
- 02:21 PM User Interface Feature #1804: add missing UI attributes
- Is this also merged with the #2005 changes? I want them both in the same update.
- 02:10 PM User Interface Feature #1804: add missing UI attributes
- Merged with latest bzr uploaded.
- 01:26 PM User Interface Feature #1804: add missing UI attributes
- Please merge with the latest bzr revision and upload for a final review. Then we will go into conversion testing (ru...
- 03:09 PM Database Feature #1992: add support for double-colon (::) syntax
- All the specified cases work nicely.
> I'm not sure I understand what is being asked here. By definition, the ATTR... - 03:00 PM Database Feature #1992: add support for double-colon (::) syntax
- > in assignment case, when the rvalue is a ATTR/METH_POLY, are there cases when the lvalue is of a different type tha...
- 02:57 PM Database Feature #1992: add support for double-colon (::) syntax
- > are there any ATTR_POLY/METH_POLY used directly in a frame-related statement?
In regard to the FUNC_POLY/ATTR_PO... - 01:55 PM Database Feature #2000: add conversion support for table and table handle parameters
- > Please fill out this matrix:
When you specify APPEND in the calling procedure it logically applies to the table ... - 12:44 PM Database Feature #2000: add conversion support for table and table handle parameters
- > Progress documentation tells that "You can only use the APPEND option for an INPUT parameter in the calling procedu...
- 12:36 PM Database Feature #2000: add conversion support for table and table handle parameters
- Leave it as Temporary if you don't have a need to change it. Less runtime change. If we find we need it to be Buffe...
- 12:34 PM Database Feature #2000: add conversion support for table and table handle parameters
- So, Eric, do you prefer to leave Temporary or switch to Buffer?
Constantin, Greg, how do you prefer to pass APPEND... - 01:21 PM Base Language Feature #2003 (Closed): conversion support for SESSION handle
- 01:16 PM Base Language Feature #1612: finish SESSION system handle support
- Committed to bzr revision 10187.
- 01:03 PM Base Language Feature #1612: finish SESSION system handle support
- Conversion testing has passed. Please check in and distribute.
- 12:02 PM Base Language Feature #1612: finish SESSION system handle support
- The update looks good. I'm testing it now.
- 11:43 AM Base Language Feature #1612: finish SESSION system handle support
- Merged with revision 10186.
- 11:10 AM Base Language Feature #1612: finish SESSION system handle support
- The code looks fine. I'll re-check after the merge.
- 03:22 AM Base Language Feature #1612: finish SESSION system handle support
- Added tests.
- 03:19 AM Base Language Feature #1612: finish SESSION system handle support
- Merged with 10183, fixed some runtime errors too (ERROR-STATUS handle could not initialize its static proxy).
- 12:02 PM Database Feature #1974 (Closed): add conversion support for (non-dynamic) database attributes
- 11:53 AM Base Language Feature #2009 (Closed): conversion support for LANDSCAPE/PORTRAIT I/O option
- 11:49 AM Base Language Feature #2011 (Closed): conversion support for _MSG() built-in
- 11:46 AM Base Language Feature #1632: implement _MSG() undocumented built-in function
- Conversion tests passed , commited to bazaar as revision 10186
- 09:03 AM Base Language Feature #1632: implement _MSG() undocumented built-in function
- The update looks good. I will start testing it shortly.
- 11:27 AM Base Language Feature #1631: implement support for LANDSCAPE/PORTRAIT in I/O statements
- Conversion test passed, commited to bazaar as revision 10185
- 09:10 AM Base Language Feature #1631: implement support for LANDSCAPE/PORTRAIT in I/O statements
- The update looks good. I will start testing it shortly. I am deleting the update that is not matching, so that ther...
- 09:07 AM Base Language Feature #1631: implement support for LANDSCAPE/PORTRAIT in I/O statements
- Attached it
- 09:06 AM Base Language Feature #1631: implement support for LANDSCAPE/PORTRAIT in I/O statements
- This is the wrong update (it is for the _MSG work). Please post the correct one.
- 04:26 AM Base Language Feature #1631: implement support for LANDSCAPE/PORTRAIT in I/O statements
- Added merged update
- 11:14 AM Base Language Feature #1979 (Closed): conversion support for DOM XML processing
- 11:10 AM Base Language Feature #1640: implement DOM XML support
- The update evl_upd20130222a.zip passed the conversion test and committed in bzr as 10184.
- 09:02 AM Base Language Feature #1640: implement DOM XML support
- The update looks good. I will start testing it shortly.
- 05:56 AM Base Language Feature #1640: implement DOM XML support
- The merged code has been uploaded for testing. The setSubType methods unified with name syntax of the SubTypeAttribut...
- 10:32 AM User Interface Feature #2039: more runtime support for frame options
- This is just for time tracking and milestone association. All history should go into the parent task #1801.
- 10:32 AM User Interface Feature #2039 (Rejected): more runtime support for frame options
- 10:32 AM User Interface Feature #2038: conversion and runtime support for frame options, attributes and methods
- This is just for time tracking and milestone association. All history should go into the parent task #1801.
- 10:32 AM User Interface Feature #2038 (Closed): conversion and runtime support for frame options, attributes and methods
- 10:30 AM User Interface Feature #2037: runtime support for frame options
- This is just for time tracking and milestone association. All history should go into the parent task #1801.
- 10:30 AM User Interface Feature #2037 (Closed): runtime support for frame options
- 10:27 AM User Interface Feature #2036: conversion support for some frame options
- This is just for time tracking and milestone association. All history should go into the parent task #1801.
- 10:25 AM User Interface Feature #2036 (Closed): conversion support for some frame options
- 10:11 AM User Interface Feature #1801: add some frame options
- High priority frame options for milestone 4:
COLOR
FONT
IN WINDOW
STREAM-IO - 03:02 AM Base Language Feature #2002: refactor some "shared" attributes
- Committed to bzr revision 10182.
Also available in: Atom