Isues spotted in review

uploadcare · Jul 16, 2024 · bfa73ce · bfa73ce
1 parent 658362a
commit bfa73ce
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 4 deletions.
diff --git a/Tests/test_file_webp_animated.py b/Tests/test_file_webp_animated.py
@@ -15,7 +15,7 @@
     skip_unless_feature,
 )
 
-pytestmark = [skip_unless_feature("webp")]
+pytestmark = skip_unless_feature("webp")
 
 
 def test_n_frames() -> None:

diff --git a/Tests/test_file_webp_metadata.py b/Tests/test_file_webp_metadata.py
@@ -10,7 +10,7 @@
 
 from .helper import mark_if_feature_version, skip_unless_feature
 
-pytestmark = [skip_unless_feature("webp")]
+pytestmark = skip_unless_feature("webp")
 
 ElementTree: ModuleType | None
 try:

diff --git a/setup.py b/setup.py
@@ -795,7 +795,12 @@ def build_extensions(self):
             _dbg("Looking for webp")
             if all(
                 _find_include_file(self, src)
-                for src in ["webp/encode.h", "webp/mux.h", "webp/demux.h"]
+                for src in [
+                    "webp/encode.h",
+                    "webp/decode.h",
+                    "webp/mux.h",
+                    "webp/demux.h",
+                ]
             ):
                 # In Google's precompiled zip it is call "libwebp":
                 if all(

diff --git a/src/_webp.c b/src/_webp.c
@@ -781,7 +781,7 @@ setup_module(PyObject *m) {
     }
 
     PyObject *v = PyUnicode_FromString(WebPDecoderVersion_str());
-    PyModule_AddObject(m, "webpdecoder_version", v ? v : Py_None);
+    PyModule_AddObject(m, "webpdecoder_version", v);
 
     return 0;
 }