Message ID | 20211203110527.19559-1-laurent.pinchart@ideasonboard.com |
---|---|
State | Accepted |
Commit | 4b24b0bfe6dd1ec716a605b683e45cd47a0bd6ee |
Headers | show |
Series |
|
Related | show |
Hi Laurent, Thanks for the patch On 12/3/21 4:35 PM, Laurent Pinchart wrote: > Commit ad2bcbe5b55c ("libcamera: camera_lens: add CameraLens as a member > of CameraSensor") incorrectly named the focusLens() function > "focusLens_" in the documentation, causing a doxygen warning. Fix it. > > Fixes: ad2bcbe5b55c ("libcamera: camera_lens: add CameraLens as a member of CameraSensor") > Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Reviewed-by: Umang Jain <umang.jain@ideasonboard.com> > --- > Han-lin, could you please make sure to have a look for doxygen warnings > in future patches ? We'll also update our integration scripts to detect > them. > --- > src/libcamera/camera_sensor.cpp | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/libcamera/camera_sensor.cpp b/src/libcamera/camera_sensor.cpp > index c89d39e1488c..4c142a5825f7 100644 > --- a/src/libcamera/camera_sensor.cpp > +++ b/src/libcamera/camera_sensor.cpp > @@ -789,7 +789,7 @@ void CameraSensor::updateControlInfo() > } > > /** > - * \fn CameraSensor::focusLens_() > + * \fn CameraSensor::focusLens() > * \brief Retrieve the focus lens controller > * > * \return The focus lens controller. nullptr if no focus lens controller is > > base-commit: 3f881dbc338e8f24f478567ee890c1f36ccb8996
Hi Laurent, On 03/12/2021 12:07, Umang Jain wrote: > Hi Laurent, > > Thanks for the patch > > On 12/3/21 4:35 PM, Laurent Pinchart wrote: >> Commit ad2bcbe5b55c ("libcamera: camera_lens: add CameraLens as a member >> of CameraSensor") incorrectly named the focusLens() function >> "focusLens_" in the documentation, causing a doxygen warning. Fix it. >> >> Fixes: ad2bcbe5b55c ("libcamera: camera_lens: add CameraLens as a >> member of CameraSensor") >> Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > > > Reviewed-by: Umang Jain <umang.jain@ideasonboard.com> Reviewed-by: Jean-Michel Hautbois <jeanmichel.hautbois@ideasonboard.com> > >> --- >> Han-lin, could you please make sure to have a look for doxygen warnings >> in future patches ? We'll also update our integration scripts to detect >> them. >> --- >> src/libcamera/camera_sensor.cpp | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/src/libcamera/camera_sensor.cpp >> b/src/libcamera/camera_sensor.cpp >> index c89d39e1488c..4c142a5825f7 100644 >> --- a/src/libcamera/camera_sensor.cpp >> +++ b/src/libcamera/camera_sensor.cpp >> @@ -789,7 +789,7 @@ void CameraSensor::updateControlInfo() >> } >> /** >> - * \fn CameraSensor::focusLens_() >> + * \fn CameraSensor::focusLens() >> * \brief Retrieve the focus lens controller >> * >> * \return The focus lens controller. nullptr if no focus lens >> controller is >> >> base-commit: 3f881dbc338e8f24f478567ee890c1f36ccb8996
Quoting Laurent Pinchart (2021-12-03 11:05:27) > Commit ad2bcbe5b55c ("libcamera: camera_lens: add CameraLens as a member > of CameraSensor") incorrectly named the focusLens() function > "focusLens_" in the documentation, causing a doxygen warning. Fix it. > > Fixes: ad2bcbe5b55c ("libcamera: camera_lens: add CameraLens as a member of CameraSensor") > Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com> > --- > Han-lin, could you please make sure to have a look for doxygen warnings > in future patches ? We'll also update our integration scripts to detect > them. Yes, sorry - looking at the logs it got flagged up - but doesn't cause a fatal stop, so it didn't trap, and I missed it. I'll have to figure out how to make doxygen -Werror ... ? -- Kieran > --- > src/libcamera/camera_sensor.cpp | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/libcamera/camera_sensor.cpp b/src/libcamera/camera_sensor.cpp > index c89d39e1488c..4c142a5825f7 100644 > --- a/src/libcamera/camera_sensor.cpp > +++ b/src/libcamera/camera_sensor.cpp > @@ -789,7 +789,7 @@ void CameraSensor::updateControlInfo() > } > > /** > - * \fn CameraSensor::focusLens_() > + * \fn CameraSensor::focusLens() > * \brief Retrieve the focus lens controller > * > * \return The focus lens controller. nullptr if no focus lens controller is > > base-commit: 3f881dbc338e8f24f478567ee890c1f36ccb8996 > -- > Regards, > > Laurent Pinchart >
On Fri, Dec 03, 2021 at 11:14:24AM +0000, Kieran Bingham wrote: > Quoting Laurent Pinchart (2021-12-03 11:05:27) > > Commit ad2bcbe5b55c ("libcamera: camera_lens: add CameraLens as a member > > of CameraSensor") incorrectly named the focusLens() function > > "focusLens_" in the documentation, causing a doxygen warning. Fix it. > > > > Fixes: ad2bcbe5b55c ("libcamera: camera_lens: add CameraLens as a member of CameraSensor") > > Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > > > Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com> > > > --- > > Han-lin, could you please make sure to have a look for doxygen warnings > > in future patches ? We'll also update our integration scripts to detect > > them. > > Yes, sorry - looking at the logs it got flagged up - but doesn't cause > a fatal stop, so it didn't trap, and I missed it. > > I'll have to figure out how to make doxygen -Werror ... ? diff --git a/Documentation/Doxyfile.in b/Documentation/Doxyfile.in index 3d20e3ca460f..0fbf00acf439 100644 --- a/Documentation/Doxyfile.in +++ b/Documentation/Doxyfile.in @@ -760,7 +760,7 @@ WARN_NO_PARAMDOC = NO # a warning is encountered. # The default value is: NO. -WARN_AS_ERROR = NO +WARN_AS_ERROR = YES # The WARN_FORMAT tag determines the format of the warning messages that doxygen # can produce. The string should contain the $file, $line, and $text tags, which It may be a bit harsh to enable that unconditionally though, as we currently require a very recent doxygen version to build without warning. This is something that could be set conditionally based on the doxygen version possibly. We already generate Doxyfile from Doxyfile.in, so it shouldn't be too hard. > > --- > > src/libcamera/camera_sensor.cpp | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/src/libcamera/camera_sensor.cpp b/src/libcamera/camera_sensor.cpp > > index c89d39e1488c..4c142a5825f7 100644 > > --- a/src/libcamera/camera_sensor.cpp > > +++ b/src/libcamera/camera_sensor.cpp > > @@ -789,7 +789,7 @@ void CameraSensor::updateControlInfo() > > } > > > > /** > > - * \fn CameraSensor::focusLens_() > > + * \fn CameraSensor::focusLens() > > * \brief Retrieve the focus lens controller > > * > > * \return The focus lens controller. nullptr if no focus lens controller is > > > > base-commit: 3f881dbc338e8f24f478567ee890c1f36ccb8996
diff --git a/src/libcamera/camera_sensor.cpp b/src/libcamera/camera_sensor.cpp index c89d39e1488c..4c142a5825f7 100644 --- a/src/libcamera/camera_sensor.cpp +++ b/src/libcamera/camera_sensor.cpp @@ -789,7 +789,7 @@ void CameraSensor::updateControlInfo() } /** - * \fn CameraSensor::focusLens_() + * \fn CameraSensor::focusLens() * \brief Retrieve the focus lens controller * * \return The focus lens controller. nullptr if no focus lens controller is
Commit ad2bcbe5b55c ("libcamera: camera_lens: add CameraLens as a member of CameraSensor") incorrectly named the focusLens() function "focusLens_" in the documentation, causing a doxygen warning. Fix it. Fixes: ad2bcbe5b55c ("libcamera: camera_lens: add CameraLens as a member of CameraSensor") Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> --- Han-lin, could you please make sure to have a look for doxygen warnings in future patches ? We'll also update our integration scripts to detect them. --- src/libcamera/camera_sensor.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) base-commit: 3f881dbc338e8f24f478567ee890c1f36ccb8996