Proposal: Bring the preprocessor in line with the established Firestorm's preprocessor

[Nensec]

Currently the preprocessor that is in Firestorm has more features that transform the code as well as simply handles things differently compared to the one in this plugin, enough so that modifying an existing script to be a lot of work. Here are some of the things I've noticed thus far:

• Includes are #include <theotherfile.lsl> in Firestorm, whereas in this plugin it is #include "theotherfile.lsl"
• When using #define SOMESYMBOL "some text" // this is some text Firestorm will omit the comment, this plugin adds it as well causing code that reads string textvar = SOMESYMBOL; to break because it becomes string textvar = "some text" // this is some text;. I feel this should be considered a bug.
• Firestorm adds some language helpers, in particular to lists allowing the use of indexes. It will transform (string)args[0] into llList2String(args,0) (with similiar llList2xxx for other data types). This is a real showstopper from switching preprocessors, at least for me.

I am not saying that one is right and one is wrong mind you 🙂

[WolfGangS]

For includes, both could in theory be supported, if we... choose to ignore the possibility of people putting " at the start / end of their folders / files

Comments after defines being included is definitely buggy behavior to me.

The last one is a little more work.

I may take a shot at 1 and 2 done after the current piece of work I'm on, but 3 might be better as it's own issue for a longer task.

[Nensec]

Great to hear!

I was actually told there is a wiki on what the Firestorm preprocessor does in more detail, hope this helps!

https://wiki.firestormviewer.org/fs_preprocessor

[WolfGangS]

Points 1 and 2 addressed in #34 and #35 respectively.

Probably best if you could refile, the 3rd item as a standalone issue, Thanks.

[Nensec]

Points 1 and 2 addressed in #34 and #35 respectively.

Probably best if you could refile, the 3rd item as a standalone issue, Thanks.

I've made #37 and #38, that should cover most of the syntax differences between preprocessors I think?

[WolfGangS]

Apart from dead code removal, but that's a much larger scope item again.

So when #34 and #35 are merged this issue can be closed.