verbrannt

Idk. I have only one occurrence of this variable in my ai.obj files, so can't tell. Also I think boss_pch.txt should be renamed to gm_pch.txt. "Boss" prefix is misleading here.

It's already reverted.

It's definitely global map ids :)

New release: https://github.com/madyanov/nasc-decompiler/releases/tag/v1.4 HF Support by Eressea GD Support by @ChaosPaladin Many NASC generator fixes Split result classes into tree Non-ASCII symbols support by Eressea

This parameter is from another AI (not default GF/Freya). AdvExt uses it, for example. Please provide part of the ai.obj where this happens.

_blank is a property defined in the default_npc class. string _blank = "";

I'm refactoring main.php right now thats the reason :)

BTW tree splitting produces too long paths in Windows, so it can't handle it properly =/ Max path length in Windows is 260 or something, so I'll limit tree depth.

So FString lookup should be removed from all places, no need to check NASC version. Anyway getString method not working now. Yeah, I thought about it.

Can you make a PR or patch?

Fixed.

Thx for h5 defs! As for precompiled headers issue, I didn't knew such syntax is possible (dots in these constants). I've added simple check for dot, and if dot exists, constant will be quoted. Checkout the master branch. Idk. Found similar code in the leaked GF AI. Class ssq_npc_priest for example: if (2 == 2) { ShowSystemMessage(talker, 1273); } else if (2 == 1) { ShowSystemMessage(talker, 1274); } select (1) { case 1: ShowSystemMessage(talker, 1275); break; case 2: ShowSystemMessage(talker, 1276); break; case 3: ShowSystemMessage(talker, 1277); break; }

Yeah, this warning because of constants in statements like select/if/while. I think NCSoft uses some sort of preprocessor for AI scripts, that generates such strange code.

Seems like code generated correctly, but looks suspicious. ... push_const 5 //unary->INTEGER_CONST push_const 100 //unary->INTEGER_CONST equal ... push_const 35002 //unary->INTEGER_CONST push_const 35001 //unary->INTEGER_CONST equal ... Stack machine in my head generates same code as decompiler :) BTW: Can I push your h5 data folder to my repository? Also the '@' prefixes in enums.json now not needed (because of generators to another languages), & type P_STATE renamed to PSTATE.

Was bug in NASC generator. Fix in master. Sounds convenient. I'll make it default behaviour.

Yea, thank you. I will apply your patch. Only reason I’m using UTF-16LE is to be able to compile result code using your compiler. I though it only supports UTF-16LE.

Decompiler can’t build correct AST for these classes. It fails only with ai.obj from the leaked GF (l2off). With another ai.obj (e.g. from advext) it should work fine. I’ll fix this after releasing support of the leaked Freya AI.

Thx. Fixed now, checkout the master branch.

Known issue. Not critical, and wont be fixed, but you can make PR.

Hi, thx for feedback. I just fixed this, you can pull from master.

https://github.com/madyanov/nasc-decompiler/releases Decompiles ai.obj to AST, then converts AST to NASC (but can be converted to any other language, such as Java, etc). Resulting NASC can be successfully compiled back with Eressea’s compiler. To run, put ai.obj file to current folder and double click on run.bat.

Thank you! Can you please explain how did you get the CNPC.h & CMaker.h files with signatures of NASC methods and their addresses (in comments)? UPD: I think I found some signatures in the OSIE project, but only methods, not variables.

@eressea Issue: You can't use voicecommands if your chat is temporary blocked by GM. So, for example, you can't .expon. I think this is critical.

@eresseaHi, can you share the DEcompiler you used for this AI? Thank you.