Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
* [PATCH 0/3] Some cleanups in linux-record.c
@ 2016-03-16 18:55 Andreas Arnez
  2016-03-16 18:56 ` [PATCH 2/3] linux-record.c: Fix whitespace issues Andreas Arnez
                   ` (2 more replies)
  0 siblings, 3 replies; 12+ messages in thread
From: Andreas Arnez @ 2016-03-16 18:55 UTC (permalink / raw)
  To: gdb-patches; +Cc: Yao Qi, Metzger, Markus T

This is a follow-up to this patch:

  https://sourceware.org/ml/gdb-patches/2016-03/msg00239.html

As pointed out by Markus Metzger, the added lines in that patch used
tabified indentation, whereas much of the existing code in the file used
spaces for indentation instead.  He suggested to "either fix it for the
entire function in a separate patch or preserve the broken indentation."
This series implements the first option.  While fixing the whitespace
issues I also found a bug and added a fix to this series as well.

Patch #1 fixes an inadvertent fall-through to the next case statement.
Patch #2 fixes all whitespace issues in linux-record.c.  Patch #3 is a new
version of the original patch above, with the following improvements:

* Consequently keep invocations of record_mem_at_reg to a single line if
  staying within the ("soft") limit of 74 chars.

* Remove braces in the handling for `readlink', since the block is not
  needed anymore.

* Use record_mem_at_reg at two further places that were overlooked before.


Andreas Arnez (3):
  linux-record: Fix bad fall-through for pipe/pipe2
  linux-record.c: Fix whitespace issues
  linux-record: Simplify with record_mem_at_reg()

 gdb/linux-record.c | 2226 +++++++++++++++++++++++-----------------------------
 1 file changed, 989 insertions(+), 1237 deletions(-)

-- 
2.5.0


^ permalink raw reply	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2016-03-17 12:51 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-03-16 18:55 [PATCH 0/3] Some cleanups in linux-record.c Andreas Arnez
2016-03-16 18:56 ` [PATCH 2/3] linux-record.c: Fix whitespace issues Andreas Arnez
2016-03-17  8:44   ` Yao Qi
2016-03-17  9:12     ` Andreas Arnez
2016-03-16 18:56 ` [PATCH 1/3] linux-record: Fix bad fall-through for pipe/pipe2 Andreas Arnez
2016-03-16 20:23   ` Marcin Kościelnicki
2016-03-17 12:51     ` Andreas Arnez
2016-03-17  8:42   ` Yao Qi
2016-03-17  9:11     ` Andreas Arnez
2016-03-16 18:57 ` [PATCH 3/3] linux-record: Simplify with record_mem_at_reg() Andreas Arnez
2016-03-17  8:45   ` Yao Qi
2016-03-17  9:11     ` Andreas Arnez

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox