[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH ] qemu-help: add category headlines
From: |
Marcel Apfelbaum |
Subject: |
Re: [Qemu-devel] [PATCH ] qemu-help: add category headlines |
Date: |
Wed, 28 Aug 2013 11:10:48 +0300 |
On Thu, 2013-08-22 at 15:48 +0300, Marcel Apfelbaum wrote:
> This patch follows Markus Armbruster suggestion:
>
> A possibly better way to group help by category: instead of adding
> categories to each line, add category headlines, like this:
>
> Controller/Bridge/Hub devices:
> name "NAME", bus "BUS"...
> ...
> USB devices:
> name "NAME", bus "BUS"...
> ...
> Storage devices:
> ...
>
> This way, showing devices with multiple categories once per category
> actually makes sense.
>
> Note that the "categories to each line" is kept for 2 reasons:
> 1. Preparation for multifunction devices
> 2. Ability to grep by category
ping
Marcel
>
> Signed-off-by: Marcel Apfelbaum <address@hidden>
> ---
> qdev-monitor.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/qdev-monitor.c b/qdev-monitor.c
> index 410cdcb..a7329b0 100644
> --- a/qdev-monitor.c
> +++ b/qdev-monitor.c
> @@ -156,6 +156,8 @@ static void qdev_print_category_devices(DeviceCategory
> category)
> DeviceClass *dc;
> GSList *list, *curr;
>
> + error_printf("%s devices:\n", qdev_category_get_name(category));
> +
> list = object_class_get_list(TYPE_DEVICE, false);
> for (curr = list; curr; curr = g_slist_next(curr)) {
> dc = (DeviceClass *)object_class_dynamic_cast(curr->data,
> TYPE_DEVICE);
Re: [Qemu-devel] [PATCH ] qemu-help: add category headlines,
Marcel Apfelbaum <=