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

Grzegorz Czajkowski

Reported 2011-02-16 07:11:18 PST

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

Attachments
Doxygen documentation for ewk cookies (5.13 KB, patch) 2011-02-16 07:24 PST, Grzegorz Czajkowski	no flags	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Grzegorz Czajkowski

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

Created attachment 82630 [details] Doxygen documentation for ewk cookies

Ryuan Choi

Comment 2 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.

Grzegorz Czajkowski

Comment 3 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.

Ryuan Choi

Comment 4 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.

Grzegorz Czajkowski

Comment 5 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 '/**< */'

Kent Tamura

Comment 6 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.

WebKit Commit Bot

Comment 7 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>

WebKit Commit Bot

Comment 8 2011-02-17 07:01:47 PST

All reviewed patches have been landed. Closing bug.

Note You need to log in before you can comment on or make changes to this bug.