search

boostorg / chrono

Boost.org chrono module
http://boost.org/libs/chrono
15 stars 52 forks source link

Remove 'stopwatches' directory #32

Closed mclow closed 6 years ago

mclow commented 6 years ago

Vicente said (May 16th) that "I'm really sorry for that. Boost.Chrono/Stopwatches was never released even if it was included in the Boost repository" and "I'll remove it definitively from the boost repository."

Lastique commented 6 years ago

Note that stopwatches are referenced in https://github.com/boostorg/chrono/blob/develop/test/cycle_count.hpp, https://github.com/boostorg/chrono/blob/develop/example/chrono_accuracy_test.cpp#L14 and https://github.com/boostorg/thread/blob/develop/example/perf_condition_variable.cpp#L14. Also, there are stopwatches-related bits in https://github.com/boostorg/chrono/blob/develop/meta/libraries.json and https://github.com/boostorg/chrono/blob/develop/doc/chrono.qbk. These places need to be cleaned up as well.

viboes commented 6 years ago

Yes, this need a more careful cleanup.

viboes commented 6 years ago

I'm taking care of removing all the adherence to stopwatches folder.

mclow commented 6 years ago

I'm taking care of removing all the adherence to stopwatches folder.

In that case, I will close this P/R.

viboes commented 6 years ago

Hi,

I've just merged this

https://github.com/boostorg/chrono/commit/ce6286ae6eca8531f79c58745a7e1c83ebdc120f

Marshall,I had some troubles with my quickbook installation and I was unable to check the doc changes. Please, could you check that doc generation has not been broken?

Thanks for recall me to do this due cleanup.