search

mayah / tinytoml

A header only C++11 library for parsing TOML
BSD 2-Clause "Simplified" License
167 stars 31 forks source link

Suggestion: rename include/toml to include/tinytoml #35

Closed cvilas closed 6 years ago

cvilas commented 6 years ago

There seems to be a convention to name top level path of header files to match the corresponding library name. This is followed by many other projects, eg: opencv, gsl. Would you consider doing the same for this library too?

Thanks for this project. This is my preferred library for parsing configuration files in my own projects.

mayah commented 6 years ago

It might be a convention, but there are lots of exceptions e.g. jsoncpp.

I feel it's too late to change the name. If I can change backward compatibility, I'd like to change a lot of things, but currently I don't want to go that way.

The lucky thing is this library is header-only. So it's easy to rename the directory for your project.

cvilas commented 6 years ago

Fair enough! Thanks.