Bai, those code paths have been fine tuned based on my limited
understanding of the gdb type information layout. You cannot use a 'fall
through' in this switch. It's a one to one match test.
What's behind your question? Are you having problems with a script
and/or symbol/type resolution? If so, send us/me a test case.
Luc
On Thu, 2008-01-10 at 17:01 +0800, baiwd wrote:
> Hi:
> I found some codes like
> case TYPE_CODE_ENUM: if(sial_is_enum(ctype)) goto match; break;
> I wonder that whether it would be better if these codes could be
> changed as
> following:
>
> Signed-off-by: Bai Weidong <baiwd@xxxxxxxxxxxxxx>
>
> --- crash-4.0-4.12/extensions/sial.c 2007-12-13 02:48:09.000000000 +0800
> +++ crash-4.0-4.12/extensions/sial.new.c 2008-01-10 16:47:15.000000000 +0800
> @@ -155,10 +155,10 @@ apigetctype(int ctype, char *name, TYPE_
> if(sial_is_typedef(ctype) && v) goto match;
> switch(TYPE_CODE(type)) {
> case TYPE_CODE_TYPEDEF: case TYPE_CODE_INT:
> - if(sial_is_typedef(ctype)) goto match; break;
> - case TYPE_CODE_ENUM: if(sial_is_enum(ctype)) goto match; break;
> - case TYPE_CODE_STRUCT: if(sial_is_struct(ctype)) goto match; break;
> - case TYPE_CODE_UNION: if(sial_is_union(ctype)) goto match; break;
> + if(sial_is_typedef(ctype)) goto match;
> + case TYPE_CODE_ENUM: if(sial_is_enum(ctype)) goto match;
> + case TYPE_CODE_STRUCT: if(sial_is_struct(ctype)) goto match;
> + case TYPE_CODE_UNION: if(sial_is_union(ctype)) goto match;
> }
> sial_dbg_named(DBG_TYPE, name, 2, "Found but no match.\n");
> }
>
> Best Regards
> Bai Weidong
>
> --
> Crash-utility mailing list
> Crash-utility@xxxxxxxxxx
> https://www.redhat.com/mailman/listinfo/crash-utility
>
Confidentiality Notice
This e-mail (including any attachments) is intended only for the recipients named above. It may contain confidential or privileged information and should not be read, copied or otherwise used by any other person. If you are not a named recipient, please notify the sender of that fact and delete the e-mail from your system.
--
Crash-utility mailing list
Crash-utility@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/crash-utility
