Filesystem::file_extension() does not return the '.' in the extension string
whereas boost::file_system::extension() does. This little difference went
unnoticed in a previous commit. The result is that "maketx foo.tif" builds a
"foo..tx" file. The proposed change greatly simplifies the code and does the
right thing.
Notes for Larry:
* The filesystem.{cpp,h} changes compile with boost::filesystem v2 and v3.
* The replace_extension() method used by the wrapper is available in Boost 1.40
(and in prior versions)
You can merge this Pull Request by running:
git pull https://github.com/darkhorse64/oiio master
Or you can view, comment on it, or merge it online at:
https://github.com/OpenImageIO/oiio/pull/253
-- Commit Summary --
* Fix ill-formed default output name for maketx
-- File Changes --
M src/include/filesystem.h (5)
M src/libutil/filesystem.cpp (8)
M src/maketx/maketx.cpp (9)
-- Patch Links --
https://github.com/OpenImageIO/oiio/pull/253.patch
https://github.com/OpenImageIO/oiio/pull/253.diff
---
Reply to this email directly or view it on GitHub:
https://github.com/OpenImageIO/oiio/pull/253
_______________________________________________
Oiio-dev mailing list
[email protected]
http://lists.openimageio.org/listinfo.cgi/oiio-dev-openimageio.org
