arbitrary_value.h does not include profiler.h

yse / easy_profiler

Lightweight profiler library for c++

MIT License

2.17k stars 189 forks source link

arbitrary_value.h does not include profiler.h #134

Closed mathieumallet closed 5 years ago

mathieumallet commented 6 years ago

This is a minor issue. When doing:

#include <easy/arbitrary_value.h>
#include <easy/profiler.h>

, a compilation error results as easy/arbitrary_value.h doesn't include easy/profiler.h. The fix would appear to be to include profile.h first, but if an app uses headers sorting then the 'normal' sorting results in a compilation error.

arbitrary_value.h should be updated to include profile.h (or, better yet, including easy/details/profiler_public_types.h)

cas4ey commented 6 years ago

Hi @mmallet-youilabs
Thanks for reporting!
Fixed :)

mathieumallet commented 6 years ago

:awwyeah: