[PATCH] Add dll trampoline code handling for windows 64bit
Stan Shebs
stanshebs@earthlink.net
Wed Mar 14 23:54:00 GMT 2012
On 3/14/12 2:03 PM, Joel Brobecker wrote:
>> There is a script called gdb_indent.sh in gdb's root dir. I applied
>> it on amd64-windows-nat.c ahead of getting out my patch thinking
>> that this would be the correct way. Some misunderstanding as it
>> produced most of your indention concerns.
> That is very odd, the script should indeed indent the sources
> in a way that follows the GNU Coding Standards. Perhaps we are
> missing some parameters in the call to that script that override
> certain defaults that may no longer match our standards. But FWIW,
> we have rarely used that script in the past few years. I think
> that this is because GNU indent, the tool that it uses underneath,
> isn't very smart and often produces a layout that's worse than
> what a human would produce.
>
I've never been able to get indent to produce exactly the same results
as Emacs' indent, even though in theory they are using the same
algorithm with the same parameters.
Probably the best recommendation for contributors is to use Emacs
indent-region on their new code fragments, and to leave everything else
in the file alone.
Stan
More information about the Gdb-patches
mailing list