<div dir="ltr">You did Rijndael in Pure Perl it must be tough to get any performance out of it.</div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, May 7, 2017 at 9:43 PM, Jim Bacon <span dir="ltr"><<a href="mailto:boftx@hotmail.com" target="_blank">boftx@hotmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I think that critiques of documentation are as important as those of style and maintainability. That also includes level of comments. What comments could be included to clarify logic as well and comments that just muddy the waters.<br>
<span class="HOEnZb"><font color="#888888"><br>
Jim<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
> On May 7, 2017, at 7:32 PM, G. Wade Johnson <<a href="mailto:gwadej@anomaly.org">gwadej@anomaly.org</a>> wrote:<br>
><br>
> The code review session sounds like an interesting new thing to try. In<br>
> order to make this work, we'll need to have 2-3 modules that people are<br>
> willing to have reviewed.<br>
><br>
> Jim has volunteered some code. Anyone else?<br>
><br>
> We also need to set some ground rules about the kind of review we are<br>
> going for.<br>
><br>
> - Level of pickiness<br>
>   - Style issues?<br>
>   - Bugs only<br>
>   - Maintainability<br>
> - Security?<br>
> - Architectural improvements?<br>
><br>
> I'd also suggest that people carefully think in terms of three<br>
> different kinds of comments:<br>
><br>
>  - Question<br>
>    - Looking for clarification<br>
>    - Might suggest minor change or documentation for clarity<br>
>  - Comment<br>
>    - Non-fatal issue that might be worth changing or considering<br>
>  - Flaw<br>
>    - Bug<br>
>    - Logic error<br>
><br>
> Obviously, we have no way of enforcing changes. And, we want to all<br>
> remain friendly after the fact. We are just looking for good quality<br>
> code in the end.<br>
><br>
> Does this sound like an approach that everyone can agree to?<br>
><br>
> G. Wade<br>
> --<br>
> Uncertainty is an uncomfortable position. But certainty is an absurd<br>
> one. -- Voltaire<br>
> ______________________________<wbr>_________________<br>
> Houston mailing list<br>
> <a href="mailto:Houston@pm.org">Houston@pm.org</a><br>
> <a href="http://mail.pm.org/mailman/listinfo/houston" rel="noreferrer" target="_blank">http://mail.pm.org/mailman/<wbr>listinfo/houston</a><br>
> Website: <a href="http://houston.pm.org/" rel="noreferrer" target="_blank">http://houston.pm.org/</a><br>
______________________________<wbr>_________________<br>
Houston mailing list<br>
<a href="mailto:Houston@pm.org">Houston@pm.org</a><br>
<a href="http://mail.pm.org/mailman/listinfo/houston" rel="noreferrer" target="_blank">http://mail.pm.org/mailman/<wbr>listinfo/houston</a><br>
Website: <a href="http://houston.pm.org/" rel="noreferrer" target="_blank">http://houston.pm.org/</a><br>
</div></div></blockquote></div><br></div>