2012/1/18 Jason Merrill <ja...@redhat.com>: > I still think the problem is that we're applying the attributes more than > once, and we should only apply them once. If we fix that, we don't need to > merge them. > > Jason
Well, I will try to search for it a bit more. Nevertheless is the chaining at this place IMHO a pretty bad idea. The merging of attributes looks more sane. Also is this merging patch pretty less invasive and would be fine for older branches, too. The issue I see by changing here do_friend and calls to attribute-modifier is, that it easily causes side-effects, which are escaping underdected for awhile. Kai