[cfe-dev] Making clang::Rewriter friendlier for subclassing

Eli Bendersky eliben at google.com
Thu May 14 19:55:32 PDT 2015

On Thu, May 14, 2015 at 7:13 PM, Manasij Mukherjee <manasij7479 at gmail.com>

> If you mean to call the base methods anyway, wouldn't it be better to have
> a Rewriter object as a member of your class ?

It is certainly possible to wrap Rewriter in another class. But this would
mean the code needs to:

1. Replace Rewriter* everywhere with the new class
2. Implement all of Rewriter's methods to call base methods + additional

This is quite cumbersome. Making some methods of Rewriter virtual, however,
would mean (1) is not needed and in (2) we only need to override methods we


> On Thu, May 14, 2015 at 12:20 PM, Eli Bendersky <eliben at google.com> wrote:
>> Hi,
>> Would there be objections to make clang::Rewriter a bit friendlier for
>> subclassing by marking some of its methods virtual? In particular
>> InsertText, ReplaceText, RemoveText and their variations.
>> The reason for this is to be able to subclass Rewriter for the purposes
>> of logging, debugging and other custom capabilities.
>> If this sounds reasonable, I'll be happy to implement this.
>> Thanks in advance,
>> Eli
>> _______________________________________________
>> cfe-dev mailing list
>> cfe-dev at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20150514/27fa0f00/attachment.html>

More information about the cfe-dev mailing list