Re: [PATCH v1 4/4] hashmap: add string interning API

From: Matthieu Moy <matthieu.moy@grenoble-inp.fr>
Date: Jul 3, 2014, 07:22 UTC
Message-ID: <vpqpphmc3em.fsf@anie.imag.fr>
In-Reply-To: <53B4863E.1020701@gmail.com>

Karsten Blees <karsten.blees@gmail.com> writes:

Show 18 quoted lines

> --- a/t/t0011-hashmap.sh
> +++ b/t/t0011-hashmap.sh
> @@ -237,4 +237,17 @@ test_expect_success 'grow / shrink' '
>  
>  '
>  
> +test_expect_success 'string interning' '
> +
> +test_hashmap "intern value1
> +intern Value1
> +intern value2
> +intern value2
> +" "value1
> +Value1
> +value2
> +value2"
> +
> +'

Indentation is broken here, but it's consistant with the current style of the file. You may want a "modernize style" patch before yours and then indent properly, but that shouldn't block inclusion.

-- 
Matthieu Moy
http://www-verimag.imag.fr/~moy/