From 6a1e8295b14a5080856659d49b65e70c76b70bd3 Mon Sep 17 00:00:00 2001
From: rocky <rb@dustyfeet.com>
Date: Thu, 14 Dec 2017 16:02:48 -0500
Subject: [PATCH] Increase test coverage. Remove some epsilon reductionsa

---
 test/bytecode_3.1/00_import.pyc | Bin 331 -> 423 bytes
 test/bytecode_3.2/00_import.pyc | Bin 331 -> 423 bytes
 test/bytecode_3.6/00_import.pyc | Bin 316 -> 316 bytes
 uncompyle6/parser.py            |  15 ++++++++++++---
 uncompyle6/parsers/parse36.py   |   2 +-
 5 files changed, 13 insertions(+), 4 deletions(-)

diff --git a/test/bytecode_3.1/00_import.pyc b/test/bytecode_3.1/00_import.pyc
index cb446008e3a9f0c7362cb4594c1e8b52e6d9841d..60d4f3f5879b68400f0e6d9682f5e1cdc2fe1ca5 100644
GIT binary patch
delta 156
zcmX@jw47PqpNE%Af2UGZG6NFe0J0r`xVU1Xe1KXiCqpV1Lo*{o3Ijtn6GKr6h>^m`
zki*T8#lw)r%MiuKkjF4FPm7U%;>-lYQf8peoYcHhkSX!;X_+~x@$sdsKu)4wl3sEt
fBaoE@CL!#Ui9g*0cz}XDi~@{Yj1r9ej6zHRfGZne

delta 64
zcmZ3^e40t#pNE%AdAeLwG6ND|2eKW2xHxd4e89wMnv9$i4<t+uWOU=<0E+T33NUgp
HN-zQdVy6l<

diff --git a/test/bytecode_3.2/00_import.pyc b/test/bytecode_3.2/00_import.pyc
index 89ca5d6a7d71fd10d47995ff89ade2623439bc33..931c34184afdc478c1dad8a9ca7d8348e8974742 100644
GIT binary patch
delta 156
zcmX@jw47N!hliI-f2UGZG6ND|0<s-|xVU1Xe1KXiCqpV1Lo*{o3Ijtn6GKr6h>^m`
zki*T8#lw)r%MiuKkjF4FPm7U%;>-lYQf8peoYcHh4j>&LpO%@E8XsTE3gjf}CFv!X
gG6GpiU=qSknfTL9fCnhZ!zjSW#VEnZ&nUzM0FF8vcmMzZ

delta 64
zcmZ3^e40r<hliI-dAeLwG6ND|0<s-|xHxd4e89wMnv9$i4<t+uWOU=<0E+T33NUgp
HN-zQdXqXC3

diff --git a/test/bytecode_3.6/00_import.pyc b/test/bytecode_3.6/00_import.pyc
index 9326c33bfe8f442ea9e50b02b8b60d3a3543ad5d..e5c460ab6f09052afc48654c3a297bd0eb7806f9 100644
GIT binary patch
delta 16
XcmdnPw1<h^n3tDJf2Yz$c2h<GBdG(^

delta 16
XcmdnPw1<h^n3tF9in{zpc2h<GB(VeI

diff --git a/uncompyle6/parser.py b/uncompyle6/parser.py
index c2e368d9..72855f60 100644
--- a/uncompyle6/parser.py
+++ b/uncompyle6/parser.py
@@ -46,14 +46,18 @@ class PythonParser(GenericASTBuilder):
             ]
         self.collect = frozenset(nt_list)
 
+        # For these items we need to keep the 1st epslion reduction since
+        # the nonterminal name is used in a semantic action.
+        self.keep_epsilon = frozenset(('kvlist_n', 'kvlist'))
+
         # ??? Do we need a debug option to skip eliding singleton reductions?
         # Time will tell if it if useful in debugging
 
         # FIXME: optional_nt is a misnomer. It's really about there being a
         # singleton reduction that we can simplify. It also happens to be optional
         # in its other derivation
-        self.optional_nt |= frozenset(['come_froms', 'suite_stmts', 'l_stmts_opt',
-                                       'c_stmts_opt'])
+        self.optional_nt |= frozenset(('come_froms', 'suite_stmts', 'l_stmts_opt',
+                                       'c_stmts_opt'))
 
         # Reduce singleton reductions in these nonterminals:
         # FIXME: would love to do expr, sstmts, stmts and
@@ -196,7 +200,12 @@ class PythonParser(GenericASTBuilder):
             else:
                 rv = args[0]
                 pass
-            rv.append(args[1])
+            # In a  list-like entity where the first item goes to epsilon,
+            # drop that and save the 2nd item as the first one
+            if len(rv) == 0 and nt not in self.keep_epsilon:
+                rv = args[1]
+            else:
+                rv.append(args[1])
         elif n == 1 and args[0] in self.singleton:
             rv = GenericASTBuilder.nonterminal(self, nt, args[0])
             del args[0] # save memory
diff --git a/uncompyle6/parsers/parse36.py b/uncompyle6/parsers/parse36.py
index 74ec3da1..a7a773a1 100644
--- a/uncompyle6/parsers/parse36.py
+++ b/uncompyle6/parsers/parse36.py
@@ -141,7 +141,7 @@ class Python36Parser(Python35Parser):
             self.addRule("""expr        ::= call_ex_kw
                             expr        ::= call_ex_kw2
                             call_ex_kw  ::= expr expr build_map_unpack_with_call
-                                           CALL_FUNCTION_EX_KW
+                                            CALL_FUNCTION_EX_KW
                             call_ex_kw2 ::= expr
                                             build_tuple_unpack_with_call
                                             build_map_unpack_with_call