[PATCH v2 03/19] qapi/block-core: Drop temporary 'prefix'

Markus Armbruster posted 19 patches 2 months, 2 weeks ago
[PATCH v2 03/19] qapi/block-core: Drop temporary 'prefix'
Posted by Markus Armbruster 2 months, 2 weeks ago
Recent commit "qapi: Smarter camel_to_upper() to reduce need for
'prefix'" added a temporary 'prefix' to delay changing the generated
code.

Revert it.  This improves XDbgBlockGraphNodeType's generated
enumeration constant prefix from
X_DBG_BLOCK_GRAPH_NODE_TYPE_BLOCK_BACKEND to
XDBG_BLOCK_GRAPH_NODE_TYPE_BLOCK_BACKEND.

Signed-off-by: Markus Armbruster <armbru@redhat.com>
---
 qapi/block-core.json | 1 -
 block.c              | 6 +++---
 2 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/qapi/block-core.json b/qapi/block-core.json
index 452047102a..a7ae2da47b 100644
--- a/qapi/block-core.json
+++ b/qapi/block-core.json
@@ -2011,7 +2011,6 @@
 # Since: 4.0
 ##
 { 'enum': 'XDbgBlockGraphNodeType',
-  'prefix': 'X_DBG_BLOCK_GRAPH_NODE_TYPE', # TODO drop
   'data': [ 'block-backend', 'block-job', 'block-driver' ] }
 
 ##
diff --git a/block.c b/block.c
index c317de9eaa..7d90007cae 100644
--- a/block.c
+++ b/block.c
@@ -6351,7 +6351,7 @@ XDbgBlockGraph *bdrv_get_xdbg_block_graph(Error **errp)
         if (!*name) {
             name = allocated_name = blk_get_attached_dev_id(blk);
         }
-        xdbg_graph_add_node(gr, blk, X_DBG_BLOCK_GRAPH_NODE_TYPE_BLOCK_BACKEND,
+        xdbg_graph_add_node(gr, blk, XDBG_BLOCK_GRAPH_NODE_TYPE_BLOCK_BACKEND,
                            name);
         g_free(allocated_name);
         if (blk_root(blk)) {
@@ -6364,7 +6364,7 @@ XDbgBlockGraph *bdrv_get_xdbg_block_graph(Error **errp)
              job = block_job_next_locked(job)) {
             GSList *el;
 
-            xdbg_graph_add_node(gr, job, X_DBG_BLOCK_GRAPH_NODE_TYPE_BLOCK_JOB,
+            xdbg_graph_add_node(gr, job, XDBG_BLOCK_GRAPH_NODE_TYPE_BLOCK_JOB,
                                 job->job.id);
             for (el = job->nodes; el; el = el->next) {
                 xdbg_graph_add_edge(gr, job, (BdrvChild *)el->data);
@@ -6373,7 +6373,7 @@ XDbgBlockGraph *bdrv_get_xdbg_block_graph(Error **errp)
     }
 
     QTAILQ_FOREACH(bs, &graph_bdrv_states, node_list) {
-        xdbg_graph_add_node(gr, bs, X_DBG_BLOCK_GRAPH_NODE_TYPE_BLOCK_DRIVER,
+        xdbg_graph_add_node(gr, bs, XDBG_BLOCK_GRAPH_NODE_TYPE_BLOCK_DRIVER,
                            bs->node_name);
         QLIST_FOREACH(child, &bs->children, next) {
             xdbg_graph_add_edge(gr, bs, child);
-- 
2.46.0
Re: [PATCH v2 03/19] qapi/block-core: Drop temporary 'prefix'
Posted by Vladimir Sementsov-Ogievskiy 1 month, 3 weeks ago
On 04.09.24 14:18, Markus Armbruster wrote:
> Recent commit "qapi: Smarter camel_to_upper() to reduce need for
> 'prefix'" added a temporary 'prefix' to delay changing the generated
> code.
> 
> Revert it.  This improves XDbgBlockGraphNodeType's generated
> enumeration constant prefix from
> X_DBG_BLOCK_GRAPH_NODE_TYPE_BLOCK_BACKEND to
> XDBG_BLOCK_GRAPH_NODE_TYPE_BLOCK_BACKEND.
> 
> Signed-off-by: Markus Armbruster<armbru@redhat.com>

Reviewed-by: Vladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru>

-- 
Best regards,
Vladimir
Re: [PATCH v2 03/19] qapi/block-core: Drop temporary 'prefix'
Posted by Vladimir Sementsov-Ogievskiy 1 month, 3 weeks ago
On 30.09.24 16:23, Vladimir Sementsov-Ogievskiy wrote:
> On 04.09.24 14:18, Markus Armbruster wrote:
>> Recent commit "qapi: Smarter camel_to_upper() to reduce need for
>> 'prefix'" added a temporary 'prefix' to delay changing the generated
>> code.
>>
>> Revert it.  This improves XDbgBlockGraphNodeType's generated
>> enumeration constant prefix from
>> X_DBG_BLOCK_GRAPH_NODE_TYPE_BLOCK_BACKEND to
>> XDBG_BLOCK_GRAPH_NODE_TYPE_BLOCK_BACKEND.
>>
>> Signed-off-by: Markus Armbruster<armbru@redhat.com>
> 
> Reviewed-by: Vladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru>
> 

Oops sorry, I see now that was merged already.

-- 
Best regards,
Vladimir