dwarf.c small botch

Dmitry Timoshkov dmitry at codeweavers.com
Tue Oct 24 23:51:55 CDT 2006


"David Anderson" <davea42 at earthlink.net> wrote:

> @@ -1095,7 +1095,7 @@ static unsigned dwarf2_map_register(int 
> 
>     switch (regno)
>     {
> -    case Wine_DW_no_register: FIXME("What the heck\n"); reg = 0; break;
> +    case Wine_DW_no_register: FIXME("What the heck map reg 0x%x\n",(unsigned int)regno); reg = 0; break;

There is no need for a cast here.

>     /* FIXME: this is a dirty hack */
>     case Wine_DW_frame_register: reg = 0;          break;
>     case  0: reg = CV_REG_EAX; break;
> @@ -1394,8 +1394,16 @@ static struct symt* dwarf2_parse_subprog
> 
>     subpgm.ctx = ctx;
>     subpgm.compiland = compiland;
> -    if (dwarf2_compute_location(ctx, di, DW_AT_frame_base, &subpgm.frame_offset, &subpgm.frame_reg))
> +    if (dwarf2_compute_location(ctx, di, DW_AT_frame_base, &subpgm.frame_offset, &subpgm.frame_reg)) {
>         TRACE("For %s got %ld/%d\n", name.u.string, subpgm.frame_offset, subpgm.frame_reg);
> + if (subpgm.frame_reg == Wine_DW_no_register) {
> +    /* Likely a constant, meaning a location list offset.
> +       We do not handle those yet. */
> +           /*FIXME("need to handle location lists\n"); */
> +           subpgm.frame_reg = 0;
> +           subpgm.frame_offset = 0;
> + }
> +    }

Please follow the formatting style of the code you're changing: in placing braces,
using spaces instead of tabs, formatting of the comments.

-- 
Dmitry.



More information about the wine-devel mailing list