The more I read threads like this the more it seems elementary should migrate to Go. :-P On Apr 1, 2013 3:29 AM, "Jaap Broekhuizen" <jaap...@gmail.com> wrote:
> I agree with Victor. Consistency matters because it makes readability and > therefore maintainability better. > > -- > Jaap > Op 1 apr. 2013 09:09 schreef "Victor" <victoredua...@gmail.com> het > volgende: > >> 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 <webe...@gmail.com> 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" < >> ser...@elementaryos.org> 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 <da...@elementaryos.org> >>> >>>> 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 <webe...@gmail.com> 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" <c...@elementaryos.org> wrote: >>>>> >>>>>> Cool, it's pretty thorough. >>>>>> >>>>>> >>>>>> On Wed, Mar 27, 2013 at 7:58 AM, David Gomes >>>>>> <da...@elementaryos.org>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@lists.launchpad.net >>>>>>> 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@lists.launchpad.net >>>>>> 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@lists.launchpad.net >>>> Unsubscribe : https://launchpad.net/~elementary-dev-community >>>> More help : https://help.launchpad.net/ListHelp >>>> >>>> >>> >>> >>> -- >>> Sergey "Shnatsel" Davidoff >>> OS architect @ elementary >>> >> >> -- >> Mailing list: https://launchpad.net/~elementary-dev-community >> Post to : elementary-dev-community@lists.launchpad.net >> 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@lists.launchpad.net Unsubscribe : https://launchpad.net/~elementary-dev-community More help : https://help.launchpad.net/ListHelp
