Activity
From 09/23/2023 to 10/22/2023
10/20/2023
- 11:34 AM Bug #7299: resolution of the target OO method in chained calls with poly arguments
- Branch 7299b was merged to trunk rev 14779 and archived
- 10:15 AM Bug #7299: resolution of the target OO method in chained calls with poly arguments
- Code Review Task Branch 7299b Revisions 14778 through 14780
The changes are good. - 08:31 AM Feature #6649: improve the performance or SourceNameMapper runtime
- Alexandru Lungu wrote:
> For the conversion side, do you mean preprocessing the path when converting to extract the ... - 08:25 AM Feature #6649: improve the performance or SourceNameMapper runtime
- I've *committed 6649b/rev.14743* with the following changes:
* renamed @convertNameDynamically@ to @fastConvertName@... - 07:30 AM Feature #6649: improve the performance or SourceNameMapper runtime
- For the conversion side, do you mean preprocessing the path when converting to extract the base name (@a.p@ from @x/y...
- 07:28 AM Feature #6649: improve the performance or SourceNameMapper runtime
- Hynek Cihlar wrote:
> Another idea I'm not sure you already considered is precalculating the denormalized Progress n... - 06:06 AM Feature #6649: improve the performance or SourceNameMapper runtime
- Hynek Cihlar wrote:
> The precalculated map would be stored per session and would have to be recalculated on input c... - 06:04 AM Feature #6649: improve the performance or SourceNameMapper runtime
- Another idea I'm not sure you already considered is precalculating the denormalized Progress names passed in to the c...
- 05:32 AM Feature #6649: improve the performance or SourceNameMapper runtime
- Hynek Cihlar wrote:
> I'm not that familiar with the anticipated test cases, but wouldn't the cache still be benefic...
10/19/2023
- 11:38 AM Feature #6649: improve the performance or SourceNameMapper runtime
- Dănuț Filimon wrote:
> Hynek Cihlar wrote:
> It seems that currently the @convertName@ method calls @getLegacyCaseS... - 09:51 AM Feature #6649: improve the performance or SourceNameMapper runtime
- Hynek Cihlar wrote:
> *convertNameDynamically:*
> * instead of
> [...]
> it would be more efficient to store cas... - 10:05 AM Bug #7299: resolution of the target OO method in chained calls with poly arguments
- Greg, please review 7299b (revs 14778 - 14780) - this contains regression fixes for #6501-359 and #7929
- 07:15 AM Bug #7299: resolution of the target OO method in chained calls with poly arguments
- 7299b rev 14776 will show linkage errors when resolving Java class names during conversion. The problem here is that...
10/18/2023
- 05:52 PM Feature #6431: implement support to allow CPINTERNAL set to UTF-8 and CP936
- Greg Shah wrote:
> We will be running these as unit tests using our ABLUnit support. At some point, your testcases ... - 05:48 PM Feature #6431: implement support to allow CPINTERNAL set to UTF-8 and CP936
- Ovidiu Maxiniuc wrote:
> Theodoros Theodorou wrote:
> > I am trying to build a prorgam in OE that generates tests. ... - 05:27 PM Feature #6431: implement support to allow CPINTERNAL set to UTF-8 and CP936
- Theodoros Theodorou wrote:
> I am trying to build a prorgam in OE that generates tests. I generated a file located a... - 05:18 PM Feature #6431: implement support to allow CPINTERNAL set to UTF-8 and CP936
- Theodoros Theodorou wrote:
> Greg Shah wrote:
> > You could make this data driven. The idea is to capture the inpu... - 05:11 PM Feature #6431: implement support to allow CPINTERNAL set to UTF-8 and CP936
- We will be running these as unit tests using our ABLUnit support. At some point, your testcases will be reworked to ...
- 05:02 PM Feature #6431: implement support to allow CPINTERNAL set to UTF-8 and CP936
- Greg Shah wrote:
> > What do you think about my idea for testing asc() and chr() with different -cpinternals?
>
>... - 04:56 PM Feature #6431: implement support to allow CPINTERNAL set to UTF-8 and CP936
- Greg Shah wrote:
> You could make this data driven. The idea is to capture the inputs and results in an array even ... - 04:19 PM Feature #6431: implement support to allow CPINTERNAL set to UTF-8 and CP936
- > What do you think about my idea for testing asc() and chr() with different -cpinternals?
I think we must do that... - 04:17 PM Feature #6431: implement support to allow CPINTERNAL set to UTF-8 and CP936
- You could make this data driven. The idea is to capture the inputs and results in an array even in a data file. Then...
- 04:09 PM Feature #6431: implement support to allow CPINTERNAL set to UTF-8 and CP936
- I am trying to build a prorgam in OE that generates tests. I generated a file located at testcases/cp/cpinternal/iso8...
10/17/2023
- 01:21 PM Feature #6649: improve the performance or SourceNameMapper runtime
- Code review 6649b revision 14742 and 14741.
Functionally the changes look OK. I'm throwing in some random ideas fo... - 03:14 AM Feature #6649: improve the performance or SourceNameMapper runtime
- Alexandru Lungu wrote:
> There is a lot of documentation there, so from this POV I think the changes are clear enoug... - 10:20 AM Feature #7045: re-implement "normal" (non-abend) stack unwinding to avoid throwing an exception
- Committed on 7045b, revision 14682.
For now @QueryOffEnd@ exception is not throw in case a query is part of a @for...
10/16/2023
- 01:30 PM Feature #6649: improve the performance or SourceNameMapper runtime
- Hynek: Please review.
- 10:22 AM Feature #7045: re-implement "normal" (non-abend) stack unwinding to avoid throwing an exception
- Committed on 7045b, revision 14681.
Because there are cases where an error still should be throw, I reintroduced ... - 09:08 AM Feature #7045: re-implement "normal" (non-abend) stack unwinding to avoid throwing an exception
- Eduard, please make a separate issue for #7045-53 if this can be reproduced without 7045b changes. We need to focus o...
10/14/2023
- 06:32 AM Bug #7299: resolution of the target OO method in chained calls with poly arguments
- Greg Shah wrote:
> It seems pretty safe. I'm OK with you merging to trunk.
I'll delay this until I get the custome...
10/13/2023
- 03:34 PM Bug #7299: resolution of the target OO method in chained calls with poly arguments
- It seems pretty safe. I'm OK with you merging to trunk.
- 03:34 PM Bug #7299: resolution of the target OO method in chained calls with poly arguments
- Code Review Task Branch 7299b Revison 14774
The change is good. - 01:38 PM Bug #7299: resolution of the target OO method in chained calls with poly arguments
- There is a regression in the conversion of #7156 project: a LPARENS can enclose an argument. I've created 7299b from...
- 11:18 AM Feature #6649: improve the performance or SourceNameMapper runtime
- I skimmed the changes in 6649b and they are quite robust. Well done!
There is a lot of documentation there, so from ... - 10:11 AM Feature #7045: re-implement "normal" (non-abend) stack unwinding to avoid throwing an exception
- During some investigation I came across @QueryConstants.RETRIEVE_MODES@ used in @CompoundQuery@.
I tried to call @...
10/11/2023
- 07:54 AM Bug #7299 (Test): resolution of the target OO method in chained calls with poly arguments
- 03:19 AM Bug #7299: resolution of the target OO method in chained calls with poly arguments
- 7299a was merged to trunk rev 14770 and archived.
10/10/2023
- 03:17 PM Bug #7299: resolution of the target OO method in chained calls with poly arguments
- Code Review Task Branch 7299a Revisions 14777 through 14784
No objections.
You can merge to trunk. - 08:33 AM Feature #3855: implement equivalent support for REST (classic appserver, PASOE REST and WEB trans...
- We have customer test environments that are 11.x so we can still access classic appserver for testing. Would that he...
- 02:28 AM Feature #3855: implement equivalent support for REST (classic appserver, PASOE REST and WEB trans...
- Greg Shah wrote:
> What parts of #3855-102 remain to be completed?
>
> Where are we with the testcases?
I thin... - 08:31 AM Feature #4658: OO serialization support
- > the object has no control of how it's data is being serialised, other than defining what is/isn't serialisable
P... - 02:13 AM Feature #4658: OO serialization support
- Greg Shah wrote:
> Marian: Do you know of any such feature (like Java's @Externalizable@) in the 4GL?
The keyword... - 08:24 AM Feature #3310 (Closed): add support for the equivalent of the OpenEdge Web Services Adapter (and ...
- 02:12 AM Feature #3310: add support for the equivalent of the OpenEdge Web Services Adapter (and ProxyGen)...
- Greg Shah wrote:
> Is there anything left to do on this task?
I don't think so. - 03:49 AM Feature #7045: re-implement "normal" (non-abend) stack unwinding to avoid throwing an exception
- Eduard, 7045b is in a "protoype" stage. There may be multiple such issues with it. In fact, the (first) implementatio...
- 03:41 AM Feature #7045: re-implement "normal" (non-abend) stack unwinding to avoid throwing an exception
- There is a problem with current implementation on 7045b....
10/09/2023
- 03:54 PM Feature #4658: OO serialization support
- > > > And here it shows that OE can have custom serializers: https://programinprogress.com/object-serialization-in-p...
- 02:46 PM Feature #4658: OO serialization support
- Greg Shah wrote:
> > And here it shows that OE can have custom serializers: https://programinprogress.com/object-se... - 02:21 PM Feature #4658: OO serialization support
- > And here it shows that OE can have custom serializers: https://programinprogress.com/object-serialization-in-progr...
- 03:51 PM Feature #3310: add support for the equivalent of the OpenEdge Web Services Adapter (and ProxyGen)...
- Is there anything left to do on this task?
- 02:43 PM Feature #3310: add support for the equivalent of the OpenEdge Web Services Adapter (and ProxyGen)...
- Greg Shah wrote:
> Is the "qualified prefixing" feature from #3310-93 the only remaining feature? I'm trying to und... - 02:38 PM Feature #3310: add support for the equivalent of the OpenEdge Web Services Adapter (and ProxyGen)...
- Is the "qualified prefixing" feature from #3310-93 the only remaining feature? I'm trying to understand what is left...
- 03:49 PM Feature #3854 (Closed): update appserver support with PASOE features
- 02:36 PM Feature #3854: update appserver support with PASOE features
- Greg Shah wrote:
> Is there any reason to leave this task open? As far as I understand it, all the remaining work i... - 02:35 PM Feature #3854: update appserver support with PASOE features
- Is there any reason to leave this task open? As far as I understand it, all the remaining work is in #6417.
- 02:42 PM Feature #3855: implement equivalent support for REST (classic appserver, PASOE REST and WEB trans...
- What parts of #3855-102 remain to be completed?
Where are we with the testcases? - 09:22 AM Bug #7299 (Review): resolution of the target OO method in chained calls with poly arguments
- Greg, please review 7299a starting from rev 14777 to 14784 (i.e. compared with 14776, the last reviewed revision).
...
10/07/2023
- 01:57 PM Bug #7299: resolution of the target OO method in chained calls with poly arguments
- 7299a was rebased and rev 14777/14778 contains fixes to add @ObjectOps.poly(Object)Arg@ wrapper for cases when a dire...
10/06/2023
- 12:17 PM Feature #6431: implement support to allow CPINTERNAL set to UTF-8 and CP936
- > As far as I understand, using @-cprcodein UTF-8@ tells OE that the source file should be read as UTF-8.
No, I do... - 12:10 PM Feature #6431: implement support to allow CPINTERNAL set to UTF-8 and CP936
- Greg Shah wrote:
> Are all your notes from #6431-13 based on OE testing? I don't want to mix any FWD results togeth... - 12:03 PM Feature #6431: implement support to allow CPINTERNAL set to UTF-8 and CP936
- Ovidiu Maxiniuc wrote:
> Indeed, this is tricky. I think this is the first time I see @-cprcodein@ OE parameter. If ... - 09:05 AM Feature #6431: implement support to allow CPINTERNAL set to UTF-8 and CP936
- Are all your notes from #6431-13 based on OE testing? I don't want to mix any FWD results together at this point, it...
- 10:37 AM Feature #7045: re-implement "normal" (non-abend) stack unwinding to avoid throwing an exception
- Eduard, please focus on this task for the upcoming Sprint.
There is a lot of information here, so feel free to ask...
10/05/2023
- 06:44 PM Feature #6431: implement support to allow CPINTERNAL set to UTF-8 and CP936
- Indeed, this is tricky. I think this is the first time I see @-cprcodein@ OE parameter. If I understand this paramete...
- 06:27 PM Feature #6431: implement support to allow CPINTERNAL set to UTF-8 and CP936
- I need help for understanding what needs to be done. Codepages look very tricky. Progress OE returns unexpected resul...
- 06:26 AM Bug #7873 (Test): conversion issues with dataset/table parameters for class events
- 06:18 AM Bug #7873: conversion issues with dataset/table parameters for class events
7873a was merged to trunk rev 14767 and archived.- 06:11 AM Bug #7873: conversion issues with dataset/table parameters for class events
- Go ahead.
- 01:15 AM Bug #7873: conversion issues with dataset/table parameters for class events
- Greg Shah wrote:
> What testing do you plan?
I've left overnight conversion for some projects and the result is OK....
10/04/2023
- 05:50 PM Feature #7875: Interactive child process launching is broken in GUI
- A good amount of background can be found in #7676. Make sure to go through it first.
This required a simple testc... - 03:55 PM Feature #7875 (New): Interactive child process launching is broken in GUI
- 04:29 PM Bug #7873: conversion issues with dataset/table parameters for class events
- Code Review Task Branch 7873a Revision 14767
I'm fine with the changes.
What testing do you plan? - 01:45 PM Bug #7873 (Review): conversion issues with dataset/table parameters for class events
- I've added support for DATASET, TABLE, DATASET-HANDLE, TABLE-HANDLE and BUFFER parameters for class events in 7873a r...
- 10:19 AM Bug #7873 (WIP): conversion issues with dataset/table parameters for class events
- Created task branch 7873a from trunk rev 14766.
- 10:19 AM Bug #7873: conversion issues with dataset/table parameters for class events
- FWD has problems converting class events with TABLE/DATASET parameters. This issue is meant to explore and fix all p...
- 10:18 AM Bug #7873 (Test): conversion issues with dataset/table parameters for class events
- 05:44 AM Feature #6410: implement additional built-in OO classes/interfaces
- Marian, I think we need 'invokeStandalone' to be emitted when the returned value is not read and 'invoke' in all othe...
10/03/2023
- 08:16 AM Feature #6410: implement additional built-in OO classes/interfaces
- Working on remaining issues on method @invoke@ and beside the known issue with @return extent@ I have a situation tha...
10/02/2023
- 10:44 AM Feature #6649: improve the performance or SourceNameMapper runtime
- *Committed 6649b/rev.14742*.
* Added r-code file searching;
* Modified @PathLookup@ into an interface;
* Short-cir... - 08:17 AM Feature #6649: improve the performance or SourceNameMapper runtime
- By removing unnecessary files from the lookup (files with an extension that don't match any legacy program name exten...
09/29/2023
- 05:07 PM Bug #7299: resolution of the target OO method in chained calls with poly arguments
- Code Review Task Branch 7299a Revisions 14756 through 14763
Overall, it looks good. The refactoring made the revi... - 12:33 PM Bug #5538 (Closed): fix readByte() for DirStream, ProcessStream, and others
- 12:33 PM Bug #5538: fix readByte() for DirStream, ProcessStream, and others
- Branch 5538a was merged to trunk rev. 14758 and archived.
- 11:16 AM Bug #5538: fix readByte() for DirStream, ProcessStream, and others
- You can merge to trunk now.
Should we set %Done to 100? - 11:15 AM Bug #5538: fix readByte() for DirStream, ProcessStream, and others
- Code Review Task Branch 5538a Revision 14758
No objections. - 09:43 AM Feature #6410: implement additional built-in OO classes/interfaces
- > what needs to be done beside finishing the method/invoke support.
Please do implement all the remaining items a... - 06:15 AM Feature #6410: implement additional built-in OO classes/interfaces
- Constantin Asofiei wrote:
> Created task branch 6410b from trunk rev 14756.
>
> The branch is available on xfer.
... - 04:59 AM Feature #6410: implement additional built-in OO classes/interfaces
- Created task branch 6410b from trunk rev 14756.
The branch is available on xfer. - 04:04 AM Feature #6410: implement additional built-in OO classes/interfaces
- Greg Shah wrote:
> In addition to the remaining items in @getMethods()@, what work is left from #6410-1?
As far a...
09/28/2023
- 03:28 PM Bug #5538: fix readByte() for DirStream, ProcessStream, and others
- Ovidiu Maxiniuc wrote:
> There are minor changes (only some comments) since my last review. One thing which I failed... - 09:28 AM Bug #5538: fix readByte() for DirStream, ProcessStream, and others
- Code Review Task Branch 5538a Revision 14756
It seems like we could rewrite the @@ as follows:... - 12:24 PM Bug #7299: resolution of the target OO method in chained calls with poly arguments
- Constantin Asofiei wrote:
> ... At least, 'exact method matching' must be done in the entire hierarchy, before goin... - 11:57 AM Bug #7299: resolution of the target OO method in chained calls with poly arguments
- The current changes are in 7299a rev 14758. I'll reconvert some projects and check real scenarios.
This adds @Obj... - 11:27 AM Bug #7299: resolution of the target OO method in chained calls with poly arguments
- Constantin Asofiei wrote:
> My assumption is that @h::f3@ doesn't set the type of the object returned (which would b... - 11:25 AM Bug #7299: resolution of the target OO method in chained calls with poly arguments
- There is something which looks like a bug in the OpenEdge method resolution. Assume there is a method like this:
<p... - 07:06 AM Feature #6649: improve the performance or SourceNameMapper runtime
- If the more generic approach of #6649-17 is still too costly, then we can certainly put some logic in to check for th...
- 06:55 AM Feature #6649: improve the performance or SourceNameMapper runtime
- But wouldn't it be faster to rule out such searches based on the file extension? Since the original lookup is done on...
- 06:27 AM Feature #6649: improve the performance or SourceNameMapper runtime
- This is exactly the kind of thing that the idea in #6649-17 would optimize. There will be no @whatever.bmp@ in the m...
- 06:21 AM Feature #6649: improve the performance or SourceNameMapper runtime
- Greg Shah wrote:
> These are all coming from @RUN@ statements?
No, I've looked up where a .bmp file is used and it ... - 06:16 AM Feature #6649: improve the performance or SourceNameMapper runtime
- Lookup on the PROPATH can be done via the SEARCH function and IIRC even by runtime (i.e. when resolving images or ini...
- 06:13 AM Feature #6649: improve the performance or SourceNameMapper runtime
- These are all coming from @RUN@ statements?
- 06:12 AM Feature #6649: improve the performance or SourceNameMapper runtime
- Alexandru Lungu wrote:
> Danut, can you share some of these extensions and the context in which they are required? I... - 05:59 AM Feature #6649: improve the performance or SourceNameMapper runtime
- Danut, can you share some of these extensions and the context in which they are required? I mean, what kind of files ...
- 04:06 AM Feature #6649: improve the performance or SourceNameMapper runtime
- I've noticed something for a while, but I am only bringing this up now because it might be important. While doing the...
09/27/2023
- 07:58 PM Bug #5538 (Test): fix readByte() for DirStream, ProcessStream, and others
- Review branch 5538a / 14756.
There are minor changes (only some comments) since my last review. One thing which I ... - 08:54 AM Bug #5538: fix readByte() for DirStream, ProcessStream, and others
- Ovidiu: Please review.
- 08:52 AM Bug #5538: fix readByte() for DirStream, ProcessStream, and others
- Please review branch 5538a rev no 14756. It has been rebased to trunk.
- 07:42 AM Feature #7842: Make FWD behave the same as OE regarding input through
- Please recall that we never use the Description field of the Redmine header. The task details always go into the fir...
09/26/2023
- 05:52 PM Feature #7842 (New): Make FWD behave the same as OE regarding input through
- 11:12 AM Bug #5538: fix readByte() for DirStream, ProcessStream, and others
- Theodoros Theodorou wrote:
> Ok. Should we close this ticket? Should we create another ticket for implementing the s... - 11:06 AM Bug #5538: fix readByte() for DirStream, ProcessStream, and others
- Ok. Should we close this ticket? Should we create another ticket for implementing the same behavior with OE? Is the i...
- 09:28 AM Feature #6410: implement additional built-in OO classes/interfaces
- In addition to the remaining items in @getMethods()@, what work is left from #6410-1?
- 05:46 AM Feature #6410: implement additional built-in OO classes/interfaces
- Constantin Asofiei wrote:
> Branch 6410a was merged to trunk rev 14751 and archived.
>
> Marian: please let us kn... - 09:24 AM Feature #6649: improve the performance or SourceNameMapper runtime
- I've committed the implementation to -*6649a/rev.14741*- *6649b/rev.14741*. Please review and advise.
* propath can ... - 08:38 AM Feature #6649: improve the performance or SourceNameMapper runtime
- > _The PROPATH may be found is in the file name (from name-map.xml)_, not in the search spec.
Yes, but a propath ... - 08:31 AM Feature #6649: improve the performance or SourceNameMapper runtime
- I retested #6649-30 using the new implementation and I got a total of *38119* calls to the new @SourceNameMapper.conv...
- 05:36 AM Feature #6649: improve the performance or SourceNameMapper runtime
- I think we can agree that the extension topic is fully understood now.
> Can you help me understand what you mean ...
09/25/2023
- 05:32 PM Bug #5538: fix readByte() for DirStream, ProcessStream, and others
- Although the natural instinct is to create good code, which behaves as one would expect (in this case to fully popula...
- 05:23 PM Bug #5538 (Review): fix readByte() for DirStream, ProcessStream, and others
- 01:55 PM Bug #5538: fix readByte() for DirStream, ProcessStream, and others
- Thank you for confirming the bug Constantin and Ovidiu. I have created a test under @testcases\oo\progress\io\input_s...
- 04:32 PM Feature #6649: improve the performance or SourceNameMapper runtime
- > 4GL supports the following file types: @.p@, @.r@, @.w@, @.cls@, @.i@. I've also found @.pf@ (parameter file), @.cf...
- 04:28 PM Feature #6649: improve the performance or SourceNameMapper runtime
- During conversion, we have a copy of the file-system of the 4GL project. That project might have some arbitrary path...
- 09:51 AM Feature #6649: improve the performance or SourceNameMapper runtime
- Alexandru Lungu wrote:
> It is normal not to be found right? I mean, if you are running start.p and the only start.p... - 06:25 AM Feature #6649: improve the performance or SourceNameMapper runtime
- Dănuț Filimon wrote:
> I've also found a problem with this idea where the possible paths from a @SinglePathLookup@ c... - 06:16 AM Feature #6649: improve the performance or SourceNameMapper runtime
- > The one place I differ is that I was not thinking we would store the propath variants in the *PathLookup helpers. I...
Also available in: Atom