Markus Armbruster e2cd0f4fb4 qapi: Replace start_optional()/end_optional() by optional() ... Semantics of end_optional() differ subtly from the other end_FOO() callbacks: when start_FOO() succeeds, the matching end_FOO() gets called regardless of what happens in between. end_optional() gets called only when everything in between succeeds as well. Entirely undocumented, like all of the visitor API. The only user of Visitor Callback end_optional() never did anything, and was removed in commit 9f9ab46. I'm about to clean up error handling in the generated visitor code, and end_optional() is in my way. No users mean no test cases, and making non-trivial cleanup transformations without test cases doesn't strike me as a good idea. Drop end_optional(), and rename start_optional() to optional(). We can always go back to a pair of callbacks when we have an actual need. Signed-off-by: Markus Armbruster <armbru@redhat.com> Reviewed-by: Eric Blake <eblake@redhat.com> Signed-off-by: Luiz Capitulino <lcapitulino@redhat.com>		2014-05-15 14:00:45 -04:00
..
qmp	qmp hmp: Consistently name Error * objects err, and not errp	2014-05-08 14:20:00 -04:00
dealloc-visitor.h	qapi: move include files to include/qobject/	2012-12-19 08:31:31 +01:00
error.h	error: Consistently name Error ** objects errp, and not err	2014-05-09 09:11:30 -04:00
opts-visitor.h	OptsVisitor: don't try to flatten overlong integer ranges	2013-08-20 11:52:00 -04:00
qmp-input-visitor.h	qapi: move include files to include/qobject/	2012-12-19 08:31:31 +01:00
qmp-output-visitor.h	qapi: move include files to include/qobject/	2012-12-19 08:31:31 +01:00
string-input-visitor.h	qapi: move include files to include/qobject/	2012-12-19 08:31:31 +01:00
string-output-visitor.h	qapi: Add human mode to StringOutputVisitor	2014-02-14 21:12:03 +01:00
visitor-impl.h	qapi: Replace start_optional()/end_optional() by optional()	2014-05-15 14:00:45 -04:00
visitor.h	qapi: Replace start_optional()/end_optional() by optional()	2014-05-15 14:00:45 -04:00