<div dir="ltr"><p dir="ltr">Hi Sven,</p><p dir="ltr">I hope Stephan does not mind I chime in on this. I am in the same situation: still wanting to contribute, but not investing too much free time.</p><div class="gmail_quote">

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">How to call a developer's attention to small pieces of code?<br>

</blockquote><div><br></div><div>In my experience the developers already know about that and there's no need to worry.</div><div>I know these weird macros create an itch one really would like to tackle right away, but it's just not worth the trouble if you can't/don't want to compile.</div>

<div><br></div><div>If, however, you find a "2do" or a "Sollte noch optimiert werden" feel free to append a TODO/FIXME.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

And especially: may I combine that with translation patches?<br></blockquote><div><br></div><div>Keep in mind that a commit should be concise and as much as possible constrained to what the commit message says.<br></div>
<div>
That way people can keep the quality work (translation of a language they don't understand) from the "clean up the mess" work.</div><div><br></div><div>In most of my commits I translate and remove the ASCII art as that yields a lot of benefits.</div>

<div><br></div><div>BTW: It's good practice to at least run ./autogen.sh before commiting (that catches these nasty syntax errors).</div><div>See here for more: <a href="https://wiki.documentfoundation.org/Development/Patch_Handling_Guidelines#Keep_German_comment_translation_separate">https://wiki.documentfoundation.org/Development/Patch_Handling_Guidelines#Keep_German_comment_translation_separate</a></div>

<div><br></div><div>Cheers,</div><div>Philipp</div></div>
</div>