On Tue, Oct 15, 2013 at 8:45 AM, Will Dietz <[email protected]> wrote:
> Author: wdietz2 > Date: Tue Oct 15 10:45:00 2013 > New Revision: 192713 > > URL: http://llvm.org/viewvc/llvm-project?rev=192713&view=rev > Log: > ModuleAssistant: Fix warning, don't return bool as a pointer. > Was this caught by a warning? If not, Clang should perhaps grow one - last I checked I don't think it caught this. (though I'll grant you it might not find many 'bugs', but some amount of confusion) > > No functionality change intended. > > Modified: > clang-tools-extra/trunk/modularize/ModuleAssistant.cpp > > Modified: clang-tools-extra/trunk/modularize/ModuleAssistant.cpp > URL: > http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/modularize/ModuleAssistant.cpp?rev=192713&r1=192712&r2=192713&view=diff > > ============================================================================== > --- clang-tools-extra/trunk/modularize/ModuleAssistant.cpp (original) > +++ clang-tools-extra/trunk/modularize/ModuleAssistant.cpp Tue Oct 15 > 10:45:00 2013 > @@ -219,7 +219,7 @@ static Module *loadModuleDescriptions( > I != E; ++I) { > // Add as a module. > if (!addModuleDescription(RootModule, *I, HeaderPrefix, Dependencies)) > - return false; > + return NULL; > } > > return RootModule; > > > _______________________________________________ > cfe-commits mailing list > [email protected] > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits >
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
