fltk-1.4 doxygen warning for new fl_draw_check

[imacarthur]

Doxygen throws the following (at least for me!) when building the 1.4 branch...

=== making documentation ===

Generating HTML documentation...

D:/Support/fltk-1.4/src/fl_draw.cxx:481: warning: argument 'X' of command @param is not found in the argument list of fl_draw_check(Fl_Rect bb, Fl_Color col)

D:/Support/fltk-1.4/src/fl_draw.cxx:481: warning: argument 'Y' of command @param is not found in the argument list of fl_draw_check(Fl_Rect bb, Fl_Color col)

D:/Support/fltk-1.4/src/fl_draw.cxx:481: warning: argument 'W' of command @param is not found in the argument list of fl_draw_check(Fl_Rect bb, Fl_Color col)

D:/Support/fltk-1.4/src/fl_draw.cxx:481: warning: argument 'H' of command @param is not found in the argument list of fl_draw_check(Fl_Rect bb, Fl_Color col)

D:/Support/fltk-1.4/FL/fl_draw.H:930: warning: The following parameters of fl_draw_check(Fl_Rect bb, Fl_Color col) are not documented:

  parameter 'bb'

  parameter 'col'

/d/Support

It just looks like the doxygen comment has fallen out of sync with the code a bit. I suggest the following patch would probably do the job:

--------------

diff --git a/src/fl_draw.cxx b/src/fl_draw.cxx

index 26f49f10f..247492f17 100644

--- a/src/fl_draw.cxx

+++ b/src/fl_draw.cxx

@@ -492,8 +492,8 @@ void fl_restore_scale(float s) {

   The size limits are implementation details and may be changed at any time.

-  \param[in]  X,Y  top left corner of the bounding box

-  \param[in]  W,H  width and height of the bounding box

+  \param[in]  bb  rectangle that defines the bounding box

+  \param[in]  col Fl_Color to draw the check mark

   \since 1.4.0

 */

[Albrecht Schlosser]

Thanks. Done in commit 4d31c5bd4f3a45.

<OT>

Hint: If you or anybody else want to send a patch with a commit message
and your own author you can do the following:

1) make sure your repo is up-to-date (`git checkout master; git pull`)
2) check out a working branch (`git checkout -b working`)
3) change code and commit your changes
4) create patch file(s): `git format-patch master..`

Note the trailing '..' !

This will create one or more patch files with names like
'nnnn-Commit-Message-Title.patch where 'nnnn' is a sequence number. Then
attach the patch file(s) to the message.

To clean up your repo:

5) git checkout master
6) git branch -D working

Note the uppercase 'D' to delete the unmerged branch.

Such patch files can be applied with `git am 000*.patch` in sequence and
with the original time stamp and author.

Of course this is not necessary, I can also apply the patch as it was
pasted in the message text. This hint is just in case someone wants to
send in a patch and keep their author and commit messages.

I put this on my todo list for the appropriate article.

</OT>