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'))

I think yes, most likely, but it's used, it works (in other editors too), it's 
not a glaring mistake and I decided to do so.

-- 
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_r207929660

Reply via email to