[cfe-commits] r49066 - in /cfe/trunk/lib/Sema: Sema.h SemaDecl.cpp

[Steve Naroff]

Author: snaroff
Date: Tue Apr  1 19:39:51 2008
New Revision: 49066

URL: http://llvm.org/viewvc/llvm-project?rev=49066&view=rev
Log:
Fix a comment typo and add a couple suggestions from Chris.

Modified:
    cfe/trunk/lib/Sema/Sema.h
    cfe/trunk/lib/Sema/SemaDecl.cpp

Modified: cfe/trunk/lib/Sema/Sema.h
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/Sema.h?rev=49066&r1=49065&r2=49066&view=diff

==============================================================================
--- cfe/trunk/lib/Sema/Sema.h (original)
+++ cfe/trunk/lib/Sema/Sema.h Tue Apr  1 19:39:51 2008
@@ -99,7 +99,8 @@
   /// with @compatibility_alias, so that we can emit errors on duplicates and
   /// find the declarations when needed. This construct is ancient and will
   /// likely never be seen. Nevertheless, it is here for compatibility.
-  typedef llvm::DenseMap<IdentifierInfo*, ObjCCompatibleAliasDecl*> ObjCAliasTy;
+  typedef llvm::DenseMap<const IdentifierInfo*, 
+                         ObjCCompatibleAliasDecl*> ObjCAliasTy;
   ObjCAliasTy ObjCAliasDecls;
   
   // Enum values used by KnownFunctionIDs (see below).

Modified: cfe/trunk/lib/Sema/SemaDecl.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/SemaDecl.cpp?rev=49066&r1=49065&r2=49066&view=diff

==============================================================================
--- cfe/trunk/lib/Sema/SemaDecl.cpp (original)
+++ cfe/trunk/lib/Sema/SemaDecl.cpp Tue Apr  1 19:39:51 2008
@@ -42,7 +42,7 @@
     if (getLangOptions().ObjC1) {
       // @interface and @compatibility_alias result in new type references. 
       // Creating a class alias is *extremely* rare.
-      ObjCAliasTy::const_iterator I = ObjCAliasDecls.find((IdentifierInfo*)&II);
+      ObjCAliasTy::const_iterator I = ObjCAliasDecls.find(&II);
       if (I != ObjCAliasDecls.end())
         return I->second->getClassInterface();
     }    
@@ -141,7 +141,7 @@
     if (getLangOptions().ObjC1) {
       // @interface and @compatibility_alias introduce typedef-like names.
       // Unlike typedef's, they can only be introduced at file-scope (and are 
-      // not therefore not scoped decls). They can, however, be shadowed by
+      // therefore not scoped decls). They can, however, be shadowed by
       // other names in IDNS_Ordinary.
       ObjCAliasTy::iterator I = ObjCAliasDecls.find(II);
       if (I != ObjCAliasDecls.end())