<div dir="ltr">Thanks Dimitar. Please find my response below.<br><div class="gmail_extra"><br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">

> Hi guys,<br>
><br>
> I know everybody is busy with release of geany 1.24 .<br>
> Can somebody please find sometime to merge<br>
</div>> PR#226<<a href="https://github.com/geany/geany/pull/226" target="_blank">https://github.com/geany/geany/pull/226</a>> so<br>
<div class="">> that it goes to geany release 1.24?<br>
><br>
> If there is some problem please do tell me. I would fix it asap.<br>
<br>
</div>Min fields are fixed, but:<br>
<br>
It still supports the buildin parser only (not regex), and the "gcc<br>
like" syntax only (though the D and HTML examples have warnings).<br>
<br>
It still assumes that "warning" is at line_idx + 2 (unless I'm<br>
missing something), though the examples for the so called "gcc type<br>
error" for JAVA and LATEX have it at line_idx + 1.<br></blockquote><div><br></div><div>Yes PR#226 is specific for C/C++ warnings and that too for gcc like compiler error messages for c/cpp which has 'warning' keyword in line_idx+2, for all other languages (other than C/C++) the PR is harmless, it falls back to the behavior geany has now, show red squiggly line irrespective of warning or error. I can do that for other programming languages too, but I am not sure whether people programming in those languages like to see different coloring scheme for warning and error.</div>

<div><br></div><div>I have not changed the regex parser because I am still trying to find how regex parsing of compiler errors works in geany, its taking longer than I expected. </div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Last and least, pull request #191 and SF patch #11 are earlier than<br>
#226. If one of them is applied, #226 will be seriously broken.<br></blockquote><div><br></div><div>I had no idea about these PRs. Thanks for pointing this out. I will check them.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

But we already discussed all this. Why do you insist on #226 in it's<br>
current form?<br></blockquote><div><br></div><div>So I guess what you are trying to suggest is:</div><div>1. Generalize the behavior for all the programming languages supported by geany. I have to find out which languages generate warning messages and which dont, this may take some time.</div>

<div>2. Remove the hard coding of line_idx+2</div><div>3. Make changes for both regex parsing as well as older parsing method</div><div><br></div><div>If I make these changes, will then you merge the PR? Or Do I need to do more changes? Please feel free to give me feedback. I am new to geany development and I am primarily use geany for C++/Python programming so I am very biased towards these languages and might miss the bigger picture.</div>

<div><br></div><div>Thanks,</div><div>Shankhoneer Chakrovarty</div></div></div></div>