Re: [PATCH v4 2/3] config: add 'type' to config_source struct that identifies config type

From: Lars Schneider <larsxschneider@gmail.com>
Date: Feb 18, 2016, 08:46 UTC
Message-ID: <46CB7F8C-2A0C-41E1-A6B7-C26448CAE052@gmail.com>
In-Reply-To: <alpine.DEB.2.20.1602171412080.6516@virtualbox>

On 17 Feb 2016, at 14:12, Johannes Schindelin <johannes.schindelin@gmx.de> wrote:

Show 11 quoted lines

> Hi Lars,
> 
> On Mon, 15 Feb 2016, larsxschneider@gmail.com wrote:
> 
>> From: Lars Schneider <larsxschneider@gmail.com>
>> 
>> Use the config type to print more detailed error messages that inform
>> the user about the origin of a config error (file, stdin, blob).
> 
> Given that you settled on `--show-origin` as the command-line option, I
> would have expected s/type/origin/g, too...

Agreed. Junio suggested "origin_type" which I like, too.

Thanks, Lars