Re: [PATCH v14 03/13] utf8: add function to align a string into given strbuf
- From
Torsten Bögershausen <tboegi@web.de>
- Date
- Aug 29, 2015, 17:10 UTC
- 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:
Show 13 quoted lines
> 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)