<div dir="ltr">I'm sorry Markus. I'm just trying to do a thorough code review.<br><div><br>On Wed, Oct 7, 2015 at 2:28 PM, Markus Neteler <<a href="mailto:neteler@osgeo.org">neteler@osgeo.org</a>> wrote:<br>><br>> On Wed, Oct 7, 2015 at 7:07 PM, Vaclav Petras <<a href="mailto:wenzeslaus@gmail.com">wenzeslaus@gmail.com</a>> wrote:<br>> > On Wed, Oct 7, 2015 at 11:50 AM, Markus Neteler <<a href="mailto:neteler@osgeo.org">neteler@osgeo.org</a>> wrote:<br>> ...<br>> > However, here again, even when I leave aside the discussion above, there is<br>> > a typo:<br>> ><br>> > G_warning(_("Input vector map <%s> is 3D"), opt.from->answer);<br>> > G_warning(_("Input vector map <%s> is 3D"), opt.to->answer);<br>> ><br>> > The second should say "Output...".<br>><br>> No, it i correct.<br>><br>> > Now you have to change that in both branches.<br>><br>> No.<br>> Please read manual or source code.<br><br></div><div>I should probably do that, but the message should encourage user to read the manual. The current message is not informing about the the problem.<br></div><div><br>> > And that's the problem. It would be better to refine it first in<br>> > trunk and then backport it all together.<br>><br>> Please don't change it before inspecting the source code.<br>> You refer to "output" which I didn't touch.<br>><br>> > This happens quite often - list of commits containing these two pairs of<br>> > commits (change, its immediate backport, fix of change, its immediate<br>> > backport).<br>><br>> Do you refer to me??<br>> Just check my backports and my error rate on it. I do a lot of this<br>> dirty work and I don't think that I have a high error rate here.<br><br></div><div>You surely do a lot of backporting work which others do not. I'm definitively not criticizing that. I'm sorry it sounded like this. What I'm criticizing is a general practice of backporting things at the very same time they are committed to trunk.<br></div><div><br>> > Sure we make mistakes, but this is unnecessary clutter.<br>><br>> Wait.<br>> Before writing all this<br>> - read the source code first<br>> - check why I made the change. See also other people having the same issue.<br>> - help to fix the real problem (#2734) rather than writing unrelated<br>> comments here.<br><br></div><div>I might be able to fix it, but for the sake of code quality, I don't want to do it not before I will understand the motivation behind the existing code.<br></div><div><br>> > Moreover,<br>> > then we claim how many commits was done in release branch between releases,<br>> > but in fact significant portion of it are just these fixes of fixes which<br>> > should have been backported after the code matured in trunk (my claim is<br>> > that even simple code needs maturation).<br>><br>> Generally agreed. But in this case not.<br>><br>> best,<br>> Markus<br>><br>> PS: IMHO this code discussion should rather happen in grass-dev than grass-user.<br><br></div><div>That's right.<br></div></div>