Please properly describe what you are trying to do. Except for the two re.UNICODE changes, I see nothing that is suitable. https://codereview.appspot.com/8277048/diff/1/MoinMoin/converter/_args_wiki.py File MoinMoin/converter/_args_wiki.py (right): https://codereview.appspot.com/8277048/diff/1/MoinMoin/converter/_args_wiki.p... MoinMoin/converter/_args_wiki.py:33: _parse_re = re.compile(_parse_rules, re.X | re.UNICODE) Maybe okay https://codereview.appspot.com/8277048/diff/1/MoinMoin/converter/_args_wiki.p... MoinMoin/converter/_args_wiki.py:48: value = (match.group(2) or match.group(3) or match.group(4)) Nope. The strings can be escaped. And they are documented this way. https://codereview.appspot.com/8277048/diff/1/MoinMoin/converter/_args_wiki.p... MoinMoin/converter/_args_wiki.py:75: value = u'"' + value + u'"' Not okay