[cfe-commits] clang formatter patch to move and rename RewriterTestContext.h
jahanian
fjahanian at apple.com
Wed Dec 12 10:00:39 PST 2012
On Dec 12, 2012, at 9:55 AM, Dmitri Gribenko <gribozavr at gmail.com> wrote:
> On Wed, Dec 12, 2012 at 7:48 PM, jahanian <fjahanian at apple.com> wrote:
>> On Dec 12, 2012, at 9:42 AM, Dmitri Gribenko <gribozavr at gmail.com> wrote:
>>> 2. There's a naming inconsistency: the class is called
>>> 'RewriterTestContext', while the header is called
>>> 'RewriterFormatContext.h'. I think it is important to have "Test" in
>>> the header name, so that it is obvious that this is not something to
>>> be used outside of tests.
>>
>> The opposite, RewriterFormatContext.h will be used outside of tests. Name of
>> the class inside RewriterFormatContext.h need be changed. Something that I planned
>> on doing later.
>
> Is this class good enough for use outside of tests? It just creates
> all dependencies of the Rewriter -- including, for example, a
> SourceManager -- so that the Rewriter works in tests. If you plan to
> change this and, e.g., pass a SourceManager from the outside, it would
> needlessly complicate tests.
I don't know enough about the formatter. This patch is an attempt to have a header in Format directory
that others can use. Any suggestion is welcome.
- fariborz
>
> 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