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
