http://codereview.appspot.com/6219052/diff/1/MoinMoin/search/queryparser/expr... File MoinMoin/search/queryparser/expressions.py (right): http://codereview.appspot.com/6219052/diff/1/MoinMoin/search/queryparser/expr... MoinMoin/search/queryparser/expressions.py:44: def __init__(self, pattern, use_re=False, case=False, highlight=1): True above, 1 here? http://codereview.appspot.com/6219052/diff/1/MoinMoin/search/queryparser/expr... MoinMoin/search/queryparser/expressions.py:232: return u'' please always do it that way (see other hunks)