The formatting I intended in the summary did not take affect.  Here is the 
referenced code as I intended it to display:

See this comment in include/clang/AST/DeclObjC.h:

  class ObjCCategoryImplDecl : public ObjCImplDecl {
  ...
  /// getIdentifier - Get the identifier that names the category
  /// interface associated with this implementation.
  /// FIXME: This is a bad API, we are hiding NamedDecl::getIdentifier()
  /// with a different meaning. For example:
  /// ((NamedDecl *)SomeCategoryImplDecl)->getIdentifier()
  /// returns the class interface name, whereas
  /// ((ObjCCategoryImplDecl *)SomeCategoryImplDecl)->getIdentifier()
  /// returns the category name.
  IdentifierInfo *getIdentifier() const {
    return Id;
  }

I did not change the interface, but clearly, it would be a good idea to do so. 
The patch uses the existing downcasted pointer to call the correct version.


REPOSITORY
  rL LLVM

http://reviews.llvm.org/D9127

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/



_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to