search

dropbox / json11

A tiny JSON library for C++11.
MIT License
2.55k stars 613 forks source link

make JsonParser invisible from outside #56

Closed Dushistov closed 8 years ago

Dushistov commented 8 years ago

JsonParser is not visible from outside of json11.cpp, so it has sense to move it into anonymous namespace to hint compiler that more strong optimizations are possible . gcc 5.2 not give any visible performance gains (~1%), but with this patch change size of library (.text section) on gcc/linux was reduceded by 5%.

Dushistov commented 8 years ago

I subscribed "Dropbox Contributor License Agreement".

j4cbo commented 8 years ago

Thanks!