elementary-dev-community team mailing list archive

[Victor <victoreduardm@xxxxxxxxx>]

Coding style is a subjective topic, and that's why discussing which one 
works best is completely pointless, since it's a matter of preferences. 
It's like discussing what is the best color.

What is important is consistency, and that's why all the new code 
proposed for merging should follow elementary's coding style guidelines 
(which are not published anywhere in the site as far as I know). 
Whenever you propose code that is styled inconsistently it only gives 
the impression that you were coding in a hurry, and we don't want to 
accept that kind of code, even though we have a ton of it already.

Thanks for your attention,
Victor.

On Sun, Mar 31, 2013 at 12:48 PM, Craig <weberc2@xxxxxxxxx> wrote:
> How do you figure? The go language community uses one and they rave 
> about it. We use them at work (c++) as well and its uses an obnoxious 
> style, but it's still more readable than a dozen different 
> conventions.
>
> On Mar 31, 2013 5:39 AM, "Sergey "Shnatsel" Davidoff" 
> <sergey@xxxxxxxxxxxxxxxx> wrote:
> > I'm afraid automatic "prettifiers" are a terrible idea because 
> > blindly restyling the code usually makes it lose any remains of 
> > readability it used to have. In other words, automatically restyled 
> > code is even less readable than code with a foreign coding style.
> >
> >
> > 2013/3/31 David Gomes <david@xxxxxxxxxxxxxxxx>
> > > I wrote this in order to check for code style errors, but it's not 
> > > perfect it's just a help-tool:
> > >
> > > https://github.com/elementary/vala-analyzer
> > >
> > > We have 'considered' using a prettifier too, but I just use Emacs 
> > > to fix some stuff on my code - a prettifier script would be too 
> > > much work and I don't know of any libraries that would help me with 
> > > the task.
> > >
> > >
> > > On Sun, Mar 31, 2013 at 3:34 AM, Craig <weberc2@xxxxxxxxx> wrote:
> > > > Good work David. Have you (elementary) considered using a 
> > > > prettifier to standardize a code style upon pushing to your trunk?
> > > >
> > > > On Mar 28, 2013 7:17 PM, "Cody Garver" <cody@xxxxxxxxxxxxxxxx> 
> > > > wrote:
> > > > > Cool, it's pretty thorough.
> > > > >
> > > > >
> > > > > On Wed, Mar 27, 2013 at 7:58 AM, David Gomes 
> > > > > <david@xxxxxxxxxxxxxxxx> wrote:
> > > > > > http://dl.dropbox.com/u/19899464/reviewstutorial.html
> > > > > >
> > > > > > Hello guys,
> > > > > >
> > > > > > From time to time somebody still has doubts on how to use 
> > > > > > Launchpad and Bazaar to review and merge branches to trunk so I 
> > > > > > wrote a tutorial. Note though that it may need expansion.
> > > > > >
> > > > > > Many times, even experienced developers who have been in the 
> > > > > > Apps Team for a long time make mistakes so even if you already 
> > > > > > know how to do it, reading the tutorial won't hurt.
> > > > > >
> > > > > > I also recommend that all developers that in the future are to 
> > > > > > join the Apps Team read this several times because even though 
> > > > > > we can always revert messed-up commits, it's better to do it 
> > > > > > right at the first time.
> > > > > >
> > > > > > Best regards,
> > > > > > David "Munchor" Gomes
> > > > > >
> > > > > > --
> > > > > > Mailing list: https://launchpad.net/~elementary-dev-community
> > > > > > Post to     : elementary-dev-community@xxxxxxxxxxxxxxxxxxx
> > > > > > Unsubscribe : https://launchpad.net/~elementary-dev-community
> > > > > > More help   : https://help.launchpad.net/ListHelp
> > > > >
> > > > >
> > > > >
> > > > > --
> > > > > Cody Garver
> > > > >
> > > > > --
> > > > > Mailing list: https://launchpad.net/~elementary-dev-community
> > > > > Post to     : elementary-dev-community@xxxxxxxxxxxxxxxxxxx
> > > > > Unsubscribe : https://launchpad.net/~elementary-dev-community
> > > > > More help   : https://help.launchpad.net/ListHelp
> > >
> > >
> > > --
> > > Mailing list: https://launchpad.net/~elementary-dev-community
> > > Post to     : elementary-dev-community@xxxxxxxxxxxxxxxxxxx
> > > Unsubscribe : https://launchpad.net/~elementary-dev-community
> > > More help   : https://help.launchpad.net/ListHelp
> >
> >
> >
> > --
> > Sergey "Shnatsel" Davidoff
> > OS architect @ elementary