Bug #10185
ADE conversion and runtime
30%
History
#2 Updated by Teodor Gorghe about 1 year ago
When passing a value from extent to a procedure, with OUTPUT TEMP-HANDLE, on compilation we have this issue:
[javac] /home/tg/gcd/testcases2/testcases/src/com/goldencode/testcases/tests/table_/parameter/output/bind/TestHandleTable.java:201: error: no suitable constructor found for TableParameter(handle[])
[javac] silent(() -> RUN_CALL_SITE_6.clone().setInHandle(hproc).run(subscript(new TableParameter(hext), 1)));
[javac] ^
[javac] constructor TableParameter.TableParameter(Temporary) is not applicable
[javac] (argument mismatch; handle[] cannot be converted to Temporary)
[javac] constructor TableParameter.TableParameter(handle) is not applicable
[javac] (argument mismatch; handle[] cannot be converted to handle)
[javac] constructor TableParameter.TableParameter(TableWrapper) is not applicable
[javac] (argument mismatch; handle[] cannot be converted to TableWrapper)
4GL code:
@Test.
method public void SameStructureExtent():
define variable hext as handle no-undo extent 1.
hext[1] = temp-table ttTest:handle.
run noopOutTable in hproc ( output table-handle hext[1] ) no-error.
support.test.AssertExt:NotErrorNotWarning().
end method.
On ABL, this test compiles and passes.
#3 Updated by Teodor Gorghe about 1 year ago
- Status changed from New to WIP
Created task branch 10185a.
#4 Updated by Teodor Gorghe about 1 year ago
- % Done changed from 0 to 100
Committed revision 15992 on task branch 10185a:
- Fixed subscript wrapper on new TableParameter()
#5 Updated by Teodor Gorghe about 1 year ago
- Status changed from WIP to Review
#6 Updated by Constantin Asofiei about 1 year ago
- reviewer Constantin Asofiei added
#7 Updated by Constantin Asofiei about 1 year ago
DATASET-HANDLEargument type- the handle array is an instance or static var in a .cls (not a local var)
- the handle array is an instance or static property in a .cls
#8 Updated by Teodor Gorghe about 1 year ago
- reviewer deleted (
Constantin Asofiei)
I don't notice any problem. The application compiles and the tests seems to work.
#9 Updated by Teodor Gorghe about 1 year ago
- reviewer Constantin Asofiei added
#10 Updated by Teodor Gorghe about 1 year ago
- Status changed from Review to WIP
- % Done changed from 100 to 30
- Subject changed from OUTPUT TABLE-HANDLE conversion problem when passing value from extent variable to ADE conversion and runtime
On FWD, HAS-LOBS attribute is supported?
This is the cause of the second conversion problem on ADE.Unsupported method or attribute KW_HAS_LOBS.
There is a test tests.buffer.attributes.TestHasLobs.cls, which has conversion issues.
#11 Updated by Teodor Gorghe about 1 year ago
Committed revision 15993 of task branch 10185a:
- added runtime and conversion support for BUFFER:HAS-LOBS.
Constantin, please check if the implementation needs to be modified. This might not be the best way. Another solution was to store this value during the conversion, into an annotation.
I am checking right now if the adm2/dataquery.p compilation works now and if there are any other issues.
#12 Updated by Teodor Gorghe about 1 year ago
With my initial conversion include files, it seems that there are no longer conversion issues. I will update this task as long as I encounter other issues. I am currently testing the runtime.
#13 Updated by Teodor Gorghe about 1 year ago
I have added adm2/query.p and it seems that we have:Unsupported method or attribute KW_IS_PART.
I am investigating this and implement this feature if needed.
#14 Updated by Teodor Gorghe about 1 year ago
Committed revision 15994 on task branch 10185a:
- Added conversion support for BUFFER:IS-PARTITIONED (currently is stubbed, always returns false).
We have another conversion problem, on adm2/browser.p, which is also an unimplemented method/attribute: Unsupported method or attribute KW_UNIQ_MAT.
#15 Updated by Teodor Gorghe about 1 year ago
- Status changed from WIP to Review
Committed revision 15995 on task branch 10185a:
- Added a partial implementation of WIDGET:UNIQUE-MATCH.
- Added some ADD-LIKE-COLUMN method overloads for fixing compilation issues on adm2/browser.p
- Added a setWidgetId method overload on using int64 id type.
Committed revision 15996 on task branch 10185a:
- Fixed compilation issues on adm2/b2b.p (the fix was suggested by Constantin).
From this stage, it seems that all .p files from adm2 gets converted and running.
#16 Updated by Teodor Gorghe about 1 year ago
Committed revision 15997 on task branch 10185a:
- Fixed compilation issues of BROWSE:NO-EMPTY-SPACE. The interface method definition was missing.
#17 Updated by Teodor Gorghe about 1 year ago
Committed revision 15998 on task branch 10185a:
- history entry fix on NameConverter.java, java_templates.tpl, Buffer.java and ComboBoxWidget.java
#18 Updated by Constantin Asofiei about 1 year ago
- Status changed from Review to Merge Pending
Conversion testing passed. Please merge 10185a after 9550a.
#19 Updated by Teodor Gorghe about 1 year ago
- Status changed from Merge Pending to Test
Branch 10185a was merged to trunk rev 16016 and archived.