avoid useless case (FDuration pointer deref to FDuration=

[SvenJo]

Description

Useless cast from FDuration to FDuration in catch_analyse.cpp

Found with the GCC warning: -Wuseless-cast

• Parameter: FDuration* first
• auto it = first; // it is also FDuration*
• std::vector<FDuration> samples;
• samples.push_back(FDuration(*it)); push_back expects FDuration, so no cast needed.

same for mean += FDuration(*it);: mean is defined as FDuration mean = FDuration(0);, so mean += *it; should be fine.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.16%. Comparing base (024cfb3) to head (fa8c0c1).
Report is 4 commits behind head on devel.

Additional details and impacted files

@@            Coverage Diff             @@
##            devel    #2823      +/-   ##
==========================================
- Coverage   91.34%   91.16%   -0.17%     
==========================================
  Files         197      197              
  Lines        8289     8374      +85     
==========================================
+ Hits         7571     7634      +63     
- Misses        718      740      +22     

[horenmar]

Thanks