search

NLESC-JCER / run-cpp-on-web

cpp2wasm blogs
Apache License 2.0
6 stars 2 forks source link

Use cmath, std::abs instead of C equivalents. #109

Closed janvdijk closed 4 years ago

janvdijk commented 4 years ago

Dear Stefan,

please consider this minor change: C++ folks may frown upon or be distracted by the usage of C's fabs, , instead of C++' std::abs,

Thanks, Jan.

sverhoeven commented 4 years ago

Thanks for the PR, indeed I don't want to have frowning C++ folks.

This PR was merged into #113 to implement every suggestions in one go.