incubator-flex-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Igor Costa <igorco...@gmail.com>
Subject Re: Code formatting for patch submissions
Date Mon, 27 Feb 2012 20:58:29 GMT
Gordon

The 2 and 3 item I agree with you to remove those asterisks from comments.

100 is in my point of view is enough due to we don't only have widescreen
monitors but tablets, that's loop back to CRT. Why? I do view doc
specification on my iPad using as my second monitor during training and
fits pretty well on my view.

Again the code guide currently on FB is much more than I can get. And code
style guide fits pretty much what we need on Apache Flex project.


Most devs and contributors of already open program by Adobe are used to
this code style, changing it is like what happened in flex 3 to flex 4
migration docs and specs. And the current style fits on the side of each
Java dev, which is 85% compatible style code guide where you can check out
here http://www.oracle.com/technetwork/java/codeconv-138413.html



Regards
Igor Costa

On Mon, Feb 27, 2012 at 5:31 PM, Omar Gonzalez <omarg.developer@gmail.com>wrote:

> On Mon, Feb 27, 2012 at 12:03 PM, Carol Frampton <cframpto@adobe.com>
> wrote:
>
> >
> >
> > On 2/27/12 1 :55PM, "Gordon Smith" <gosmith@adobe.com> wrote:
> >
> > >I wrote Adobe's coding guidelines for ActionScript and I noticed a few
> > >ways in which this formatting doesn't match what the framework's current
> > >conventions:
> > >
> > >1. We wrap lines at 80 characters.
> >
> > Gordon we changed the wrap to 100 characters last year when we updated
> the
> > spec.  If you use FlashBuilder you can configure the Text Editor to show
> > the print margin at 100 which is a handy reference.
> >
> > Carol
>
>
> I think 100 sounds more reasonable. 80 is old school CRT days, I'm pretty
> sure most people are on widescreen monitors now. Don't want to get carried
> away either and allow never ending lines, 100 seems reasonable.
>
> --
> Omar Gonzalez
> s9tpepper@apache.org
> Apache Flex PPMC Member
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message