On Apr 4, 2014, at 5:40 PM, Andrew Bell <notifications@github.com> wrote:

> In src/StageFactory.cpp:
> 
> > @@ -62,6 +62,7 @@
> >  //
> >  MAKE_READER_CREATOR(FauxReader, pdal::drivers::faux::Reader)
> >  MAKE_READER_CREATOR(LasReader, pdal::drivers::las::Reader)
> > +MAKE_READER_CREATOR(BpfReader, pdal::BpfReader)
> 
> I know this is different than what has been done, but I think one should have a reason to introduce a namespace and I don't think we have one. I'm not sure why all of the code isn't simply in the "pdal" namespace. I can think of quite a few disadvantages to the separate namespaces and can't thing of an advantage.
> 
> Thoughts?

Every driver explicitly namespaced is my doing. The reasons for doing so are as follows:

* Seems like every damn SDK has their own "Point" class, and this allows people to bury stuff inside a driver to prevent clashes
* When you're using an IDE, the namespacing allows for easy traversal rather than a polluted "pdal::" namespace with 30 readers, filters, and writers. As more objects are added, the worse this behavior gets.
* There's no runtime cost for it

What is your bullet list of disadvantages?