<p><a class="user-mention" data-hovercard-type="user" data-hovercard-url="/hovercards?user_id=564520" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/kugel-">@kugel-</a> said:</p>
<blockquote>
<p>Why even post a PR if you're reluctant to even minor suggestions?</p>
</blockquote>
<p>There is nothing wrong with making suggestions for improvements "while you are there" but it also should be ok for contributors to say "not just now".  The Geany crew (me included) have a tendency to make perfection the enemy of improvement.</p>
<p>Of course we should object to bad changes, changes that cause problems or are poorly implemented, even changes that we simply don't like, but that shouldn't stop changes that are reasonable interim steps being made, even if they are not the final state we would prefer.</p>
<p>This change isn't hugely urgent, since the code being changed is working its only a safety net changing a macro to a function, removing the risk of local names from later changes being captured.  This would be important if there were regular changes in the code using it, but there have not been many changes there.</p>
<p>But the change does make an improvement, so why not if its an itch <a class="user-mention" data-hovercard-type="user" data-hovercard-url="/hovercards?user_id=1107820" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/ntrel">@ntrel</a> wants to scratch.</p>
<p><a class="user-mention" data-hovercard-type="user" data-hovercard-url="/hovercards?user_id=1107820" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/ntrel">@ntrel</a> said:</p>
<blockquote>
<p>So reviewers can check my changes don't introduce bugs.</p>
</blockquote>
<p>Yes, please do NOT commit directly, although review and test can be annoying and takes time its important.  We don't have a big enough crew to have to do the extra effort to go back and find and fix accidentally introduced bugs.</p>
<p>And we all know somebody else testing <em>always</em> tries something we didn't think of <g-emoji class="g-emoji" alias="grin" fallback-src="https://github.githubassets.com/images/icons/emoji/unicode/1f601.png">😁</g-emoji></p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/geany/geany/pull/2256?email_source=notifications&email_token=AAIOWJYHUVFFTLDGWXEN6Q3QEM7TBA5CNFSM4ILBSKBKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4HITZI#issuecomment-521046501">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAIOWJ73CUQOG7IXQDKHTN3QEM7TBANCNFSM4ILBSKBA">mute the thread</a>.<img src="https://github.com/notifications/beacon/AAIOWJ4723C74I3AQSECGNLQEM7TBA5CNFSM4ILBSKBKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4HITZI.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/geany/geany/pull/2256?email_source=notifications\u0026email_token=AAIOWJYHUVFFTLDGWXEN6Q3QEM7TBA5CNFSM4ILBSKBKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4HITZI#issuecomment-521046501",
"url": "https://github.com/geany/geany/pull/2256?email_source=notifications\u0026email_token=AAIOWJYHUVFFTLDGWXEN6Q3QEM7TBA5CNFSM4ILBSKBKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4HITZI#issuecomment-521046501",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>