Project

General

Profile

Activity

From 01/31/2021 to 03/01/2021

03/01/2021

09:59 AM Feature #5160: About cast() function support
Please post a standalone example so that we can recreate the problem here. Greg Shah
09:46 AM Feature #5160: About cast() function support
I just pasted "*" due to the data safety of current project source code names. This error occurs in filenames without... Deividas Varzgalys
05:28 AM Feature #4965: Parameter annotation - qualified
Please post a small test which shows this problem. The reference is decremented somewhere where it should not be. Constantin Asofiei
05:17 AM Feature #4965: Parameter annotation - qualified
Constantin Asofiei wrote:
> Each object var definition should be declared outside the top-level block (in the Java m... Marian Edu
03:57 AM Feature #4965: Parameter annotation - qualified
Marian Edu wrote:
> This is part of the 'OO SDK' and as per your suggestion everything under @com.goldencode.p2j.oo@... Constantin Asofiei
03:48 AM Feature #4965: Parameter annotation - qualified
Constantin Asofiei wrote:
> Also, I see lots of classes have @null@ as the referent in i.e. its @BlockManager.functi... Marian Edu

02/26/2021

07:51 PM Feature #4378: properly handle clob/lonchar assignment, especially the implicit codepage conversion
Actually the dereference operation pose other issues. The COPY-LOB is also not working correctly when a dereferenced ... Ovidiu Maxiniuc

02/25/2021

04:32 AM Feature #4965 (Test): Parameter annotation - qualified
Greg Shah

02/24/2021

11:19 AM Feature #5160: About cast() function support
> ./abl/*.p:741:37: unexpected token: GetData
> ...
> 2.Error processing .*.p --- Caused by: java.lang.NullPointerE... Greg Shah
11:08 AM Feature #5160: About cast() function support
> unexpected token: strict-entity-resolution
Correct, in your version of FWD we did not recognise this attribute a... Greg Shah
08:43 AM Feature #4965: Parameter annotation - qualified
Marian Edu wrote:
> How should we proceed were, a new merge or just wait for the new 4384 branch and fix whatever ne... Constantin Asofiei
07:55 AM Feature #4965: Parameter annotation - qualified
Constantin Asofiei wrote:
> Marian, there are some compile errors -
> [...]
>
> Is this a typo? Shouldn't each ... Marian Edu
08:40 AM Bug #4749 (Test): Conversion error for object indeterminate extent output parameter
Code Review Task Branch 3821c Revision 12052
Overall, the changes are good.
Please add javadoc to @ArrayAssigne... Greg Shah

02/23/2021

12:50 PM Feature #4384: Builtin OO Implementation
Branch 4384i was created from branch 3821c revision 12052.
Marian: please review revision 12051, it fixes some com... Constantin Asofiei
12:45 PM Bug #4749: Conversion error for object indeterminate extent output parameter
Constantin Asofiei wrote:
> As for the dynamic array issues - the root cause was the fact that the Java reference fo... Constantin Asofiei
12:21 PM Feature #4965: Parameter annotation - qualified
> Greg: I think we need a 4384i branch?
Yes. Please make one as soon as you are happy with the 3821c starting point. Greg Shah
12:18 PM Feature #4965: Parameter annotation - qualified
Marian, there are some compile errors - ... Constantin Asofiei
12:12 PM Feature #4965: Parameter annotation - qualified
Constantin Asofiei wrote:
> Marian, 4384h on xfer is still on rev 12004 - although I think is a little old, is still... Marian Edu
12:04 PM Feature #4965: Parameter annotation - qualified
Marian, 4384h on xfer is still on rev 12004 - although I think is a little old, is still good, because I could use it... Constantin Asofiei
02:46 AM Feature #4965: Parameter annotation - qualified
Greg Shah wrote:
> > I've pushed 3821c revision 12046 to xfer. Do you want to branch from that, apply your changes ... Marian Edu
10:26 AM Feature #5160: About cast() function support
Yes, GeneralInfoPart is my class.... Deividas Varzgalys
08:44 AM Feature #5160: About cast() function support
> Does your tool support cast() function in that way?
Yes, FWD does support chaining a method call off of a @cast(... Greg Shah
08:40 AM Feature #5160: About cast() function support
From Deividas:
> I have issue in conversion process that error show up with cast() function
> ... Greg Shah
05:05 AM Feature #5160 (New): About cast() function support
Deividas Varzgalys

02/22/2021

07:20 AM Feature #4965: Parameter annotation - qualified
> I've pushed 3821c revision 12046 to xfer. Do you want to branch from that, apply your changes and provide a bzr se... Greg Shah
07:18 AM Feature #4965: Parameter annotation - qualified
Greg Shah wrote:
> > since there seems to be some automatic changes applied to 'oo' packages should we make a new b... Marian Edu
06:55 AM Feature #4965: Parameter annotation - qualified
> since there seems to be some automatic changes applied to 'oo' packages should we make a new branch to incorporate... Greg Shah
04:09 AM Feature #4965: Parameter annotation - qualified
Marian Edu wrote:
> Constantin, just to clarify, the default to P.L.O is not for the (newly added) return type but f... Constantin Asofiei
03:08 AM Feature #4965: Parameter annotation - qualified
Constantin Asofiei wrote:
> I agree that the return type should be added. But the approach to 'default to PLO' if t... Marian Edu
08:14 PM Feature #4965: Parameter annotation - qualified
Perhaps the default value for @LegacySignature.returns()@ should be "VOID".
As for the poly cases, I think they sh... Greg Shah
03:24 AM Bug #5029: OO constructor with array input parameter
Constantin Asofiei wrote:
> Marian, regarding #5029-15 - I've tried duplicating using this test:
> [...]
>
> But... Marian Edu

02/21/2021

11:28 AM Feature #5146: record additional legacy class and method data in annotations
3821c rev 12042 to 12044 fixes the 'extent', 'qualified' and adds 'returns' annotation to all legacy p2j.oo methods. ... Constantin Asofiei
11:27 AM Feature #4965 (WIP): Parameter annotation - qualified
All changes are in 3821c revs 12042 to 12044.
I did not touch the poly methods/parameters:... Constantin Asofiei
09:27 AM Feature #4965: Parameter annotation - qualified
I've built a tool which automatically edits the source code for the @com.goldencode.p2j.oo@ classes and injects @qual... Constantin Asofiei

02/19/2021

04:15 PM Bug #4978: Static method call in base class
Lets assume you have a protected instance method @Base.foo@ and a class @Extension@ which extends @Base@.
# in a met... Constantin Asofiei
03:57 PM Bug #4978: Static method call in base class
We need to be careful using qualified/unqualified. Most of the time when we say qualified, we mean the "package" por... Greg Shah
01:28 PM Bug #4978 (WIP): Static method call in base class
The same behavior where the static member from a super-class can be accessed from a sub-class is allowed for variable... Constantin Asofiei
03:11 PM Feature #4965: Parameter annotation - qualified
Greg Shah wrote:
> > I already have code to calculate the return type using reflection and the existing annotations.... Constantin Asofiei
11:38 AM Feature #4965: Parameter annotation - qualified
> I already have code to calculate the return type using reflection and the existing annotations.
3821c rev 12032 ... Greg Shah
11:09 AM Feature #4965: Parameter annotation - qualified
I already have code to calculate the return type using reflection and the existing annotations. I will check it in s... Greg Shah
11:05 AM Feature #4965: Parameter annotation - qualified
It was easy to write a simple tool to check the @p2j.oo@ methods which have @LegacySignature@. The list of problems ... Constantin Asofiei
09:59 AM Feature #4965: Parameter annotation - qualified
> You mentioned at some point that the method's return type is not in LegacySignature - where was this logged?
See... Greg Shah
09:47 AM Feature #4965: Parameter annotation - qualified
Greg, you mentioned at some point that the method's return type is not in LegacySignature - where was this logged?
... Constantin Asofiei
01:55 PM Bug #4749: Conversion error for object indeterminate extent output parameter
The conversion issue for @ObjectExtentGlobal@ is already fixed.
As for the dynamic array issues - the root cause w... Constantin Asofiei
11:14 AM Bug #5029: OO constructor with array input parameter
Marian, regarding #5029-15 - I've tried duplicating using this test:... Constantin Asofiei
10:55 AM Feature #5146: record additional legacy class and method data in annotations
Regarding the return type for function and OO methods - my only problem is that there are ~3000 hand-written classes ... Constantin Asofiei

02/18/2021

12:17 PM Bug #5035: Temp-Table index on P.L.O field
Greg Shah wrote:
> > is there a way to resolve an instance from its ID, an equivalent for handle function?
>
> Ma... Marian Edu
10:24 AM Bug #5035: Temp-Table index on P.L.O field
> is there a way to resolve an instance from its ID, an equivalent for handle function?
Marian: We need your thoug... Greg Shah
10:22 AM Bug #4749 (WIP): Conversion error for object indeterminate extent output parameter
Greg Shah

02/16/2021

04:42 PM Feature #4384: Builtin OO Implementation
I've added wiki documentation that lists our currently "supported built-in OO classes":/projects/p2j/wiki/Object_Orie... Greg Shah
01:58 PM Bug #5029 (Test): OO constructor with array input parameter
Greg Shah
01:12 PM Bug #5029: OO constructor with array input parameter
Constantin Asofiei wrote:
> OK, that's because my tests always had the INPUT modifier. I'll fix this.
>
> For no... Constantin Asofiei

02/15/2021

08:20 AM Feature #4384: Builtin OO Implementation
Please use this patch. It will be in 3821c in the next couple of days.... Constantin Asofiei
08:04 AM Feature #4384: Builtin OO Implementation
Marian, the root cause is because the reference is incremented when the INPUT parameter is initialized in the propert... Constantin Asofiei
02:37 AM Feature #4384: Builtin OO Implementation
Constantin Asofiei wrote:
> As long as you use @TypeFactory.object@ or @UndoableFactory.object@ to initialize a prop... Marian Edu

02/14/2021

10:51 AM Feature #4384: Builtin OO Implementation
As long as you use @TypeFactory.object@ or @UndoableFactory.object@ to initialize a property/var, then reference coun... Constantin Asofiei
10:47 AM Feature #4384: Builtin OO Implementation
Constantin Asofiei wrote:
> Marian, I had to revert your @object.setUnknown@ change which decremented the reference ... Marian Edu

02/13/2021

05:51 PM Feature #4384: Builtin OO Implementation
Marian, I had to revert your @object.setUnknown@ change which decremented the reference - only @ObjectVar@ is allowed... Constantin Asofiei

02/12/2021

10:41 AM Feature #5146: record additional legacy class and method data in annotations
* We should be able to record (in an annotation) whether the converted class was a @CLASS@, @INTERFACE@ or @ENUM@ in ... Greg Shah
10:40 AM Feature #5146 (New): record additional legacy class and method data in annotations
Greg Shah

02/11/2021

08:32 AM Bug #5029: OO constructor with array input parameter
My current work touches how 4GL moves the data from the arguments to the parameters, via the stack, for the integer/i... Constantin Asofiei
08:24 AM Bug #5029: OO constructor with array input parameter
There is another thing with indeterminate extent output parameters, in 4GL it doesn't matter if the extent was alread... Marian Edu
08:20 AM Bug #5029: OO constructor with array input parameter
OK, that's because my tests always had the INPUT modifier. I'll fix this.
For now, change it like @INPUT cInExt@ ... Constantin Asofiei
08:14 AM Bug #5029: OO constructor with array input parameter
Constantin Asofiei wrote:
> What are you using exactly as argument? A OO property? A class variable? Something else?... Marian Edu
08:11 AM Bug #5029: OO constructor with array input parameter
What are you using exactly as argument? A OO property? A class variable? Something else? Constantin Asofiei
08:07 AM Bug #5029: OO constructor with array input parameter
Constantin, conversion is not using @InputOutputExtentParameter@ anymore but it's missing the cast to Object so it st... Marian Edu

02/10/2021

09:07 AM Feature #4384: Builtin OO Implementation
Marian, @LegacyClass@ is missing legacy constructor method(s). Please add them.
If you debug into the @newInstanc... Constantin Asofiei
08:51 AM Feature #4384: Builtin OO Implementation
Constantin, another thing that we've found is when using any instance level methods in @LegacyClass@ we get a nice NP... Marian Edu

02/08/2021

09:24 AM Feature #4384: Builtin OO Implementation
Constantin Asofiei wrote:
> Marian, the root cause is @REGISTRY.set(registry);@ instead of @REGISTRY.get().assign(r... Marian Edu
09:14 AM Feature #4384: Builtin OO Implementation
Marian, the root cause is @REGISTRY.set(registry);@ instead of @REGISTRY.get().assign(registry);@ in @HttpHeaderBuil... Constantin Asofiei
08:52 AM Feature #4384: Builtin OO Implementation
Constantin Asofiei wrote:
> What test do you use? Can you isolate this in something simple?
This is the simplest o... Marian Edu
07:21 AM Feature #4384: Builtin OO Implementation
Marian Edu wrote:
> Right, but for an object this means the reference is the same... two objects pointing to the sam... Constantin Asofiei
07:19 AM Feature #4384: Builtin OO Implementation
Constantin Asofiei wrote:
> Marian, please give me an example where you need equals/hashCode in the collection imple... Marian Edu
06:29 AM Feature #4384: Builtin OO Implementation
Marian, please give me an example where you need equals/hashCode in the collection implementations.
As a rule of t... Constantin Asofiei
02:47 AM Feature #4384: Builtin OO Implementation
Greg Shah wrote:
> Branch 4384h was created from branch 3821c revision 11992. Please note that some changes from 43... Marian Edu

02/05/2021

08:23 AM Bug #4824 (Closed): Conversion error for property with indeterminate extent
Greg Shah
07:28 AM Bug #4824: Conversion error for property with indeterminate extent
This seems to work on 4384h, at least for @oo/Discovery/OpenEdge/Core/IObjectArrayHolder.cls@. Marian Edu
08:22 AM Bug #4827 (Closed): OO Conversion issue with method resolve (erasure)
Greg Shah
07:23 AM Bug #4827: OO Conversion issue with method resolve (erasure)
Greg, just tried that on 4384h and the conversion on those erasure quirks tests seems to work now, if I find anything... Marian Edu
06:01 AM Feature #4384: Builtin OO Implementation
Branch 4384h was created from branch 3821c revision 11992. Please note that some changes from 4384g were modified to... Greg Shah

02/04/2021

05:27 AM Bug #5029: OO constructor with array input parameter
Constantin Asofiei wrote:
> Marian, many issues related to extent parameters and others are fixed in 3821c 11984. T... Marian Edu
05:16 AM Bug #5029: OO constructor with array input parameter
Marian, many issues related to extent parameters and others are fixed in 3821c 11984. This will be in the 4384h.
... Constantin Asofiei

02/03/2021

04:52 PM Feature #4384: Builtin OO Implementation
I have merged the 4384g changes as 3821c revision 11982.
I will create task branch 4384h from 3821c tomorrow, when... Greg Shah

02/02/2021

05:59 PM Feature #4378: properly handle clob/lonchar assignment, especially the implicit codepage conversion
I guess so. I have not yet investigated how the @FIX-CODEPAGE@ function works, just spotted the compile-time error an... Ovidiu Maxiniuc
04:58 PM Feature #4378: properly handle clob/lonchar assignment, especially the implicit codepage conversion
Can I assume that @FIX-CODEPAGE(hHandle::longCharFiled) = 'utf-8' NO-ERROR.@ will alter the codepage of @longCharFile... Constantin Asofiei
04:47 PM Feature #4378: properly handle clob/lonchar assignment, especially the implicit codepage conversion
I encountered a conversion issue which I think is really related to this tracker.
If we have a @fixedLongchar@ var... Ovidiu Maxiniuc
11:42 AM Bug #5123: Remove sessionFree AppServer configuration parameter
FWD has undocumented @sessionFree@ AppServer configuration parameter. The only thing it does, it defines if session-f... Stanislav Lomany
11:40 AM Bug #5123 (New): Remove sessionFree AppServer configuration parameter
Stanislav Lomany

02/01/2021

08:26 AM Feature #4384: Builtin OO Implementation
Greg Shah wrote:
> To be clear: if we merge the recent changes from 3821c into 4384g, then the revision history of 4... Marian Edu
08:23 AM Feature #4384: Builtin OO Implementation
To be clear: if we merge the recent changes from 3821c into 4384g, then the revision history of 4384g changes would b... Greg Shah
08:20 AM Feature #4384: Builtin OO Implementation
This is the 3821c changes being put on top of 4384g. I'm looking to see the opposite patch. The idea is to start wi... Greg Shah
03:13 AM Feature #4384: Builtin OO Implementation
Greg Shah wrote:
> Use @bzr send@ to package up the revision(s) in the new local branch of 3821c. Post the output h... Marian Edu

01/31/2021

03:08 PM Feature #4384: Builtin OO Implementation
Use @bzr send@ to package up the revision(s) in the new local branch of 3821c. Post the output here and I will apply... Greg Shah
11:26 AM Feature #4384: Builtin OO Implementation
Greg Shah wrote:
> I've pushed 3821c revision 11969 to xfer. We have merged 4397a (TT serialization and dataset fix... Marian Edu
 

Also available in: Atom