Also note the test failures on Python 3.3 - `ValueError: BoolOp with less
than 2 values`.

The spec doesn't say that `And`, `Or` or `Not` have to contain multiple
values - and obviously `Not` must be able to have a single value. So in
each case, we should have a simple check like `if len(expr.values)==1: expr
= expr.values[0]`.

To be safe, we might also need to handle cases with an empty And/Or/Not
node.


On 5 February 2013 18:07, ju1ius <notifications@github.com> wrote:

> In xdg/Menu.py:
>
> > @@ -312,32 +313,46 @@ def parseMerge(self, type="all"):
> >          self.order.append(["Merge", type])
> >
> >
> > +class RuleTransformer(ast.NodeTransformer):
>
> Yep, forgot to delete it...
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/takluyver/pyxdg/pull/3/files#r2897152>.
>
>