Sick of Galaxy syntax errors? This tool should be able to show you the culprit line.
Instructions:
- Extract scripts from Core.SC2Mod
- Copy natives.galaxy inside Core.SC2Mod/TriggerLibs/
- run lapin with "-B Core.SC2Mod/ TriggerLibs/myfile.galaxy"
Syntax and static type analysis. Download attachment!
Web version available at http://www.galaxyvalidator.com/ (http://www.galaxyvalidator.com/)
Type checking e.g. http://galaxyvalidator.com/results/46a5c72eb6794ecebd1fcbce4d218e28/ (http://galaxyvalidator.com/results/46a5c72eb6794ecebd1fcbce4d218e28/)
Enjoy,
Lapin
i <3 u
There are some constructs that might not be supported. If so, please provide me with your source file. Or even better, with a simplified example of the syntax.
0.3 out w\ gui
Nice tool, already saved me a bit of time so far :)
one thing it doesn't seem to be able to validate though: the += -= *= etc operators, which i use regularly (a += 1; beats a = a + 1; in my book :P)
Version 0.4 is out.
Added the += syntax.
It will also validate the files you include in your script. So my recommendation is to make a IncludeAll.galaxy file so you can run the program on just one file which will validate all your files.
awesome :)
my 7zip won't extract the files :X
nevermind, got it to work.
unfortunately, it's not helping me find the current error ;\
If it's a type error it wont be able to catch it. Let me know what the error is when you know so maybe I can add detection for that sort of errors.
well the most usefull error validation would probably be name typo's... though that would require a bit more work i guess. But even if it just generated a list of function calls it could not find in the specified includes. (as in, put each function definition in a list and each function call in a separate list and check which calls are not in the definition list :))
It could generate a few false positives if not all includes etc are parsed, but those could be put in a separate function dictionary text file (compile once with a working build and copy/paste everything i guess ;))
same with variables per-function but that is probably even more work ;)
Version 0.5 is out
i added a simple function check with this version. I also provided a list of "built-in" functions.
Quote from: LAPIN on March 07, 2010, 09:43:45 PM
Version 0.5 is out
i added a simple function check with this version. I also provided a list of "built-in" functions.
nice ;D
:D
New version with basic struct/pointers support. Also detects a few more things.
Note: validate.bat will look in the current working directory now! And not in TriggerLibs.
I'm looking for someone to host a web asp .net front-end for this tool.
Update: http://www.galaxyvalidator.com/ (http://www.galaxyvalidator.com/)
Any help is welcomed.
Nice work.
Good job man.
I've made a simple GUI for your tool. Consider it a beta. Let me know if you have any suggestions.
http://www.mediafire.com/?tl1yidd1zyy (http://www.mediafire.com/?tl1yidd1zyy)
Cool. gj on the editor. I added it to the first post.
It's not really an editor at the moment. It simply lets you select a file to validate and lists file contents with added line numbering and validation results. I've changed it a bit, so you may want to update your link.
http://www.mediafire.com/?ym5tyli2ij3 (http://www.mediafire.com/?ym5tyli2ij3)
Changelog:
- Auto-validates a file on load.
- Textboxes are now read-only to avoid confusion.
- If it doesn't find Lapin.exe in current directory it prompts a user to find it's path.
- Status bar now works correctly.
My current goal is to link it to Notepad++ so you can validate any file you're working on by pressing e.g. F5, to get more VS-like feeling.
Gj. In your next version it would be nice to indicate the tool version number so people know if they have to update.
New version 0.7.0.
Adds warnings for undeclared varaibles (which can be annoying a bit?).
Adds error when calling a function with the wrong number of arguments.
A welcome pair of improvements! That wrong # of arguments thing has caught me several times when switching between stock and stockunitnext functions.
Feature suggestion:
allows you to run the prog with a command line switch and the name of a function, and it would return which file and line that function definition is on?
Not really high priority as we can still search manually and after a while you get to know your way around the files pretty well, but it would be nice at times and will help later when these projects start to get bigger.
Really great!
Just found out some bugs:
int func(void);
struct Link { int value; Link* next; };
Thanks for the feedback. Is void f(void) allowed? Or only void f()?
New version out 0.8.0. It allows you to read base scripts files to reduce the number of warnings.
If yo take time to setup your project, you should not get any false positives. So warnings about undeclared funtion/variables will actaully be helpful.
Hth,
Lapin
This would be really nice if it could open up the mpq files and get all natives from there instead of having them extracted.
This is great man. Haven't mentioned that yet, but this is much appreciated.
Seems to have trouble with this array declaration. The code is working.
static int[c_maxPlayers][c_OM_NextMemoryIndex] omOvermindMemory;
both of the size parameters are previously defined constants.
Nice tool! :thumbsup:
@LAPIN
ATM I'm developing a editor for Galaxy, I'd got the syntax highlighting and code suggestions working at 100%, now it only needs to follow the language rules. Do you know where I can find info about its syntaxis?. Also, I would like to ask your permission to use your program (and its future revisions) within my editor.
kazibi.-
@Astazha: i need to add support for multi-dim arrays. thats in my todo.
@kazibi: feel free to integrate it in your editor. I do not know of a good source for the language syntax. I just ask people what works and what doesn't work. :-)
Edit: Oops, I do NOT know... :-)
@Astazha: i need to add support for multi-dim arrays. thats in my todo.
@kazibi: feel free to integrate it in your editor. I do know of a good source for the language syntax. I just ask people what works and what doesn't work. :-)
0.9.1 add support for multi-dim arrays.
- Array sizing does not support operators like:
int[c_myConstant+1] myArray;
This is easy enough to get around.
- Access to multi-dim arrays does not seem to work. This code:
result=omOvermindMemory[memIndex][c_OM_InitializedIndexOffset];
stops the validate on a parse error. memIndex is an int.
Thanks again for all of your work Lapin.
Thanks for your great work.
The only thing I am missing now is proper type checking.
There is now a web version available at http://www.galaxyvalidator.com/ (http://www.galaxyvalidator.com/).
Also if you have suggestions for the site (galaxyvalidator.com) you can direct them at me on here (though I'm likely to miss it), via email (dcramer@gmail), or in this thread: http://sc2.nibbits.com/forums/15/view/435/galaxy-syntax-validator (http://sc2.nibbits.com/forums/15/view/435/galaxy-syntax-validator)
New version with static type analysis. This is the initial release of this feature...
Tested with "major" existing AIs.
Let me know if some type conversions are not right.
Have fun,
Lapin
Online version is now running 1.0.0
Version 1.1.0 is out. With small addition for more type checking.