Re: [PATCHv6 06/23] notes.h/c: Propagate combine_notes_fn return value to add_note() and beyond

Thiago Farina wrote:

quoted

On Tue, Nov 9, 2010 at 7:49 PM, Johan Herland [off-list ref] wrote:

quoted

              die("confused: combine_notes_overwrite failed");

This message looks like more a debug information. Could it be
rewritten to be more user friendly / more informative?

Sverre mentioned the same thing.  The intent is the same as the existing

		die("should not happen, someone must be hit on the forehead");

--- i.e., to provoke bug reports if it happens, without the appearance

of begging. :)

So how about:

		die("BUG: combine_notes_overwrite failed");

Something involving assert() could also work, but that tends to be
a bit ugly.