Skip to content

Commit

Permalink
Fix warnings: "extra ‘;’ [-Wpedantic]"
Browse files Browse the repository at this point in the history
  • Loading branch information
s-trinh committed Nov 26, 2024
1 parent 8206553 commit c60ddae
Show file tree
Hide file tree
Showing 15 changed files with 18 additions and 18 deletions.
2 changes: 1 addition & 1 deletion modules/core/src/image/vpColormap.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1005,7 +1005,7 @@ unsigned char m_winterSrgbBytes[256][3] = {
{0, 245, 132}, {0, 246, 131}, {0, 247, 131}, {0, 248, 131}, {0, 249, 130}, {0, 250, 130}, {0, 251, 129},
{0, 252, 129}, {0, 253, 128}, {0, 254, 128}, {0, 255, 127}
};
};
}

BEGIN_VISP_NAMESPACE
/*!
Expand Down
2 changes: 1 addition & 1 deletion modules/core/src/image/vpImageConvert_yuv.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ void vpSAT(int &c)
c = val_255;
}
}
};
}

BEGIN_VISP_NAMESPACE
/*!
Expand Down
8 changes: 4 additions & 4 deletions modules/core/src/math/matrix/vpColVector.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -969,7 +969,7 @@ std::ostream &vpColVector::cppPrint(std::ostream &os, const std::string &matrixN
}
std::cout << std::endl;
return os;
};
}

std::ostream &vpColVector::csvPrint(std::ostream &os) const
{
Expand All @@ -980,7 +980,7 @@ std::ostream &vpColVector::csvPrint(std::ostream &os) const
os << std::endl;
}
return os;
};
}

std::ostream &vpColVector::maplePrint(std::ostream &os) const
{
Expand All @@ -993,7 +993,7 @@ std::ostream &vpColVector::maplePrint(std::ostream &os) const
}
os << "])" << std::endl;
return os;
};
}

std::ostream &vpColVector::matlabPrint(std::ostream &os) const
{
Expand All @@ -1009,7 +1009,7 @@ std::ostream &vpColVector::matlabPrint(std::ostream &os) const
}
}
return os;
};
}

#if defined(VISP_BUILD_DEPRECATED_FUNCTIONS)

Expand Down
2 changes: 1 addition & 1 deletion modules/core/src/tools/geometry/vpPlane.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -414,5 +414,5 @@ void vpPlane::changeFrame(const vpHomogeneousMatrix &cMo)
VISP_EXPORT std::ostream &operator<<(std::ostream &os, const vpPlane &p)
{
return (os << "(" << p.getA() << "," << p.getB() << "," << p.getC() << "," << p.getD() << ") ");
};
}
END_VISP_NAMESPACE
2 changes: 1 addition & 1 deletion modules/core/src/tools/geometry/vpTriangle.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -100,7 +100,7 @@ vpTriangle &vpTriangle::operator=(const vpTriangle &tri)
apex2 = tri.apex2;
apex3 = tri.apex3;
return *this;
};
}

/*!
Initialise the triangle thanks to the three 2D points \f$ iP1 \f$, \f$ iP2
Expand Down
2 changes: 1 addition & 1 deletion modules/core/src/tools/time/vpTime.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -374,7 +374,7 @@ std::string getDateTime(const std::string &format)
}

#ifndef DOXYGEN_SHOULD_SKIP_THIS
};
}
#endif

vpChrono::vpChrono() : m_durationMs(), m_lastTimePoint() { }
Expand Down
2 changes: 1 addition & 1 deletion modules/robot/src/haptic-device/virtuose/vpVirtuose.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1054,5 +1054,5 @@ void vpVirtuose::stopPeriodicFunction()
END_VISP_NAMESPACE
#else
// Work around to avoid warning
void dummy_vpVirtuose() { };
void dummy_vpVirtuose() { }
#endif
2 changes: 1 addition & 1 deletion modules/sensor/src/force-torque/vpForceTorqueIitSensor.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -242,5 +242,5 @@ END_VISP_NAMESPACE
#else
// Work around to avoid warning:
// libvisp_sensor.a(vpForceTorqueIitSensor.cpp.o) has no symbols
void dummy_vpForceTorqueIitSensor() { };
void dummy_vpForceTorqueIitSensor() { }
#endif
Original file line number Diff line number Diff line change
Expand Up @@ -1463,5 +1463,5 @@ END_VISP_NAMESPACE
#else
// Work around to avoid warning:
// libvisp_flycapture.a(vpFlyCaptureGrabber.cpp.o) has no symbols
void dummy_vpFlyCaptureGrabber() { };
void dummy_vpFlyCaptureGrabber() { }
#endif
2 changes: 1 addition & 1 deletion modules/sensor/src/framegrabber/pylon/vpPylonFactory.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -86,5 +86,5 @@ END_VISP_NAMESPACE
#else
// Work around to avoid warning:
// libvisp_pylon.a(vpPylonFactory.cpp.o) has no symbols
void dummy_vpPylonFactory() { };
void dummy_vpPylonFactory() { }
#endif // #ifdef VISP_HAVE_PYLON
Original file line number Diff line number Diff line change
Expand Up @@ -815,5 +815,5 @@ END_VISP_NAMESPACE
#else
// Work around to avoid warning:
// libvisp_pylon.a(vpPylonGrabberGigE.cpp.o) has no symbols
void dummy_vpPylonGrabberGigE() { };
void dummy_vpPylonGrabberGigE() { }
#endif // #ifdef VISP_HAVE_PYLON
Original file line number Diff line number Diff line change
Expand Up @@ -797,5 +797,5 @@ END_VISP_NAMESPACE
#else
// Work around to avoid warning:
// libvisp_pylon.a(vpPylonGrabberUsb.cpp.o) has no symbols
void dummy_vpPylonGrabberUsb() { };
void dummy_vpPylonGrabberUsb() { }
#endif // #ifdef VISP_HAVE_PYLON
2 changes: 1 addition & 1 deletion modules/sensor/src/mocap/vpMocapQualisys.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -315,5 +315,5 @@ END_VISP_NAMESPACE
#else
// Work around to avoid warning:
// libvisp_sensor.a(vpMocapQualisys.cpp.o) has no symbols
void dummy_vpMocapQualisys() { };
void dummy_vpMocapQualisys() { }
#endif
2 changes: 1 addition & 1 deletion modules/sensor/src/mocap/vpMocapVicon.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -272,5 +272,5 @@ END_VISP_NAMESPACE
#else
// Work around to avoid warning:
// libvisp_sensor.a(vpMocapVicon.cpp.o) has no symbols
void dummy_vpMocapVicon() { };
void dummy_vpMocapVicon() { }
#endif
2 changes: 1 addition & 1 deletion modules/tracker/blob/src/dots/vpDot.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -916,6 +916,6 @@ void vpDot::display(const vpImage<vpRGBa> &I, const vpImagePoint &cog, const std
details about the orientation of the frame see the vpImagePoint
documentation) to the stream \e os, and returns a reference to the stream.
*/
VISP_EXPORT std::ostream &operator<<(std::ostream &os, vpDot &d) { return (os << "(" << d.getCog() << ")"); };
VISP_EXPORT std::ostream &operator<<(std::ostream &os, vpDot &d) { return (os << "(" << d.getCog() << ")"); }

Check warning on line 919 in modules/tracker/blob/src/dots/vpDot.cpp

View check run for this annotation

Codecov / codecov/patch

modules/tracker/blob/src/dots/vpDot.cpp#L919

Added line #L919 was not covered by tests

END_VISP_NAMESPACE

0 comments on commit c60ddae

Please sign in to comment.