Activity
From 05/11/2025 to 06/09/2025
06/09/2025
- 10:39 AM Feature #5630 (Closed): add missing classes referenced during parsing to the conversion list in f...
- 10:39 AM Bug #6082 (Closed): automatically add to conversion list all non-skeleton .cls dependencies
06/06/2025
- 11:24 AM Feature #5586: move all conversion artifacts/outputs into the cvtdb or into a dedicated directory...
- In those cases, we should not call @getPathToConversionFolder()@. Just bypass it when the input is @null@. In other...
- 09:49 AM Feature #5586: move all conversion artifacts/outputs into the cvtdb or into a dedicated directory...
- The functions in #5586-77, the @visitAst@ methods.
In the case of @com.goldencode.p2j.uast.SymbolResolver#loadClass... - 09:37 AM Feature #5586: move all conversion artifacts/outputs into the cvtdb or into a dedicated directory...
- > I have found that there are many places that might call getPathToConversionFolder with a null filename
This seem... - 08:59 AM Feature #5586: move all conversion artifacts/outputs into the cvtdb or into a dedicated directory...
- While testing the latest revision I have found that there are many places that might call @getPathToConversionFolder@...
- 10:17 AM Feature #6319: IntelliJ plugin
- I see that the IntelliJ LSP support isn't maintained anymore. Also, according to "jetbrains":https://plugins.jetbrain...
- 07:15 AM Bug #9863: Fileset eXclude broken on native Windows
- I managed to setup the w11 vm and reproduce the issue. Currently investigating.
- 04:54 AM Bug #9863: Fileset eXclude broken on native Windows
- Șerban Bursuc wrote:
> Dănuț Filimon wrote:
> > Roger, is there a minimal requirement for the w11 vm? I've made min... - 04:53 AM Bug #9863: Fileset eXclude broken on native Windows
- Dănuț Filimon wrote:
> Roger, is there a minimal requirement for the w11 vm? I've made mine with 4gb base memory and... - 04:45 AM Bug #9863: Fileset eXclude broken on native Windows
- Roger, is there a minimal requirement for the w11 vm? I've made mine with 4gb base memory and 2 processors and it han...
- 02:49 AM Bug #9863: Fileset eXclude broken on native Windows
- Roger Borrello wrote:
> Dănuț Filimon wrote:
> > Hi Roger, I am trying to setup a Windows11 instance to start worki...
06/05/2025
- 11:35 AM Bug #9863: Fileset eXclude broken on native Windows
- Dănuț Filimon wrote:
> Hi Roger, I am trying to setup a Windows11 instance to start working on this issue and have b... - 06:17 AM Bug #9863: Fileset eXclude broken on native Windows
- Hi Roger, I am trying to setup a Windows11 instance to start working on this issue and have been meeting with some is...
- 10:09 AM Feature #5586: move all conversion artifacts/outputs into the cvtdb or into a dedicated directory...
- > Created r15974 with the following changes:
Please add comments to these code locations to explain that @// these... - 08:26 AM Feature #5586: move all conversion artifacts/outputs into the cvtdb or into a dedicated directory...
- Stefan, I mentioned in our weekly meeting that I plan to run a conversion on a customer project. At the moment, the i...
- 08:25 AM Feature #5586: move all conversion artifacts/outputs into the cvtdb or into a dedicated directory...
- Also added r15975 which removes a redundant null check from @getPathToConversionFolder@. This check has been moved in...
- 06:46 AM Feature #5586: move all conversion artifacts/outputs into the cvtdb or into a dedicated directory...
- Created r15974 with the following changes:...
- 02:28 AM Feature #5586: move all conversion artifacts/outputs into the cvtdb or into a dedicated directory...
- Ovidiu Maxiniuc wrote:
> I think you should simply ignore these locations. These "in-mem" files are volatile resourc... - 08:45 AM Bug #10099: Tilde consumes the octals/follow-up characters, when in OE this doesn't happen
- When preprocessing the following test: (same behavior is present with @~@ directly)...
- 08:39 AM Bug #10099 (Internal Test): Tilde consumes the octals/follow-up characters, when in OE this doesn...
- 08:40 AM Bug #9917 (Review): Fileset not ignoring case
- Committed *9917a/14971*. Use Set instead of List for listImpl() and sortFiles() + a few small changes.
- 08:00 AM Bug #9917: Fileset not ignoring case
- I rebased *9917a* to latest *trunk/15967*, the branch is now at revision *15970*.
I plan to commit the change for... - 04:22 AM Bug #9917: Fileset not ignoring case
- I committed *9917a/15947*. This only contains changes for the frame and menu issue, the idea is that in @convertSourc...
06/04/2025
- 05:20 PM Feature #5586: move all conversion artifacts/outputs into the cvtdb or into a dedicated directory...
- I think you should simply ignore these locations. These "in-mem" files are volatile resources, not related to static ...
- 08:05 AM Feature #5586: move all conversion artifacts/outputs into the cvtdb or into a dedicated directory...
- Ovidiu Maxiniuc wrote:
> I have looked at +5586a / r15973+.
>
> While I agree with using the constant suffixes fo... - 02:48 PM Bug #6082: automatically add to conversion list all non-skeleton .cls dependencies
- Greg Shah wrote:
> What other related features are still not complete? The intention here is to resolve the problem... - 02:03 PM Bug #6082: automatically add to conversion list all non-skeleton .cls dependencies
- What other related features are still not complete? The intention here is to resolve the problem that explciitly or ...
- 02:06 PM Feature #5630: add missing classes referenced during parsing to the conversion list in file list ...
- For now, that approach does resolve this issue. In the future, we would also want to be able to use the annotations ...
- 11:56 AM Bug #9917: Fileset not ignoring case
- > I'd expect the right start.p to be selected here. What do you think Greg?
Yes. On a case-insensitive approach, ... - 09:34 AM Bug #9917: Fileset not ignoring case
- Dănuț Filimon wrote:
> [...]
>
> This fixes the issue, but I've just found an issue.
>
> Scenarios:
> # zfile... - 09:30 AM Bug #9917: Fileset not ignoring case
- ...
- 08:10 AM Bug #9917 (WIP): Fileset not ignoring case
- Dănuț Filimon wrote:
> I think the naming convention for frames should be changed. When handling a frame with the na... - 03:59 AM Bug #9917: Fileset not ignoring case
- I think the naming convention for frames should be changed. When handling a frame with the name @test-form@, the name...
- 03:33 AM Bug #9917: Fileset not ignoring case
- I ran a conversion for a customer application that uses case-sensitive = false and it failed.
The scenario:
* 715...
06/03/2025
- 02:30 PM Feature #5630: add missing classes referenced during parsing to the conversion list in file list ...
- Greg Shah wrote:
> We probably also should implement this even in blacklist mode. In other words: what should we do ... - 08:15 AM Feature #5630: add missing classes referenced during parsing to the conversion list in file list ...
- Does trunk rev 15852 (changes from #7255) resolve this task?
- 02:24 PM Bug #6082: automatically add to conversion list all non-skeleton .cls dependencies
- Greg Shah wrote:
> Does trunk rev 15852 (changes from #7255) resolve this task?
If the scope of this task is stri... - 08:15 AM Bug #6082: automatically add to conversion list all non-skeleton .cls dependencies
- Does trunk rev 15852 (changes from #7255) resolve this task?
- 01:36 PM Feature #5586: move all conversion artifacts/outputs into the cvtdb or into a dedicated directory...
- I have looked at +5586a / r15973+.
While I agree with using the constant suffixes for file types, I do not think w... - 08:56 AM Feature #6256: improved profile support
- Greg Shah wrote:
> Are you sure that the requirements of #6256-47 and #6256-50 are already handled? I didn't think ... - 08:41 AM Feature #6256: improved profile support
- Code Review Task Branch 6256a Revisions 15942 and 15943
The change is good.
Are you sure that the requirements ... - 07:53 AM Feature #6256 (Review): improved profile support
- Indeed, BarTest.cls is added during the @Scanning Progress Source (preprocessor, lexer, parser, persist ASTs)@ step.
... - 05:06 AM Feature #6256: improved profile support
- Constantin Asofiei wrote:
> No, @X@ directive works only with @./abl/@ paths - there can't be relative paths.
>
>... - 03:23 AM Feature #6256: improved profile support
- A breakdown of the remaining parameters:
* includeRemap: #6256-76 is really helpful, I also found that the tag is no...
06/02/2025
- 11:25 AM Feature #6256: improved profile support
- I don't remember that code. It isn't currently used in the project associated with #4069.
Still, if it was needed... - 10:53 AM Feature #6256: improved profile support
- @includeRemap@ is from:...
- 10:43 AM Feature #6256: improved profile support
- > > I updated https://proj.goldencode.com/projects/p2j/wiki/Chapter_6_Project_Setup#Profiles, but I am not sure how t...
- 09:46 AM Feature #6256: improved profile support
- Constantin Asofiei wrote:
> > * paths: ?
This is the PROPATH for the profile.
- 09:45 AM Feature #6256: improved profile support
- Dănuț Filimon wrote:
> I updated https://proj.goldencode.com/projects/p2j/wiki/Chapter_6_Project_Setup#Profiles, but... - 09:37 AM Feature #6256: improved profile support
- Dănuț Filimon wrote:
> Dănuț Filimon wrote:
> > I also want to add the following scenario:
> > * When there is a *... - 09:34 AM Feature #6256: improved profile support
- Dănuț Filimon wrote:
> I also want to add the following scenario:
> * When there is a *default* profile defined, bu... - 07:55 AM Feature #6256: improved profile support
- I updated https://proj.goldencode.com/projects/p2j/wiki/Chapter_6_Project_Setup#Profiles, but I am not sure how to de...
- 06:43 AM Feature #6256: improved profile support
- Constantin Asofiei wrote:
> If profiles are used, then:
> * if the profile has a fileset, then that is used
> * if... - 06:12 AM Feature #6256: improved profile support
- I committed *6256a/15492*. Ignore the -z fileset, but only when there's a profile fileset specified.
05/30/2025
- 09:49 AM Bug #6308: post string literal alternative coding quirk in 4GL preprocessor
- A present bug:
Preprocessing this in OE allows the file to be included:... - 04:58 AM Bug #9918: tilde doesn't duplicate semicolon in alternative codings
- That seems reasonable.
- 02:52 AM Bug #9918: tilde doesn't duplicate semicolon in alternative codings
- I have found that when preprocessing the following in OE:...
05/29/2025
- 10:15 AM Feature #6256: improved profile support
- Greg Shah wrote:
> > When a profile is used using the -p option and we also use ListType.FILESET (-z), it should beh... - 08:06 AM Feature #6256: improved profile support
- Alexandru Lungu wrote:
> > So we intend for profiles to be self-contained definitions of applications/modules. I pre... - 08:00 AM Feature #6256: improved profile support
- > So we intend for profiles to be self-contained definitions of applications/modules. I prefer not to mix random -z f...
- 07:37 AM Feature #6256: improved profile support
- > When a profile is used using the -p option and we also use ListType.FILESET (-z), it should behave like -z was not ...
- 07:05 AM Feature #6256: improved profile support
- The ProfileConfig is created when the p2j.cfg.xml is read (Configuration.loadXml()), the following properties must be...
- 06:10 AM Feature #6256: improved profile support
- Greg Shah wrote:
> Please plan to make additions to our documentation to explain this syntax.
I will do so in thi... - 06:06 AM Bug #9917: Fileset not ignoring case
- I tested and didn't find a scenario where this breaks. The customer application I'm working on uses uses the default ...
- 02:50 AM Bug #9917: Fileset not ignoring case
- I committed 9917a/15946 to address #9917-7. I will do a few more tests.
05/28/2025
- 12:53 PM Bug #9917 (Internal Test): Fileset not ignoring case
- Code Review Task Branch 9917a Revision 15945
Please only initialize @directoryCache@ when @!isCaseSensitive()@ is ... - 07:56 AM Bug #9917 (Review): Fileset not ignoring case
- I confirmed that the scenario doesn't work. About #9917-2, the compare method is not what I expect to use in this cas...
- 11:07 AM Support #9919: discuss and define the MVP for the first pass at IDE support
- Interesting results. I will have to check the licensing of LSP4J, not sure yet if it is safe to use.
I wasn't pla... - 10:27 AM Feature #6256: improved profile support
- Please plan to make additions to our documentation to explain this syntax.
- 09:03 AM Feature #6256: improved profile support
- I've experimented and the following worked:...
- 08:46 AM Feature #6256: improved profile support
- Dănuț Filimon wrote:
> Dănuț Filimon wrote:
> > I am trying to understand how p2j.cfg.xml is used in this context, ... - 08:29 AM Feature #6256: improved profile support
- Dănuț Filimon wrote:
> I am trying to understand how p2j.cfg.xml is used in this context, I've found the following d... - 05:31 AM Feature #6256: improved profile support
- I am trying to understand how p2j.cfg.xml is used in this context, I've found the following documentation https://pro...
- 04:59 AM Feature #6256: improved profile support
- No, @X@ directive works only with @./abl/@ paths - there can't be relative paths.
> During pre-scan phase, the fil... - 04:26 AM Feature #6256: improved profile support
- For #6256-53, I think the fix needs to involve the p2j.cfg.xml (the propath and basepath parameters). In @FileListFac...
- 09:48 AM Bug #9886: Double slashes should produce a new line during preprocessing
- Will start the CHUI regression tests after #5586. I have added another rev, 15911. The changes:
* flag is not initia... - 04:49 AM Bug #6308: post string literal alternative coding quirk in 4GL preprocessor
- I had to add this in @condtext@ in the Parser:...
- 04:48 AM Bug #6308: post string literal alternative coding quirk in 4GL preprocessor
- I have fixed this issue regarding @&THEN@.
It seems that when preprocessing ...
05/27/2025
- 09:47 AM Feature #6256: improved profile support
- Constantin Asofiei wrote:
> There is a problem when the 'exclude' filter is used to not add files to conversion. Du... - 07:45 AM Feature #6256: improved profile support
- Created task branch *6256a* from latest *trunk/15941*.
- 04:04 AM Feature #7180: create a centralized manager for the conversion list and per-file/project status a...
- Committed *7180a/15952*. Simplified transformArtifactNames() and removed containsArtifact() from ArtifactManager.
...
05/26/2025
- 10:12 AM Support #9919: discuss and define the MVP for the first pass at IDE support
- I have experimented a bit with "LSP4J":https://github.com/eclipse-lsp4j/lsp4j, which is a framework that offers suppo...
- 07:49 AM Feature #6256: improved profile support
- Although I added some notes in #6256-45, those are wider requirements for app-by-app conversion. We won't work those...
- 07:39 AM Feature #7180: create a centralized manager for the conversion list and per-file/project status a...
- I decided that I will hold off the changes to UastHints due to #7180-163, I shifted my focus to getting rid of the Fi...
05/23/2025
- 08:24 AM Bug #6308: post string literal alternative coding quirk in 4GL preprocessor
- Stefan Vieru wrote:
>
> [...]
> My interpretation is that OE just "eats" what is after &THEN.
> How should I a... - 06:22 AM Bug #6308: post string literal alternative coding quirk in 4GL preprocessor
- Another bug:...
- 04:18 AM Bug #6308: post string literal alternative coding quirk in 4GL preprocessor
- Same behavior described in #6859-19 for @&ENDIF@
- 03:44 AM Bug #6308: post string literal alternative coding quirk in 4GL preprocessor
- I've created tests for all types of combinations between
* &scoped-define
* &if
* &then
* &elseif
* &else
* &e... - 07:49 AM Feature #7180: create a centralized manager for the conversion list and per-file/project status a...
- One issue with using artifacts in UastHints which is problematic. The example is @AstGenerator.processFile()@ where w...
- 06:14 AM Feature #7180: create a centralized manager for the conversion list and per-file/project status a...
- I was going into the customer rules for the ChUI project and noticed a @getSourceFor()@ call which returns a filename...
- 05:53 AM Feature #7180: create a centralized manager for the conversion list and per-file/project status a...
- Rebased *7180a* to latest *trunk/15932*, the branch is now at revision *15951*.
05/22/2025
- 09:51 AM Bug #10028: FWD needs full support for unicode/octal escape sequences
- I finished preprocessor testcases for printable ASCII characters (octal representation). I continue with code page-de...
- 06:52 AM Bug #10028: FWD needs full support for unicode/octal escape sequences
- Agreed.
- 03:26 AM Bug #10028: FWD needs full support for unicode/octal escape sequences
- A thought about the case like @define variable ch as character initial "~000doesntmatter":U no-undo.@.
In FWD, the...
05/21/2025
- 11:53 AM Bug #7255: FWD resolves a OO file not included in conversion, if it exists on disk and on PROPATH
- Roger Borrello wrote:
> Constantin Asofiei wrote:
> > Your case-sensitivity is wrong - use @F ./abl/test.p@. I don... - 11:31 AM Bug #7255: FWD resolves a OO file not included in conversion, if it exists on disk and on PROPATH
- Constantin Asofiei wrote:
> Your case-sensitivity is wrong - use @F ./abl/test.p@. I don't recall exactly, but I th... - 11:28 AM Bug #7255: FWD resolves a OO file not included in conversion, if it exists on disk and on PROPATH
- Your case-sensitivity is wrong - use @F ./abl/test.p@. I don't recall exactly, but I think it was discussed in anoth...
- 11:25 AM Bug #7255: FWD resolves a OO file not included in conversion, if it exists on disk and on PROPATH
- Would the below testcase have been covered by this task?
On Ubuntu, create @./abl/test.p@ (just @message "Hello wo... - 11:40 AM Bug #9917: Fileset not ignoring case
- Below is a recreate:
On Ubuntu, create @./abl/test.p@ (just @message "Hello world!"@ and lowercase filename) and @... - 10:38 AM Feature #5586: move all conversion artifacts/outputs into the cvtdb or into a dedicated directory...
- You can post in #7823 for help on that test suite.
- 09:21 AM Feature #5586: move all conversion artifacts/outputs into the cvtdb or into a dedicated directory...
- The failing ones are :
* [...]
- 08:56 AM Feature #5586: move all conversion artifacts/outputs into the cvtdb or into a dedicated directory...
- I used the regression suite, should i upload the results here for any support? i have 6 fails
I built the tests and ... - 07:14 AM Bug #10028 (WIP): FWD needs full support for unicode/octal escape sequences
05/20/2025
- 08:53 AM Feature #7180: create a centralized manager for the conversion list and per-file/project status a...
- Dănuț Filimon wrote:
> The revision is *7180a/15937*. *Constantin*, can you take a look at the last revision and let... - 07:13 AM Feature #7180: create a centralized manager for the conversion list and per-file/project status a...
- I've done an ETF conversion with 7180a/15912 (before rebase) and it was successful, I'll start runtime tests and chec...
- 05:34 AM Feature #7180: create a centralized manager for the conversion list and per-file/project status a...
- Rebased *7180a* to latest *trunk/15918*, the branch is now at revision *15936*.
I also committed changes to src/co... - 03:38 AM Feature #7180: create a centralized manager for the conversion list and per-file/project status a...
- *9408a* was merged to trunk, I am planning a rebase because the branch is affected. See #9408-21.
- 02:16 AM Feature #7180: create a centralized manager for the conversion list and per-file/project status a...
- Committed *7180a/15912*. Missed conversion changes for the #7180-151 commit, getSchemaFile() and getSchemaFileName() ...
- 07:56 AM Feature #5586: move all conversion artifacts/outputs into the cvtdb or into a dedicated directory...
- New revno: 15902
- 07:47 AM Feature #5586: move all conversion artifacts/outputs into the cvtdb or into a dedicated directory...
- I already checked callgraph and it works.
Regarding analytics I had an issue where it would look for rootnodes insid...
05/19/2025
- 12:05 PM Bug #9886: Double slashes should produce a new line during preprocessing
- > I have rebased and tested this with a customer application. The "deploy" step passed.
Was this an application th... - 12:03 PM Feature #5586: move all conversion artifacts/outputs into the cvtdb or into a dedicated directory...
- Have you re-tested Hotel GUI (including analytics and callgraph)?
Please make sure to test conversion of the ChUI ... - 12:02 PM Bug #6308: post string literal alternative coding quirk in 4GL preprocessor
- Code Review Task Branch 6308a Revision 15903
I'm a little nervous about this change.
Please add preprocessor te... - 07:45 AM Feature #7180: create a centralized manager for the conversion list and per-file/project status a...
- I committed the #7180-151 - #7180-154 changes to *7180a/15911*.
05/16/2025
- 09:03 AM Feature #7180: create a centralized manager for the conversion list and per-file/project status a...
- Dănuț Filimon wrote:
> Greg Shah wrote:
> > > I plan to get rid of plain ".ast" or ".dict" usages and turn them int... - 08:33 AM Feature #7180: create a centralized manager for the conversion list and per-file/project status a...
- Greg Shah wrote:
> > I plan to get rid of plain ".ast" or ".dict" usages and turn them into ArtifactManager variable... - 08:13 AM Feature #7180: create a centralized manager for the conversion list and per-file/project status a...
- > I plan to get rid of plain ".ast" or ".dict" usages and turn them into ArtifactManager variables to be used in my n...
- 07:52 AM Feature #7180: create a centralized manager for the conversion list and per-file/project status a...
- Committed *7180a/15910*. Renamed methods, fixed javadoc. Also found a few issues which had to be addressed, then foun...
05/14/2025
- 07:25 AM Bug #9886: Double slashes should produce a new line during preprocessing
- I have rebased and tested this with a customer application. The "deploy" step passed.
- 07:25 AM Feature #5586: move all conversion artifacts/outputs into the cvtdb or into a dedicated directory...
- I tested this with the two customer applications. Both have passed with no issues during deploy.
- 01:46 AM Bug #6308: post string literal alternative coding quirk in 4GL preprocessor
- When preprocessing the following ...
05/13/2025
- 08:12 AM Bug #6308: post string literal alternative coding quirk in 4GL preprocessor
- > As a summary, in text.g I added this to now match @& @ and consume it since that's the behavior in 4GL.
Please s... - 06:38 AM Feature #5586 (Internal Test): move all conversion artifacts/outputs into the cvtdb or into a ded...
- Code Review Task Branch 5586a Revisions 15899 through 15901
The changes are good. - 06:33 AM Feature #7180 (Review): create a centralized manager for the conversion list and per-file/project...
- Greg, please review 7180a.
There might be an issue with the ScanDriver collections which were changes from synchr... - 05:42 AM Feature #7180: create a centralized manager for the conversion list and per-file/project status a...
- *7180a/15909* contains the changes mentioned in #7180-148, including the compatibility changes for the trunk rebase. ...
- 04:10 AM Feature #7180: create a centralized manager for the conversion list and per-file/project status a...
- Eduard Soltan wrote:
> I have been trying the test with 7180a-15894.patch, the following test case:
>
> FooTest.c...
05/12/2025
- 09:39 AM Bug #6308: post string literal alternative coding quirk in 4GL preprocessor
- I reverted this change since it wasn't producing what I expected, but in return I added:...
Also available in: Atom