@elextr commented on this pull request.
> @@ -418,7 +418,7 @@ static unsigned int re_string_context_at (const > re_string_t *input, int idx, #define re_string_skip_bytes(pstr,idx) ((pstr)->cur_idx += (idx)) #define re_string_set_index(pstr,idx) ((pstr)->cur_idx = (idx)) -#ifdef WIN32 +#ifdef _WIN32 Just to summarise, since we have had several misunderstandings in this thread, resulting in a long conversation on a two line change: 1. for reasons presented above we will likely need to make the upgrade of this part of ctags at some point 2. that would normally mean we need to maintain changes (or upstream them) as queried at the start of the thread 3. but since upstream has already fixed the problem any upgrade will contain a fix, even if its not what we change, so we don't need to keep our two line fix or upgrade quickly 4. so since its a large upgrade making it at leisure when its actually needed is better than rushing it @b4n conversation resolved? -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3878#discussion_r1615977685 You are receiving this because you are subscribed to this thread. Message ID: <geany/geany/pull/3878/review/[email protected]>
