Ok, I'm going to file a bug for style checker. And, add you guys to CC. :-)<div><br></div><div>Thank you,</div><div>Gyuyoung.<br><br><div class="gmail_quote">On Sat, Dec 31, 2011 at 7:09 AM, David Levin <span dir="ltr"><<a href="mailto:levin@google.com" target="_blank">levin@google.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Feel free to patch the style checker as appropriate. I suspect there may be some changes needed here:<div><a href="http://trac.webkit.org/browser/trunk/Tools/Scripts/webkitpy/style/checker.py#L180" target="_blank">http://trac.webkit.org/browser/trunk/Tools/Scripts/webkitpy/style/checker.py#L180</a></div>



<div><br></div><div>Best wishes,</div><div>dave<br><br><div class="gmail_quote"><div><div>On Tue, Dec 27, 2011 at 11:54 PM, Gyuyoung Kim <span dir="ltr"><<a href="mailto:gyuyoung.kim@samsung.com" target="_blank">gyuyoung.kim@samsung.com</a>></span> wrote:<br>



</div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div>




<div>
<span style="text-indent:0px;letter-spacing:normal;font:13px/18px Arial;text-transform:none;white-space:normal;word-spacing:0px">
<p>Hello WebKit folks,</p>
<p><br></p>
<p>As you may know, I have changed the coding style of EFL port via Bug 68209. This is to eliminate unnecessary difficulties</p>
<p>that reviewers have experienced when they review EFL bugs, due to EFL specific coding style.</p>
<p> </p>
<p>Bug 68209 - [EFL] Change WebKit EFL coding style </p>
<p>(<a href="https://bugs.webkit.org/show_bug.cgi?id=68209" target="_blank">https://bugs.webkit.org/show_bug.cgi?id=68209</a>)</p>
<p> </p>
<p>Major changes are as below,</p>
<p> </p>
<p>1. Use full variable name instead of abbreviation name.</p>
<p>2. Use standard boolean type instead of EFL boolean type.</p>
<p>3. Place '*' operator immediately after the data type. (For example, use char* userAgent. instead of char *userAgent.)</p>
<p>4. Use C++ type casting instead of C type casting.</p>
<p>5. Remove void parameter from internal functions.</p>
<p>6. Use camelCase instead of  underbar(_) in variable names.</p>
<p> </p>
<p>I made a wiki page for this rule.  I will update this page whenever EFL port coding style guide is changed.</p>
<p>(<a href="http://trac.webkit.org/wiki/EFLWebKitCodingStyle" target="_blank">http://trac.webkit.org/wiki/EFLWebKitCodingStyle</a>)</p>
<p> </p>
<p>To sum up, EFL port adheres to WebKit coding style except for public header from now.</p>
<p>(Because public header is used by EFL applications, public header will continue to follow the coding style rule of EFL.)</p>
<p> </p>
<p>We hope that reviewing of EFL port patches will be much easier after this change.</p>
<p> </p>
<p>Happy new year !!</p>
<p> </p>
<p>Regards,</p>
<p>Gyuyoung Kim.</p></span></div><img border="0" width="0" height="0"><br></div></div><div>_______________________________________________<br>
webkit-dev mailing list<br>
<a href="mailto:webkit-dev@lists.webkit.org" target="_blank">webkit-dev@lists.webkit.org</a><br>
<a href="http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev" target="_blank">http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev</a><br>
<br></div></blockquote></div><br></div>
<br>_______________________________________________<br>
webkit-dev mailing list<br>
<a href="mailto:webkit-dev@lists.webkit.org" target="_blank">webkit-dev@lists.webkit.org</a><br>
<a href="http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev" target="_blank">http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev</a><br>
<br></blockquote></div><br></div>