<div dir="ltr">Am 2016-04-09 um 12:20 schrieb Giuliano Colla:<br>> An answer such as: "Your patch is good, it adds a desirable functionality, <div>> but without proper documentation, nobody will be able to take advantage of it. </div><div>> Please add some concise comments on variable and procedure usage, and we </div><div>> will be glad to commit it." would do a lot of good, IMHO.<br>> Should this become a general rule, the situation might strongly improve with time.<br><div class="gmail_extra"><br></div><div class="gmail_extra">Agreed :-)  In reality though we shouldn't exclude patches just because they lack the above.  We should only be encouraging decent documentation, not enforcing it.</div><div class="gmail_extra"><br></div><div class="gmail_extra">"Yay, we've got a patch that finally fixes that annoying TreeView issue, but rats, we can't commit it because those variables aren't sufficiently documented"</div><div class="gmail_extra"><br></div><div class="gmail_extra">Mike</div></div></div>