Bug 54553 - [EFL] Doxygen documentation for ewk cookies
Summary: [EFL] Doxygen documentation for ewk cookies
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Text (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Minor
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-02-16 07:11 PST by Grzegorz Czajkowski
Modified: 2011-02-17 07:01 PST (History)
4 users (show)

See Also:

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 an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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.