[PATCH] Improve #pragma comment(lib) support
Aaron Ballman
aaron at aaronballman.com
Thu May 23 14:07:06 PDT 2013
On Thu, May 23, 2013 at 5:05 PM, Aaron Ballman <aaron at aaronballman.com> wrote:
> On Thu, May 23, 2013 at 5:02 PM, Rui Ueyama <ruiu at google.com> wrote:
>> - Consumer.HandleDependentLibrary(Arg);
>> + // If the argument does not end in .lib, automatically add the suffix.
>> This
>> + // matches the behavior of MSVC.
>> + std::string ArgStr = Arg;
>> + StringRef End = Arg.substr( Arg.size() - 4 );
>>
>> Looks like it's not safe if Arg.size() < 4.
>
> Hmm, that's a good point due to the unsigned subtraction. Easy enough
> to resolve.
Also, just caught the formatting issue with the extra spaces; also
easy enough to resolve.
~Aaron
More information about the cfe-commits
mailing list