search

certik / yaml-cpp

Automatically exported from code.google.com/p/yaml-cpp
MIT License
0 stars 0 forks source link

Remove boost dependency #205

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
Make boost a hard dependency only for shared_ptr is a bad idea. Please use 
std::tr1::shared_ptr as default. And make the use of boost optional. I try to 
avoid boost as much as possible, and others do too.

Original issue reported on code.google.com by p...@geromueller.de on 21 May 2013 at 1:52

GoogleCodeExporter commented 9 years ago 
shared_ptr isn't the only reason for using boost - yaml-cpp also uses 
Boost.Iterator. I couldn't find a standard implementation of that, but it's 
possible I missed it - do you know of one?

Original comment by jbe...@gmail.com on 1 Jun 2013 at 8:09

GoogleCodeExporter commented 9 years ago 
I don't see why you need a library, just inherit from std::iterator and do it 
by yourself.
I will stick with 0.3 for the moment and change to another library in the 
future, because i do not like the new API, the boost dependency in this 3rd 
party library and the recent code bloat (node/*).

Original comment by km808...@gmail.com on 1 Jun 2013 at 10:06

GoogleCodeExporter commented 9 years ago 
Sounds good to me.

By the way: Boost.Iterator does a significant amount of dirty work (see 
iterator_facade, e.g.). Also, on my system, the 0.5.1 binary is 16% larger than 
the 0.3.0 binary, which doesn't strike me as a significant amount of code 
bloat, given the more powerful API.

Original comment by jbe...@gmail.com on 3 Jun 2013 at 2:00