Hi Sukender,

On Wed, Apr 20, 2011 at 11:27 PM, Sukender <[email protected]> wrote:
> Well I moved the PathIterator to public also for the PluginImageWriter 
> (another submission).

I haven't review PLuginImageWriter yet, should get to this today.

> Don't you think list<std::string> is a bit a burden (I mean to copy a path 
> into small strings istead of just iterating over the original string)? My 
> "models" for this were actually boost.filesystem and C++0x.

Performance when loading files is not something that will be affected
by a small copy of strings as it's the IO that it bottleneck.  My main
concern is keeping public API simple and implementation details
hidden, so if we need to change the implementation it doesn't effect
other places.

Robert.
_______________________________________________
osg-submissions mailing list
[email protected]
http://lists.openscenegraph.org/listinfo.cgi/osg-submissions-openscenegraph.org

Reply via email to