elementary-dev-community team mailing list archive

[Craig <weberc2@xxxxxxxxx>]

Personally, I like that I can write code without thinking about the style
and then have it styled automatically when I push. It lets me focus on the
logic of my program rather than whether it obeys a style guideline. This is
especially useful because I participate in projects involving several
current languages and each with its own style guideline.

I'm not saying we need something like gofmt, but it's foolish to imply that
such a tool is useless (especially when we are manually investing time
correcting code that could be done automatically).

If an appropriate tool doesn't exist, I don't recommend developing one, but
I don't see how you can mock gofmt when I can validate my style with no
overhead whatsoever while you are doing it manually. Lol. ;-)
On Apr 1, 2013 9:28 AM, "David Gomes" <david@xxxxxxxxxxxxxxxx> wrote:

> Fortunately, most of the developers can write good code. And when they
> fail to do so we have other developers who review their code.
>
> We don't need a fancy tool like gofmt that just changes our code.
>
>
> On Mon, Apr 1, 2013 at 3:25 PM, Craig <weberc2@xxxxxxxxx> wrote:
>
>> 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" <jaapz.b@xxxxxxxxx> wrote:
>>
>>> I agree with Victor. Consistency matters because it makes readability
>>> and therefore maintainability better.
>>>
>>> --
>>> Jaap
>>> Op 1 apr. 2013 09:09 schreef "Victor" <victoreduardm@xxxxxxxxx> 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 <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
>>>>>
>>>>
>>>> --
>>>> 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
>>
>>
>