From: Torsten Bögershausen <tboegi@web.de>
Date: Sat, 29 Aug 2015 17:10:16 GMT
Subject: Re: [PATCH v14 03/13] utf8: add function to align a string into given strbuf
Message-ID: <55E1E778.4030400@web.de>
In-Reply-To: <1440857537-13968-4-git-send-email-Karthik.188@gmail.com>

On 29.08.15 16:12, Karthik Nayak wrote:
> diff --git a/utf8.h b/utf8.h
> index 5a9e94b..7930b44 100644
> --- a/utf8.h
> +++ b/utf8.h
> @@ -55,4 +55,19 @@ int mbs_chrlen(const char **text, size_t *remainder_p, const char *encoding);
>   */
>  int is_hfs_dotgit(const char *path);
>  
> +typedef enum {
> +	ALIGN_LEFT,
> +	ALIGN_MIDDLE,
> +	ALIGN_RIGHT
> +} align_type;
should this be called strbuf_align_type ?

And is there a reason why the is in utf.c and not in stbuf.c ?

(I know that there is a lot of strbuf in utf8.c, but I hadn't managed to send a patch
to move everything into strbuf.c and make utf8.c un-aware of all strbub-business)