test for membership should be 'not in'

Change-Id: I493bad18ed3f728bcf42049377d7f4c6039264c8

2 files changed, 4 insertions, 4 deletions

diff --git a/writerfilter/source/ooxml/factoryimpl.py b/writerfilter/source/ooxml/factoryimpl.py
index db47039f58e9..4834c89d8fb4 100644
--- a/writerfilter/source/ooxml/factoryimpl.py
+++ b/writerfilter/source/ooxml/factoryimpl.py
@@ -127,7 +127,7 @@ std::string fastTokenToId(sal_uInt32 nToken)
 
     aliases = []
     for alias in sorted(ooxUrlAliases.values()):
-        if not alias in aliases:
+        if alias not in aliases:
             aliases.append(alias)
             print("""    case oox::NMSP_%s:
         sResult += "%s:";
@@ -139,7 +139,7 @@ std::string fastTokenToId(sal_uInt32 nToken)
 
     tokens = [""]
     for token in [t.getAttribute("localname") for t in getElementsByTagNamesNS(model, "http://relaxng.org/ns/structure/1.0", ["element", "attribute"])]:
-        if not token in tokens:
+        if token not in tokens:
             tokens.append(token)
             print("""    case oox::XML_%s:
         sResult += "%s";
diff --git a/writerfilter/source/ooxml/modelpreprocess.py b/writerfilter/source/ooxml/modelpreprocess.py
index 8d2c3a1f33a1..f81496094d47 100644
--- a/writerfilter/source/ooxml/modelpreprocess.py
+++ b/writerfilter/source/ooxml/modelpreprocess.py
@@ -37,10 +37,10 @@ def parseNamespaces(fro):
 def check(model):
     defines = [i.getAttribute("name") for i in model.getElementsByTagName("define")]
     for reference in [i.getAttribute("name") for i in model.getElementsByTagName("ref")]:
-        if not reference in defines:
+        if reference not in defines:
             raise Exception("Unknown define element with name '%s'" % reference)
     for start in [i.getAttribute("name") for i in model.getElementsByTagName("start")]:
-        if not start in defines:
+        if start not in defines:
             raise Exception("Unknown start element with name '%s'" % start)