[cfe-dev] Making clang::Rewriter friendlier for subclassing
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:
>> 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,
>> cfe-dev mailing list
>> cfe-dev at cs.uiuc.edu
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the cfe-dev