diff options
| author | Pat Gavlin <pagavlin@microsoft.com> | 2015-02-21 10:23:28 -0800 |
|---|---|---|
| committer | Pat Gavlin <pagavlin@microsoft.com> | 2015-02-21 15:37:57 -0800 |
| commit | f15d3cd41552da6187a2b7898071ed27ff900a5c (patch) | |
| tree | f59affc71aaa127b1855f5bf567053bf0bd41bce /.gitignore | |
| parent | 78227eab0cb818b14d74dc8096631ada729574db (diff) | |
| download | coreclr-f15d3cd41552da6187a2b7898071ed27ff900a5c.tar.gz coreclr-f15d3cd41552da6187a2b7898071ed27ff900a5c.tar.bz2 coreclr-f15d3cd41552da6187a2b7898071ed27ff900a5c.zip | |
Fix warnings generated when building with Clang on Linux.
There were three classes of warnings that needed to be fixed:
1) Warnings for unused typedefs
2) Warnings for pointer to boolena conversions that were always true
3) Warnings for taking the absolute value of unsigned values
(1) accounted for the majority of the warnings. All of these warnings
originated from macro definitions debugreturn.h, staticcontract.h,
or pal_assert.h. In the first two cases, typedefs were used as
compile-time checks to ensure certain invariants inside of method or
function bodies. These cases were addressed by using an empty member
of the typedef under an "if (0)" guard. In the third case, typedefs
were used for compile-time assertions that may appear in any location
in a source file; these have been updated to use C++11's static_assert.
(2) and (3) were much more straightforward: the former was addressed by
explicitly comparing the source pointer against NULL and the latter by
removing the offending calls to abs().
This change also contains a small update to .gitignore to ignore the
binaries directory and CMake artifacts created by build.sh.
Diffstat (limited to '.gitignore')
| -rw-r--r-- | .gitignore | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/.gitignore b/.gitignore index 83808394eb..e69eb76426 100644 --- a/.gitignore +++ b/.gitignore @@ -22,6 +22,7 @@ bld/ [Bb]in/ [Oo]bj/ msbuild.log +binaries # Roslyn stuff *.sln.ide @@ -254,3 +255,7 @@ TestWrappers_x64_release Vagrantfile .vagrant +# CMake files +CMakeFiles/ +cmake_install.cmake +CMakeCache.txt |