[Geany-Devel] Request to merge PR #226

Dimitar Zhekov dimitar.zhekov at xxxxx
Fri Mar 28 18:25:41 UTC 2014

On Thu, 27 Mar 2014 16:27:07 -0700
Shankhoneer Chakrovarty <shankhoneer at gmail.com> wrote:

> Hi guys,
> I know everybody is busy with release of geany 1.24 .
> Can somebody please find sometime to merge
> PR#226<https://github.com/geany/geany/pull/226> so
> that it goes to geany release 1.24?
> If there is some problem please do tell me. I would fix it asap.

Min fields are fixed, but:

It still supports the buildin parser only (not regex), and the "gcc
like" syntax only (though the D and HTML examples have warnings).

It still assumes that "warning" is at line_idx + 2 (unless I'm
missing something), though the examples for the so called "gcc type
error" for JAVA and LATEX have it at line_idx + 1.

Last and least, pull request #191 and SF patch #11 are earlier than
#226. If one of them is applied, #226 will be seriously broken.

But we already discussed all this. Why do you insist on #226 in it's
current form?

E-gards: Jimmy

More information about the Devel mailing list