Re: [PATCH 0/1] MR7879: dbghelp: Bail out when encountering managed symbols.

[eric pouech (＠epo)]

eric pouech (@epo) commented about dlls/dbghelp/msc.c:
>              FIXME("Unexpected S_FRAMEPROC %d (%p %p) %x\n", top_frame_size, top_func, curr_func, i);
>          break;
>
> -
>    /* bail out when encountering managed debug symbols, in order to avoid asserts later */
> -
>    case S_TOKENREF:
> -
>    case S_GMANPROC:
> -
>    case S_LMANPROC:
> -
>        ERR("Unsupported mamaged symbol id %x, stopping pdb parsing\n", sym->generic.id);
> -
>        return FALSE;

the issue of returning FALSE is that it will also drop all codeview symbols following that entry.

isn't moving the 3 codeview symbols in the block of 'the symbols we can safely ignore for now' sufficient ?