Bug 54553

Summary:

[EFL] Doxygen documentation for ewk cookies

Product:

WebKit

Reporter:

Grzegorz Czajkowski <g.czajkowski>

Component:

Text

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Minor

CC:

commit-queue, gyuyoung.kim, l.slachciak, ryuan.choi

Priority:

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Attachments:

Description	Flags
Doxygen documentation for ewk cookies	none

Description Grzegorz Czajkowski 2011-02-16 07:11:18 PST

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

Comment 1 Grzegorz Czajkowski 2011-02-16 07:24:35 PST

Created attachment 82630 [details]
Doxygen documentation for ewk cookies

Comment 2 Ryuan Choi 2011-02-17 01:11:10 PST

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

Comment 3 Grzegorz Czajkowski 2011-02-17 01:50:06 PST

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.

Comment 4 Ryuan Choi 2011-02-17 05:17:16 PST

(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.

Comment 5 Grzegorz Czajkowski 2011-02-17 05:24:40 PST

(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 '/**< */'

Comment 6 Kent Tamura 2011-02-17 06:41:03 PST

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

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

Comment 7 WebKit Commit Bot 2011-02-17 07:01:42 PST

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

Clearing flags on attachment: 82630

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

Comment 8 WebKit Commit Bot 2011-02-17 07:01:47 PST

All reviewed patches have been landed.  Closing bug.