I see. We should change the asl style guide then. The underscore seems
really odd to me. I don't know why we decided on this.

Anyhow this package follows the google styleguide which has template
parameters named like classes, e.g. without the trailing underscore.
On 16 Dec 2013 11:30, "Markus Achtelik" <notifications@github.com> wrote:

> In msf_updates/include/msf_updates/pose_sensor_handler/pose_measurement.h:
>
> > @@ -34,6 +34,14 @@ enum {
> >   */
> >  typedef msf_core::MSF_Measurement<geometry_msgs::PoseWithCovarianceStamped,
> >      Eigen::Matrix<double, nMeasurements, nMeasurements>, msf_updates::EKFState> PoseMeasurementBase;
> > +
> > +template<
> > +    int StateL_ = EKFState::StateDefinition_T::L,
>
> hmm, I did this according to
> https://github.com/ethz-asl/programming_styleguide/wiki/Cpp-Coding-Style-Guidelines#9-names-representing-template-parameters-should-be-named-like-types-but-with-an-trailing-underscore-or-with-a-single-capital-letter-in-case-there-are-few-arguments-and-their-meaning-is-clear
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/ethz-asl/ethzasl_msf/pull/40/files#r8362945>
> .
>