[PATCH] Support for double width characters.

Alexander Kornienko alexfh at google.com
Wed Aug 7 08:07:52 PDT 2013


Removing the space was needed to check for off-by-one errors from upper
side.
On Aug 7, 2013 5:04 PM, "Daniel Jasper" <djasper at google.com> wrote:

>
>   Looks good.
>
>
> ================
> Comment at: unittests/Format/FormatTest.cpp:5694
> @@ +5693,3 @@
> +            "\"十\"",
> +            format("\"一 二 三 四 五六 七 八 九 十\"",
> +                   getLLVMStyleWithColumns(11)));
> ----------------
> Why was removing the space necessary or what is that now supposed to show?
>
>
> http://llvm-reviews.chandlerc.com/D1311
>
> BRANCH
>   svn
>
> ARCANIST PROJECT
>   clang
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130807/53fac088/attachment.html>


More information about the cfe-commits mailing list