Re: [PATCH 04/17] add_to_alternates_file: don't add duplicate entries

2015-08-11 Thread Jeff King
On Tue, Aug 11, 2015 at 06:00:20AM +0200, Michael Haggerty wrote: Instead of using hold_lock_file_for_append, let's copy the file line by line, which ensures all records are properly terminated. If we see an extra line, we can simply abort the update (there is no point in even copying the

Re: [PATCH 04/17] add_to_alternates_file: don't add duplicate entries

2015-08-10 Thread Michael Haggerty
On 08/10/2015 11:34 AM, Jeff King wrote: The add_to_alternates_file function blindly uses hold_lock_file_for_append to copy the existing contents, and then adds the new line to it. This has two minor problems: 1. We might add duplicate entries, which are ugly and inefficient. 2.

[PATCH 04/17] add_to_alternates_file: don't add duplicate entries

2015-08-10 Thread Jeff King
The add_to_alternates_file function blindly uses hold_lock_file_for_append to copy the existing contents, and then adds the new line to it. This has two minor problems: 1. We might add duplicate entries, which are ugly and inefficient. 2. We do not check that the file ends with a