FWD » Core Development » Base Language

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