Oiio generates numerous compilation warnings with Visual Studio. Trying to 
remove them all is a daunting task so I went for the middle ground and removed 
only those warnings present in the public oiio header files and in the command 
line tools. The remaining warnings, in the internals of the oiio library, can 
then be shut up conveniently by simply disabling all warnings in the library's 
.vcproj file.

The fixes I made are a mixture of actual code changes, where I thought that the 
code would be improved as a result of removing the warning, and insertions of 
simple msvc #pragma warning disable commands, in those cases where the warning 
is clearly harmless and there was nothing to gain by modifying the code to 
remove it.


You can merge this Pull Request by running:

  git pull https://github.com/mgmt1969/oiio master

Or you can view, comment on it, or merge it online at:

  https://github.com/OpenImageIO/oiio/pull/188

-- Commit Summary --

* Removed compilation warnings for the oiio tools.
* Removed unnecessary code from my previous commit.

-- File Changes --

M src/iconvert/iconvert.cpp (2)
M src/idiff/idiff.cpp (6)
M src/iinfo/iinfo.cpp (4)
M src/include/argparse.h (4)
M src/include/colortransfer.h (14)
M src/include/fmath.h (11)
M src/include/imagebuf.h (4)
M src/include/imagebufalgo.h (4)
M src/include/imageio.h (6)
M src/include/paramlist.h (4)
M src/include/refcnt.h (4)
M src/include/thread.h (3)
M src/include/typedesc.h (21)
M src/include/ustring.h (4)
M src/iprocess/iprocess.cpp (2)
M src/iv/imageviewer.cpp (2)
M src/iv/imageviewer.h (4)
M src/iv/ivgl.cpp (4)
M src/iv/ivmain.cpp (9)
M src/maketx/maketx.cpp (6)
M src/oiiotool/diff.cpp (2)
M src/oiiotool/oiiotool.cpp (24)
M src/oiiotool/oiiotool.h (4)
M src/oiiotool/printinfo.cpp (4)

-- Patch Links --

  https://github.com/OpenImageIO/oiio/pull/188.patch
  https://github.com/OpenImageIO/oiio/pull/188.diff

-- 
Reply to this email directly or view it on GitHub:
https://github.com/OpenImageIO/oiio/pull/188
_______________________________________________
Oiio-dev mailing list
[email protected]
http://lists.openimageio.org/listinfo.cgi/oiio-dev-openimageio.org

Reply via email to