[cfe-commits] [PATCH] clang/test/Index: 3 of tests require libxml2, following up to r166130.

NAKAMURA Takumi geek4civic at gmail.com
Thu Oct 18 05:55:32 PDT 2012


Dmitri, I'm ok for you to update!

2012/10/18 Dmitri Gribenko <gribozavr at gmail.com>:
> Hi Takumi,
>
> About the patch, adding the feature is not required here.  The correct
> fix is to remove the CommentXMLValid from CHECK lines.  (The test
> checks that there are no CommentXMLInvalid lines, so it is OK).
>
> Please tell me if you want me to make this change.
>
> On Thu, Oct 18, 2012 at 5:30 AM, NAKAMURA Takumi <geek4civic at gmail.com> wrote:
>> FIXME: Could we add the feature 'libxml2' or rename 'xmllint' to one?
>
> Given my suggestion above, is this still required?
>
> Dmitri
>
> --
> main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if
> (j){printf("%d\n",i);}}} /*Dmitri Gribenko <gribozavr at gmail.com>*/



More information about the cfe-commits mailing list