FWD » Core Development » Conversion Tools

Example 4gl:

USING java.io.ByteArrayOutputStream FROM JAVA.
DEFINE VARIABLE bos AS ByteArrayOutputStream.
bos = new ByteArrayOutputStream(10000).

USING java.io.DataOutputStream FROM JAVA.
DEFINE VARIABLE out AS DataOutputStream.
out = new DataOutputStream(bos).

Converts to Java as:

   ...
   @LegacySignature(type = Type.VARIABLE, name = "bos")
   jobject<? extends ByteArrayOutputStream> bos = UndoableFactory.jobject(ByteArrayOutputStream.class);

   @LegacySignature(type = Type.VARIABLE, name = "out")
   jobject<? extends DataOutputStream> out = UndoableFactory.jobject(DataOutputStream.class);

   @LegacySignature(type = Type.MAIN, name = "test-java.p")
   public void execute()
   {
      externalProcedure(TestJava.this, new Block((Init) () -> 
      {
         ObjectOps.register(out, bos);
      }, 
      (Body) () -> 
      {
         bos.assign(new java.io.ByteArrayOutputStream(10000));
         out.assign(new java.io.DataOutputStream(bos));
      }));
   }

Please try this patch:

### Eclipse Workspace Patch 1.0
#P p2j
Index: rules/convert/variable_references.rules
===================================================================
--- rules/convert/variable_references.rules    (revision 2943)
+++ rules/convert/variable_references.rules    (working copy)
@@ -901,6 +901,13 @@
                <rule on="false">not(this.indexPos == 1 and parent.type == prog.object_invocation)
                   <action>lastid = closestPeerId</action>

+                  <!-- java variables need a 'ref' if passed as an argument to a Java-style call -->
+                  <rule>parent.type == prog.parameter and 
+                        evalLib("is_java_cls", this)  and
+                        evalLib("is_java_cls", parent.parent.parent)
+                     <action>lastid = createJavaAst(java.method_call, "ref", lastid)</action>
+                  </rule>
+
                   <!-- emit variable/accessor -->
                   <action>createPeerAst(methodType, methodTxt, lastid)</action>

Constantin Asofiei wrote:

Please try this patch:
[...]

It works, thank you! Will you commit the patch?