Bug 54553 – [EFL] Doxygen documentation for ewk cookies

• Status: RESOLVED
• Resolution: FIXED
• Component: Text

[Grzegorz Czajkowski]

Doxygen documentation for ewk_cookies.h and ewk_cookies.cpp has been added.

[Grzegorz Czajkowski]

Created attachment 82630 [details]
Doxygen documentation for ewk cookies

[Ryuan Choi]

I am not a reviewer, but I think that some of c applications can complain c++ styled comments.

[Grzegorz Czajkowski]

Hi Ryuan,

I've just wanted to improve current comments and add missing ones to have a html documentation for WebKit-Efl. The comments are compatible with Doxygen documentation system not C or C++ styled comments.

[Ryuan Choi]

(In reply to comment #3)
> Hi Ryuan,
> 
> I've just wanted to improve current comments and add missing ones to have a html documentation for WebKit-Efl. The comments are compatible with Doxygen documentation system not C or C++ styled comments.

Although c99, gcc(without -ansi) will not complain ///, I believe that we can avoid it using /**< any comments */ in header files.

[Grzegorz Czajkowski]

(In reply to comment #4)
> (In reply to comment #3)
> > Hi Ryuan,
> > 
> > I've just wanted to improve current comments and add missing ones to have a html documentation for WebKit-Efl. The comments are compatible with Doxygen documentation system not C or C++ styled comments.
> 
> Although c99, gcc(without -ansi) will not complain ///, I believe that we can avoid it using /**< any comments */ in header files.

Ok, I will replace '///' to '/**< */'

[Kent Tamura]

Comment on attachment 82630 [details]
Doxygen documentation for ewk cookies

rs=me though I don't know the syntax of Doxygen.

[WebKit Commit Bot]

Comment on attachment 82630 [details]
Doxygen documentation for ewk cookies

Clearing flags on attachment: 82630

Committed r78823: <http://trac.webkit.org/changeset/78823>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.