[PATCH] Fix DW_CFA_restore_extended parsing

Luis Machado luisgpm@linux.vnet.ibm.com
Fri Dec 21 14:02:00 GMT 2007


On Thu, 2007-12-20 at 14:03 -0800, Jim Blandy wrote:
> On Dec 20, 2007 12:22 PM, Luis Machado <luisgpm@linux.vnet.ibm.com> wrote:
> > This simple patch fixes the issue. Any thoughts? Ok to commit?
> 
> The only difference between those two opcodes is in their encoding ---
> the action to be performed is identical.  It seems silly to duplicate
> the code.  Could you pull the common code out into its own function,
> and then have both cases simply parse their arguments and call the
> function?

Like so?


-- 
Luis Machado
Software Engineer 
IBM Linux Technology Center
-------------- next part --------------
A non-text attachment was scrubbed...
Name: dwarf-fde-parse.diff
Type: text/x-patch
Size: 3088 bytes
Desc: not available
URL: <http://sourceware.org/pipermail/gdb-patches/attachments/20071221/b3efd473/attachment.bin>


More information about the Gdb-patches mailing list