Re: [PATCH 4/5] diff: use SWAP macro

2017-01-30 Thread Junio C Hamano
René Scharfe writes: > Use the macro SWAP to exchange the value of pairs of variables instead > of swapping them manually with the help of a temporary variable. The > resulting code is shorter and easier to read. > > The two cases were not transformed by the semantic patch swap.cocci > because i

Re: [PATCH 4/5] diff: use SWAP macro

2017-01-30 Thread René Scharfe
Am 30.01.2017 um 17:04 schrieb Johannes Schindelin: Hi René, On Sat, 28 Jan 2017, René Scharfe wrote: Use the macro SWAP to exchange the value of pairs of variables instead of swapping them manually with the help of a temporary variable. The resulting code is shorter and easier to read. The

Re: [PATCH 4/5] diff: use SWAP macro

2017-01-30 Thread Johannes Schindelin
Hi René, On Sat, 28 Jan 2017, René Scharfe wrote: > Use the macro SWAP to exchange the value of pairs of variables instead > of swapping them manually with the help of a temporary variable. The > resulting code is shorter and easier to read. > > The two cases were not transformed by the semanti

[PATCH 4/5] diff: use SWAP macro

2017-01-28 Thread René Scharfe
Use the macro SWAP to exchange the value of pairs of variables instead of swapping them manually with the help of a temporary variable. The resulting code is shorter and easier to read. The two cases were not transformed by the semantic patch swap.cocci because it's extra careful and handles only