qapi/pragma: Tidy up comments

[PATCH] qapi/pragma: Tidy up comments

Markus Armbruster posted 1 patch 1 year, 11 months ago

Download mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20220510081433.3289762-1-armbru@redhat.com

Maintainers: Eric Blake <eblake@redhat.com>, Markus Armbruster <armbru@redhat.com>

qapi/pragma.json | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)

Expand all Fold all

[PATCH] qapi/pragma: Tidy up comments

Posted by Markus Armbruster 1 year, 11 months ago

Commit 05ebf841ef "qapi: Enforce command naming rules" inserted new
code between a comment and the code it applies to.  Move the comment
back to its code, and add one for the new code.

Signed-off-by: Markus Armbruster <armbru@redhat.com>
---
 qapi/pragma.json | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/qapi/pragma.json b/qapi/pragma.json
index e6a021c19c..7f810b0e97 100644
--- a/qapi/pragma.json
+++ b/qapi/pragma.json
@@ -6,7 +6,7 @@
 # Whitelists to permit QAPI rule violations; think twice before you
 # add to them!
 { 'pragma': {
-    # Commands allowed to return a non-dictionary:
+    # Command names containing '_'
     'command-name-exceptions': [
         'add_client',
         'block_resize',
@@ -24,6 +24,7 @@
         'system_powerdown',
         'system_reset',
         'system_wakeup' ],
+    # Commands allowed to return a non-dictionary
     'command-returns-exceptions': [
         'human-monitor-command',
         'qom-get',
-- 
2.35.1

Re: [PATCH] qapi/pragma: Tidy up comments

Posted by Markus Armbruster 1 year, 11 months ago

Markus Armbruster <armbru@redhat.com> writes:

> Commit 05ebf841ef "qapi: Enforce command naming rules" inserted new
> code between a comment and the code it applies to.  Move the comment
> back to its code, and add one for the new code.
>
> Signed-off-by: Markus Armbruster <armbru@redhat.com>

Queued.