It's not called out in the gwt guidelines, but I don't think I'm giving away
any secrets by saying 80/100 is the Google convention.
On May 31, 2011 6:21 AM, <[email protected]> wrote:
> I wasn't able to find that in the style guide or discussion when it was
> changed. 80 code / 100 comments feels inconsistent.
>
>
> On 2011/05/29 23:06:55, zundel wrote:
>> Leaving comments at 80 was intentional as per the style guide, I
> thought.
>> On May 29, 2011 6:43 PM, <mailto:[email protected]> wrote:
>> > Reviewers: zundel, rjrjr,
>> >
>> > Description:
>> > Format comments to 100 columns to match code format
>> >
>> >
>> > Please review this at http://gwt-code-reviews.appspot.com/1450808/
>> >
>> > Affected files:
>> > M eclipse/settings/code-style/gwt-format.xml
>> >
>> >
>> > Index: eclipse/settings/code-style/gwt-format.xml
>> > ===================================================================
>> > --- eclipse/settings/code-style/gwt-format.xml (revision 10241)
>> > +++ eclipse/settings/code-style/gwt-format.xml (working copy)
>> > @@ -64,7 +64,7 @@
>> > <setting
> id="org.eclipse.jdt.core.formatter.comment.indent_root_tags"
>> > value="true"/>
>> > <setting
>> >
>
>
id="org.eclipse.jdt.core.formatter.comment.insert_new_line_before_root_tags"
>
>> > value="insert"/>
>> > <setting
>> >
> id="org.eclipse.jdt.core.formatter.comment.insert_new_line_for_parameter"
>> > value="do not insert"/>
>> > -<setting id="org.eclipse.jdt.core.formatter.comment.line_length"
>> > value="80"/>
>> > +<setting id="org.eclipse.jdt.core.formatter.comment.line_length"
>> > value="100"/>
>> > <setting
>> >
> id="org.eclipse.jdt.core.formatter.comment.new_lines_at_block_boundaries"
>> > value="true"/>
>> > <setting
>> >
>
>
id="org.eclipse.jdt.core.formatter.comment.new_lines_at_javadoc_boundaries"
>> > value="true"/>
>> > <setting id="org.eclipse.jdt.core.formatter.compact_else_if"
>> value="true"/>
>> >
>> >
>
>
>
> http://gwt-code-reviews.appspot.com/1450808/

-- 
http://groups.google.com/group/Google-Web-Toolkit-Contributors

Reply via email to