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
