Thread (24 messages) flat view 24 messages, 3 authors, 2016-07-10

Re: [PATCH 0/8] write_file cleanups

From: René Scharfe <hidden>
Date: 2016-07-08 18:44:55

Am 08.07.2016 um 11:04 schrieb Jeff King:
Here it is. There actually weren't that many spots to clean up, as quite
a few of them have a "twist" where they want to do something clever,
like open the file and feed the descriptor to a sub-function, or open
with funny things like O_EXCL.

But still, the diffstat is pleasing:

  builtin/am.c     | 25 +++++++----------
  builtin/branch.c |  5 +---
  builtin/config.c |  2 +-
  builtin/merge.c  | 45 ++++--------------------------
  cache.h          | 17 ++++++++++--
  wrapper.c        | 52 ++++++++---------------------------
  6 files changed, 44 insertions(+), 102 deletions(-)

and that even includes adding some function documentation.
Haha, feels like Candy Crush. :)

(Sent a single small patch, lots of places get patched as if by magic.)

Thanks,
René
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help