[haiku-bugs] Re: [Haiku] #8135: Support debug info in separate file
- From: "anevilyak" <trac@xxxxxxxxxxxx>
- Date: Sun, 29 Jul 2012 11:15:20 -0000
#8135: Support debug info in separate file
-------------------------------------+----------------------------
Reporter: bonefish | Owner: anevilyak
Type: enhancement | Status: closed
Priority: low | Milestone: R1
Component: Applications/Debugger | Version: R1/Development
Resolution: fixed | Keywords:
Blocked By: | Blocking:
Has a Patch: 1 | Platform: All
-------------------------------------+----------------------------
Changes (by anevilyak):
* status: in-progress => closed
* resolution: => fixed
Comment:
Replying to [comment:17 bonefish]:
> The `objcopy` bug might be worth a bug report. :-)
>
Will do.
> Regarding your first caveat, if you used Haiku's build system to build
the release and the debug version of the file, problems are to be
expected, since one is built with and the other without optimization. In
the Linux world usually the release versions are built with both
optimization and debug info (and where possible, the debug info is split
into another file/package). The result is not optimal for debugging -- due
to the optimization inlining and reordering code -- but after all it is a
release build.
That was it indeed, my mistake.
> Regarding your patch:
> * Not important, but I would swap the `_GetDebugInfoPath()` parameters.
Output parameters usually come last (though there are exceptions when
there are other parameters with default values).
> * I don't know why you made the `dirLocations` array `int16`. It should
be `directory_which`. Also the `-1` element is not needed. The number of
elements is `sizeof(dirLocations) / sizeof(dirLocations[0])`. Moreover the
array could be const.
Adjustments made and checked in in hrev44428. Thanks!
--
Ticket URL: <http://dev.haiku-os.org/ticket/8135#comment:18>
Haiku <http://dev.haiku-os.org>
Haiku - the operating system.
Other related posts:
