search

MRPT / mrpt

:zap: The Mobile Robot Programming Toolkit (MRPT)
https://docs.mrpt.org/reference/latest/
BSD 3-Clause "New" or "Revised" License
1.94k stars 628 forks source link

Update ExprTk #1214

Closed ArashPartow closed 2 years ago

codecov[bot] commented 2 years ago

Codecov Report

Merging #1214 (2cba4f0) into develop (bb66875) will decrease coverage by 0.04%. The diff coverage is n/a.

:exclamation: Current head 2cba4f0 differs from pull request most recent head a5c88dd. Consider uploading reports for the commit a5c88dd to get more accurate results

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #1214      +/-   ##
===========================================
- Coverage    39.23%   39.19%   -0.05%     
===========================================
  Files         1284     1284              
  Lines       109506   109631     +125     
===========================================
+ Hits         42962    42967       +5     
- Misses       66544    66664     +120
Impacted Files Coverage Δ
libs/expr/include/mrpt/3rdparty/exprtk.hpp 24.47% <ø> (-0.36%) :arrow_down:
libs/core/include/mrpt/core/WorkerThreadsPool.h 100.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update bb66875...a5c88dd. Read the comment docs.

jlblancoc commented 2 years ago

Thanks, @ArashPartow !

This goes through our unit tests, but just in case: any new feature or backwards incompatible change worth knowing and/or documenting?

ArashPartow commented 2 years ago

@jlblancoc: There are no interface changes that will affect MRPT. All the changes are internal implementation.