Fix validation errors (#417)

* Change Metadata.ValueNotNulTerminated to warning. * Fix typo in KTXmetalPixelFormat key.
KhronosGroup · May 5, 2021 · 78cd2b0 · 78cd2b0
1 parent 6a38a06
commit 78cd2b0
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/tools/ktx2check/ktx2check.cpp b/tools/ktx2check/ktx2check.cpp
@@ -345,7 +345,7 @@ struct {
         ERROR | 0x0087, "Unrecognized metadata \"%s\" found with KTX or ktx prefix found."
     };
     issue ValueNotNulTerminated {
-        ERROR | 0x0088, "%s value missing required NUL termination."
+        WARNING | 0x0088, "%s value missing encouraged NUL termination."
     };
     issue InvalidValue {
         ERROR | 0x0089, "%s has invalid value."
@@ -748,7 +748,7 @@ vector<ktxValidator::metadataValidator> ktxValidator::metadataValidators {
     { "KTXorientation", &ktxValidator::validateOrientation },
     { "KTXglFormat", &ktxValidator::validateGlFormat },
     { "KTXdxgiFormat__", &ktxValidator::validateDxgiFormat },
-    { "KTXMetalPixelFormat", &ktxValidator::validateMetalPixelFormat },
+    { "KTXmetalPixelFormat", &ktxValidator::validateMetalPixelFormat },
     { "KTXswizzle", &ktxValidator::validateSwizzle },
     { "KTXwriter", &ktxValidator::validateWriter },
     { "KTXwriterScParams", &ktxValidator::validateWriterScParams },
@@ -1638,7 +1638,7 @@ ktxValidator::validateOrientation(validationContext& ctx, char* key,
     }
 
     if (value[valueLen-1] != '\0')
-        addIssue(logger::eError, Metadata.ValueNotNulTerminated, key);
+        addIssue(logger::eWarning, Metadata.ValueNotNulTerminated, key);
 
     if (valueLen != ctx.dimensionCount + 1)
         addIssue(logger::eError, Metadata.InvalidValue, key);
@@ -1687,7 +1687,7 @@ ktxValidator::validateSwizzle(validationContext& ctx, char* key,
                               uint8_t* value, uint32_t valueLen)
 {
     if (value[valueLen-1] != '\0')
-        addIssue(logger::eError, Metadata.ValueNotNulTerminated, key);
+        addIssue(logger::eWarning, Metadata.ValueNotNulTerminated, key);
     if (!regex_match((char*)value, regex("^[rgba01]{4}$")))
         addIssue(logger::eError, Metadata.InvalidValue, key);
 }
@@ -1697,15 +1697,15 @@ ktxValidator::validateWriter(validationContext& ctx, char* key,
                              uint8_t* value, uint32_t valueLen)
 {
     if (value[valueLen-1] != '\0')
-        addIssue(logger::eError, Metadata.ValueNotNulTerminated, key);
+        addIssue(logger::eWarning, Metadata.ValueNotNulTerminated, key);
 }
 
 void
 ktxValidator::validateWriterScParams(validationContext& ctx, char* key,
                                      uint8_t* value, uint32_t valueLen)
 {
     if (value[valueLen-1] != '\0')
-        addIssue(logger::eError, Metadata.ValueNotNulTerminated, key);
+        addIssue(logger::eWarning, Metadata.ValueNotNulTerminated, key);
 }
 
 void