<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<div class="moz-cite-prefix">On 06/21/2013 06:45 PM, Glyph wrote:<br>
</div>
<blockquote
cite="mid:0183FFA8-FC78-4380-8231-D6E0EF2194E8@twistedmatrix.com"
type="cite">
<meta http-equiv="Content-Type" content="text/html;
charset=ISO-8859-1">
<br>
<div>
<div>On Jun 21, 2013, at 7:20 AM, William McCall <<a
moz-do-not-send="true"
href="mailto:william.mccall@gmail.com">william.mccall@gmail.com</a>>
wrote:</div>
<br class="Apple-interchange-newline">
<blockquote type="cite"><span style="font-family: Monaco;
font-size: 12px; font-style: normal; font-variant: normal;
font-weight: normal; letter-spacing: normal; line-height:
normal; orphans: auto; text-align: start; text-indent: 0px;
text-transform: none; white-space: normal; widows: auto;
word-spacing: 0px; -webkit-text-stroke-width: 0px; float:
none; display: inline !important;">Comments welcome!</span><br
style="font-family: Monaco; font-size: 12px; font-style:
normal; font-variant: normal; font-weight: normal;
letter-spacing: normal; line-height: normal; orphans: auto;
text-align: start; text-indent: 0px; text-transform: none;
white-space: normal; widows: auto; word-spacing: 0px;
-webkit-text-stroke-width: 0px;">
</blockquote>
</div>
<br>
<div>The main comment I have is: no tests!!! ;)</div>
<div><br>
</div>
<div>If you're going to submit this as a patch, it would of course
be very helpful to have unit test coverage.</div>
<div><br>
</div>
<div>If you need help figuring out how to do that, feel free to
ask questions.</div>
<div><br>
</div>
<div>-glyph</div>
<div><br>
</div>
<br>
</blockquote>
<br>
Thanks for the comments and support, Glyph. I'll work on that soon.
<br>
<br>
--WM<br>
</body>
</html>