HaxeFoundation / hxjava

Haxe Java support library. Build scripts and support code.
The Unlicense
46 stars 14 forks source link

retain LocalVariableTable #37

Open posxposy opened 3 years ago

posxposy commented 3 years ago

Hi :) It works for nightly Haxe, and argument names are parsing fine from jars. But I can't run Haxe's unit tests, so somebody should do it instead of me :)

Closes #34


But be aware, probably something was changed in the nightly builds, so this line: https://github.com/HaxeFoundation/hxjava/blob/master/jar-tool/Main.hx#L14 is not available anymore with nightly Haxe, and should be rewritten like this: import org.objectweb.asm.Opcodes and Opcodes_Statics should be used like this: Opcodes.ASM7 instead of Opcodes_Statics.ASM7

It will work for autocompletion, but the compilation process will be failed because of:

Main.hx:75: characters 17-21 : Class<org.objectweb.asm.Opcodes> has no field ASM7
Main.hx:75: characters 17-21 : Class<org.objectweb.asm.Opcodes> has no field ASM7
Main.hx:101: characters 17-21 : Class<org.objectweb.asm.Opcodes> has no field ASM7
Main.hx:101: characters 17-21 : Class<org.objectweb.asm.Opcodes> has no field ASM7

So, current sources work with Haxe 4.1.4, but not work with the nightly build.

RealyUniqueName commented 3 years ago

I've pushed a fix for Haxe dev. Try it, please.

posxposy commented 3 years ago

Ah, looks like this Main.hx:75: characters 17-21 : Class<org.objectweb.asm.Opcodes> has no field ASM7 error happend to me before, because I used Haxe dev for autocompletion and Haxe 4.1.4 for compilation 🤦‍♂️ Sorry for mislead information :)

I've pushed a fix for Haxe dev. Try it, please.

Yep, thank you, works fine!