[Lldb-commits] [lldb] r246123 - Silence some MSVC warnings.
Stephane Sezer via lldb-commits
lldb-commits at lists.llvm.org
Wed Aug 26 16:55:14 PDT 2015
Author: sas
Date: Wed Aug 26 18:55:14 2015
New Revision: 246123
URL: http://llvm.org/viewvc/llvm-project?rev=246123&view=rev
Log:
Silence some MSVC warnings.
Summary:
Just `assert("string" && false)` instead of `assert("string" == NULL)`.
This avoid errors like
[...]\Core\SourceManager.cpp(647): warning C4130: '==' : logical operation on address of string constant
Reviewers: zturner
Subscribers: lldb-commits
Differential Revision: http://reviews.llvm.org/D12388
Modified:
lldb/trunk/source/Core/SourceManager.cpp
Modified: lldb/trunk/source/Core/SourceManager.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/source/Core/SourceManager.cpp?rev=246123&r1=246122&r2=246123&view=diff
==============================================================================
--- lldb/trunk/source/Core/SourceManager.cpp (original)
+++ lldb/trunk/source/Core/SourceManager.cpp Wed Aug 26 18:55:14 2015
@@ -644,14 +644,14 @@ SourceManager::File::CalculateLineOffset
else
{
// Some lines have been populated, start where we last left off
- assert("Not implemented yet" == NULL);
+ assert("Not implemented yet" && false);
}
}
else
{
// Calculate all line offsets up to "line"
- assert("Not implemented yet" == NULL);
+ assert("Not implemented yet" && false);
}
return false;
}
More information about the lldb-commits
mailing list