<div dir="ltr"><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">FYI, I don't against general refactoring, when I agree it's really make code cleaner, readable.<br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">I against your PR because I didn't feel it really make code cleaner, readable.</div><div class="gmail_default"><span style="font-family:arial,helvetica,sans-serif">I already commented about it on the PR.</span><br></div><div class="gmail_default"><div class="gmail_default" style="font-family:sans-serif;text-decoration-style:initial;text-decoration-color:initial"><font face="arial, helvetica, sans-serif"><a href="https://github.com/python/cpython/pull/7909#issuecomment-400219905">https://github.com/python/cpython/pull/7909#issuecomment-400219905</a></font><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><span style="font-family:arial,helvetica,sans-serif"><br></span></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><span style="font-family:arial,helvetica,sans-serif">So it's not problem about general policy about refactoring / clean up.</span></div><div class="gmail_default"><span style="font-family:arial,helvetica,sans-serif">It's just my preference.  If Victor and </span><font face="arial, helvetica, sans-serif">Serhiy prefer the PR, I'm OK to merge it.</font></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><span style="font-family:arial,helvetica,sans-serif"><br></span></div><font face="arial, helvetica, sans-serif">Regards,</font><br class="gmail-Apple-interchange-newline"><br></div>-- <br><div dir="ltr" class="gmail_signature">INADA Naoki  <<a href="mailto:songofacandy@gmail.com" target="_blank">songofacandy@gmail.com</a>></div></div>