<html>
<head>
<meta http-equiv="content-type" content="text/html; charset=UTF-8">
</head>
<body><p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">On Friday, January 29, 2021 5:05:19 AM WET Richard Kimberly Heck wrote:</p>
<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> José, can you look at these please?</p>
<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> </p>
<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">> Riki</p>
<br /><p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">I think that Thibaut wants to impose black[1] on us. :-D</p>
<br /><p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">Most of the changes are cosmetic, e.g. one import per line, or to use the same indentation for documentation.</p>
<br /><p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">The only bits where I am unsure are on the regular expressions (I am trimming the expression to make it easier to read):</p>
<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">-    declare = re.compile('\\s*(\[([^,]*)(,.*)*\])*$')</p>
<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">+    declare = re.compile('\\s*(\[([^,]*)(,.*)*])*$')</p>
<br /><p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">The left bracket is escaped but not the right one. What do other, more knowledgeable about the black magic of regular expressions think?</p>
<br /><p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">The are other issues like mistakes in the documentation strings. For the moment those are harmless, I am referring to e.g. "\c" since \c it is not an escape sequence it gets transformed to "\\c".</p>
<br /><br /><p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">With the extent of the patches I fear that there could be bugs (unintended changes) lurking specially in relation to python 2.</p>
<br /><p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">In particular this just reinforces my feeling that configure.py should get an overhaul. :-)</p>
<br /><p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">This is my preliminary overview. I will keep looking into the patches.</p>
<br /><br /><p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">[1] From Henry Ford's T model "You can choose any color as long as it is black".</p>
<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;"><a href="https://black.readthedocs.io/en/stable/">https://black.readthedocs.io/en/stable/</a> </p>
<br /><p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">-- </p>
<p style="margin-top:0;margin-bottom:0;margin-left:0;margin-right:0;">José Abílio</p>
</body>
</html>