On Apr 9, 2013, at 1:47 PM, Douglas Gregor <[email protected]> wrote:
> > On Apr 9, 2013, at 10:52 AM, Fariborz Jahanian <[email protected]> wrote: > >> Author: fjahanian >> Date: Tue Apr 9 12:52:29 2013 >> New Revision: 179108 >> >> URL: http://llvm.org/viewvc/llvm-project?rev=179108&view=rev >> Log: >> Objective-C: This patch fixes a none-issuance of warning >> when result type of protocol property and getter method >> differ by fixing a more serious problem. When a forward >> protocol declaration comes between its definition and >> its use in class protocol list, the forward protocol >> ast was being used in building the protocol list. >> // rdar://12522752 > > Why is this a problem? The protocol list could have a reference to the > protocol declaration that was actually visible, and the callers that need > definitions should use getDefinition(). When we lookup the property in class’s protocol list, we hit forward declarations in the list. It is better to fix the list in one place rather than everywhere we need to look into the list. Does this cause a problem for modules? If so, I can move this to where property is looked up. But I think this is a better fix. - Fariborz
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
