Tool AdicHelper - A cJass parser

[Nestharus]

It goes-
JASS Compiler
vJASS Compiler -> JASS
cJASS Compiler -> JASS/vJASS

 

[Van Damm]

Everything that works in vJass, works in cJass.

The compiler chain is: World Editor -> AdicHelper -> JassHelper -> resulting war3map.j

 

[Frozenhelfir]

Woops, I must just be retarded

Ran into a peculiar bug with cJass that I don't recall running into with vJass

struct A{
    group immune
}

private group immune

void B(){
    A D = A.create()
    immune = D.immune
}

It tries to add the scope prefix onto D.immune, and then says it isn't a member of A.

Found another one:

A bah = A.create()
struct A{
}

It doesn't recognize struct A if you declare the global above it.

 

[quraji]

Will be using this if only to use defines...who doesn't love them?

 

[Nestharus]

A bah = A.create()
struct A{
}

keyword A...

 

[Van Damm]

New development version is out: 1.4.1.1


In this dev version we've implemented removal of unused functions from resulting map script. Example follows:

void f_00 (int i) {}
void f_01 () {}

After saving the map with Optimizer turned on you won't find the [ljass]f_00[/ljass] function in the resulting map script. But if you modify the code:

void f_00 (int i) {}
void f_01 () { f_00 (0) }

then both functions will remain in the map script. The first function was removed because it's considered optional (you won't execute it using Execute func because it takes arguments and there are no direct calls to it), [ljass]f_01[/ljass] could possibly be executed somewhere, so it won't be removed.


And now a question to users: we want some keyword to indicate functions without arguments that are not executed by ExecuteFunc and can be easily removed if not used elsewhere. We suggest optional:

optional void fx () {}

 

[TriggerHappy]

All of these script optimizations shouldn't be done in cJass -_-

cJass should strictly just be a precompiler, use an optimizer for script optimizations.

 

[Van Damm]

This optimization is done only if "Enable AdicOptimizer" menu entry is checked. I hope ADOLF will move all other optimizations there too =)

And, btw, Vex's optimizer doesn't remove excessive code.

 

[Jesus4Lyf]

I like what you're doing. I think optimisations should be done at this level, personally.

I have a request. For this request, I think I'd throw out everything for cJass.

1. Allow overriding the "destroy" method completely on structs (but allow that method to call the real "destroy").
2. Allow multiple modules to write multiple onInit methods, and if possible, onAllocate and onDeallocate or something.

What do you think?

Edit: Don't worry, seems I finally contacted Vex.
And it looks like he's willing to fix stuff.

 

[Narks]

For your optimizer, maybe make it create a report of all the functions that could be optimized, but have not (because of execfunc stuff)?

 

[Van Damm]

For your optimizer, maybe make it create a report of all the functions that could be optimized, but have not (because of execfunc stuff)?

Interesting idea... We'll think aboout that.

 

[Troll-Brain]

I hope ADOLF will move all other optimizations there too =)

I also hope it, including the removing of comments/empty lines.

 

[Frozenhelfir]

AdicHelper seems to crash when compiling AIDS, does anyone else get this problem?

 

[Sevion]

Ditto. But somehow, I've never had it when I used both AIDS and cJASS So so far, it's not been a problem. But it would prove to be problematic if I did use AIDS and cJASS at the same time.

 

[Lyerae]

Or if the cJASS compiler had something like //! cjass and //! endcjass, which it ignores anything not in those tags.

 

[Sevion]

Or perhaps a way you can mark a trigger to not parse? Like, if you could right click the trigger on the left pane, and click "Don't cJASS Parse" or something.

 

[Frozenhelfir]

OR just have it ignore things in those tags. This way you only have to use it on a few systems that don't work if you are coding the whole map in cjass like I am

 

[Sevion]

Well, with the tags, you'd have to scroll back and forth in the library and that can get tiring if it's a large one. (Or be smart and use the scroll bar...)

If you cold easily just right click the trigger object and click "Don't Parse" that'd be simpler. Not as simple to code, but simpler in the end result.

 

[Van Damm]

@Frozenhelfir:
The latest development version (1.4.1.20) compiles the latest version of AIDS without errors. You just need to update from development branch if you use stable version.

 

[Frozenhelfir]

Cool stuff ! Thanks