search

Storyyeller / Krakatau

Java decompiler, assembler, and disassembler
GNU General Public License v3.0
1.95k stars 220 forks source link

Jython compatibility #24

Closed alexkasko closed 2 years ago

alexkasko commented 10 years ago

Hi,

I've successfully used this project with Jython 2.7b1 (you may see results in my github project).

I applied this patch - https://gist.github.com/alexkasko/6781413 , besides jython compatibility changes (packages naming and struct creation in assembler.py) it also changes some \n to in generated source code (these changes are, of course, optional).

Thanks for the great tools!

Storyyeller commented 10 years ago

Is there any particular reason you wanted to remove all the newlines from the generated source code?

Anyway, I'll look into the relative package thing. I'm not to familiar with Jython, but it would be nice to support it if possible.

alexkasko commented 10 years ago

No, no particular reason, I've just made it looks more like typical java code (with braces on the same line).

Besides relative imports only error was _format_ops['>'] thing, I don't understand what is going on here, but jython cannot create PyStruct with empty string argument, and '>' argument fixed it (and I hope didn't broke anything).

Storyyeller commented 2 years ago

Closing this as I'm rewriting Krakatau in Rust.