diff mbox series

[committed] benchtests: make compare_strings.py accept string as attribute value

Message ID 20220308141711.1013545-1-siddhesh@sourceware.org
State New
Headers show
Series [committed] benchtests: make compare_strings.py accept string as attribute value | expand

Commit Message

Siddhesh Poyarekar March 8, 2022, 2:17 p.m. UTC 
From: Su Lifan <su-lifan@linux.alibaba.com>

Commit ac759b1fbf28a82d99afde9046f8b72c7cba5dae added attribute
"overlap" to bench-memmove-walk, whose value is a string. This change
makes compare_strings.py fail since benchout_strings.schema.json
requires the values of attributes to be number.

This patch relaxes such constraint.
Reviewed-by: Siddhesh Poyarekar <siddhesh@sourceware.org>
---

I've fixed up the patch slightly before applying since it had been
mangled by your mail client.  Please make sure you use git-send-email or
another method in future that sends out patches without modification.

Also, this contribution is legally trivial but in future please make
sure that you either confirm your copyright assignment to the FSF or
provide a DCO.  The contribution checklist has more details about all of
this:

https://sourceware.org/glibc/wiki/Contribution%20checklist

Thank you for contributing!

 benchtests/scripts/benchout_strings.schema.json | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)
diff mbox series

Patch

diff --git a/benchtests/scripts/benchout_strings.schema.json b/benchtests/scripts/benchout_strings.schema.json
index 7c9a67134b..bfd9b4e6c0 100644
--- a/benchtests/scripts/benchout_strings.schema.json
+++ b/benchtests/scripts/benchout_strings.schema.json
@@ -28,7 +28,12 @@ 
 		    "items": {"type": "number"}
                    }
 		 },
-		 "additionalProperties": {"type": "number"},
+		 "additionalProperties": {
+		   "oneOf": [
+		     { "type": "number" },
+		     { "type": "string" }
+		   ]
+		 },
 		 "minProperties": 2
 	      }
 	    }