> On 07 Mar 2018, at 19:04, Eric Sunshine <[email protected]> wrote:
>
> On Wed, Mar 7, 2018 at 12:30 PM, <[email protected]> wrote:
>> Check that new content is valid with respect to the user defined
>> 'working-tree-encoding' attribute.
>>
>> Signed-off-by: Lars Schneider <[email protected]>
>> ---
>> diff --git a/convert.c b/convert.c
>> @@ -266,6 +266,58 @@ static int will_convert_lf_to_crlf(size_t len, struct
>> text_stat *stats,
>> +static int validate_encoding(const char *path, const char *enc,
>> + const char *data, size_t len, int die_on_error)
>> +{
>> + /* We only check for UTF here as UTF?? can be an alias for UTF-?? */
>> + if (startscase_with(enc, "UTF")) {
>> + /*
>> + * Check for detectable errors in UTF encodings
>> + */
>> + if (has_prohibited_utf_bom(enc, data, len)) {
>> + const char *error_msg = _(
>> + "BOM is prohibited in '%s' if encoded as
>> %s");
>> + /*
>> + * This advice is shown for UTF-??BE and UTF-??LE
>> encodings.
>> + * We cut off the last two characters of the
>> encoding name
>> + # to generate the encoding name suitable for BOMs.
>
> s/#/*/
Of course!
>> diff --git a/t/t0028-working-tree-encoding.sh
>> b/t/t0028-working-tree-encoding.sh
>> @@ -62,6 +62,46 @@ test_expect_success 'check $GIT_DIR/info/attributes
>> support' '
>> for i in 16 32
>> do
>> + test_expect_success "check prohibited UTF-${i} BOM" '
>> + test_when_finished "git reset --hard HEAD" &&
>> +
>> + echo "*.utf${i}be text working-tree-encoding=utf-${i}be"
>> >>.gitattributes &&
>> + echo "*.utf${i}le text working-tree-encoding=utf-${i}le"
>> >>.gitattributes &&
>
> v10 is checking only hyphenated lowercase encoding name; earlier
> versions checked uppercase. For better coverage, it would be nice to
> check several combinations: all uppercase, all lowercase, mixed case,
> hyphenated, not hyphenated.
>
> I'm not suggesting running all the tests repeatedly but rather just
> varying the format of the encoding name in these tests you're adding.
> For instance, the above could instead be:
>
> echo "*.utf${i}be text working-tree-encoding=UTF-${i}be" >>.gitattributes
> &&
> echo "*.utf${i}le text working-tree-encoding=utf${i}LE" >>.gitattributes &&
>
> or something.
The casing is a good idea - I will do that. I don't want to do "hyphenated, not
hyphenated" as this would make the tests fail on macOS (and I believe on
Windows).
Thanks,
Lars
