[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