Adding conventional PID computation
Merging changes from other branch
Fixing documentation
adding doxygen deprecation flags
Cleaning up documentation, making argument names in function declaration match those in the implementation
Make public getControllerByName method thread-safe.

Existing virtual non-threadsafe method has been suffixed with -Impl and pushed
to protected class scope. In-class uses call getControllerByNameImpl, as the
lock has already been acquired.
Merge pull request #12 from jbohren-forks/feature-conventional-pid-error-sign

Adding conventional PID computation, deprecating current PID computation
Adding test to expose Pid class zero-division vulnerability
Merge branch 'test-bad-integral-bounds' into fix-PID-unbounded-i_error
Merge pull request #15 from pal-robotics/master

Make public getControllerByName method thread-safe.
Specifying div-by-zero test, adding other integral term tests
Merge branch 'test-bad-integral-bounds' into fix-PID-unbounded-i_error
Merge branch 'master' into test-bad-integral-bounds
Resolving conflict from new Pid API
Merge pull request #16 from jbohren-forks/test-bad-integral-bounds

Tests for bad integral bounds in control_toolbox::Pid
Merge typo
This makes the internal computations of updatePid() keep the same sign that they did before the API change
adding documentation warning
bringing back old updatePid function contents
Merge branch 'master' into fix-pid-backwards-compatibility