[Github-comments] [geany/geany] Add AutoIt syntax highlighting and Ctags parser for AutoIt (#1752)

[Skif-off]

Skif-off commented on this pull request.



> +	vString *name = vStringNew ();
+	const unsigned char *line;
+
+	while ((line = readLineFromInputFile ()) != NULL)
+	{
+		const unsigned char* p = line;
+		if (p [0] == '#')
+		{
+			/* min. string "#region" > 7 */
+			if ((p [1] == 'R' || p [1] == 'r') &&
+				strlen ((const char *) p) > 8 &&
+				(p [2] == 'E' || p [2] == 'e') &&
+				(p [3] == 'G' || p [3] == 'g') &&
+				(p [4] == 'I' || p [4] == 'i') &&
+				(p [5] == 'O' || p [5] == 'o') &&
+				(p [6] == 'N' || p [6] == 'n'))

> results in showing a region named ```test``` in the symbols tree.

I cheked it again and now I see, I don't know what it was :) It's not valid syntax, in this case Geany does not highlight ```#Region``` and I think it's good notice to check.

> Additionally with your examples, the leading ```;``` is included in the displayed region name, is that intentional? If not, you probably could simply use ...

I could not find the info (documentation, official and not official forum) how to use it correctly and what will be region name. I checked all variants with AutoIt's tool that checks the syntax of AutoIt scripts and decided to leave so.

This is often used, I personally use form ```#Region ;test``` because in this case name ```;test``` has a different color than the color of ```#Region```, I like this.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/1752#discussion_r207909069
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20180806/d9b55057/attachment.html>