[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 2/3] tracetool: clarify that "formats" means "format
From: |
Stefan Hajnoczi |
Subject: |
[Qemu-devel] [PATCH 2/3] tracetool: clarify that "formats" means "format strings" |
Date: |
Wed, 10 Jan 2018 20:25:52 +0000 |
The terminology used by tracetool is not consistent with C sprintf or
docs/devel/tracing.txt. The word "formats" is sometimes used to mean
"format strings".
This patch clarifies comments and error messages that contain this word.
Note that the error message lines are longer than 80 characters but I
have not wrapped them to aid grepping.
Signed-off-by: Stefan Hajnoczi <address@hidden>
---
scripts/tracetool/__init__.py | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/scripts/tracetool/__init__.py b/scripts/tracetool/__init__.py
index fbccaaa0cf..5db1f5c395 100644
--- a/scripts/tracetool/__init__.py
+++ b/scripts/tracetool/__init__.py
@@ -173,7 +173,7 @@ class Event(object):
props : list of str
Property names.
fmt : str, list of str
- Event printing format (or formats).
+ Event printing format string(s).
args : Arguments
Event arguments.
orig : Event or None
@@ -237,9 +237,9 @@ class Event(object):
if "tcg-exec" in props:
raise ValueError("Invalid property 'tcg-exec'")
if "tcg" not in props and not isinstance(fmt, str):
- raise ValueError("Only events with 'tcg' property can have two
formats")
+ raise ValueError("Only events with 'tcg' property can have two
format strings")
if "tcg" in props and isinstance(fmt, str):
- raise ValueError("Events with 'tcg' property must have two
formats")
+ raise ValueError("Events with 'tcg' property must have two format
strings")
event = Event(name, props, fmt, args)
@@ -263,7 +263,7 @@ class Event(object):
_FMT = re.compile("(%[\d\.]*\w+|%.*PRI\S+)")
def formats(self):
- """List of argument print formats."""
+ """List conversion specifiers in the argument print format string."""
assert not isinstance(self.fmt, list)
return self._FMT.findall(self.fmt)
--
2.14.3
- [Qemu-devel] [PATCH 0/3] tracetool: improve error messages, Stefan Hajnoczi, 2018/01/10
- [Qemu-devel] [PATCH 1/3] tracetool: prefix parse errors with line numbers, Stefan Hajnoczi, 2018/01/10
- [Qemu-devel] [PATCH 3/3] tracetool: report error on foo() instead of foo(void), Stefan Hajnoczi, 2018/01/10
- [Qemu-devel] [PATCH 2/3] tracetool: clarify that "formats" means "format strings",
Stefan Hajnoczi <=
- Re: [Qemu-devel] [PATCH 0/3] tracetool: improve error messages, Eric Blake, 2018/01/10
- Re: [Qemu-devel] [PATCH 0/3] tracetool: improve error messages, no-reply, 2018/01/10
- Re: [Qemu-devel] [PATCH 0/3] tracetool: improve error messages, Stefan Hajnoczi, 2018/01/15