| M5 remove {} brackets feature. [message #225169] |
Sun, 05 March 2006 19:47  |
Eclipse User |
|
|
|
Hi JDT guys,
I haven't been following 3.2 dev lately, so I might have missed the
discussion
on this topic. so if I did, I apologize in advance.
It seems to me that this feature contradicts a popular programming paradigm
of placing the braces for every for/if/while. I don't want to argue
about whether
this is a good paradigm or not, but I think using braces leads to safer
and less error
prone code (ok, that was a little bit of arguing).
I think that Eclipse should encourage safer programming techniques, and this
feature (IMHO) goes the opposite way. I don't think it should be removed
once it has
been implemented, but maybe a warning in the preview window will be
appropriate.
Personally I'd add the word "recommended" to the relevant section of the
clean up dialog.
Except that - amazing job. I had no idea Eclipse/JDT could be made
better, but
it's getting better with every new milestone...
--
Genady Beryozkin
http://www.genady.net/
|
|
|
|
| Re: M5 remove {} brackets feature. [message #225292 is a reply to message #225173] |
Mon, 06 March 2006 09:58  |
| Eclipse User |
|
|
|
There's also bug 128780, see more comments there.
The Quick Fix you saw in Ed's blog is not intended, we'll remove that
one; it's making the list of QF too long. But for 'clean up' we want to
provide symetical options. The opinions on using braces or not are not
clear-cut as many here want to make believe. I'm a bracket fan, but I
think I'm only one of ten here in the lab.
Martin
Genady wrote:
> I now see that Ed wrote about it too:
> http://blogs.zdnet.com/Burnette/?p=16
>
> Genady Beryozkin
> http://www.genady.net/
>
>
>
> Genady wrote:
>
>> Hi JDT guys,
>>
>> I haven't been following 3.2 dev lately, so I might have missed the
>> discussion
>> on this topic. so if I did, I apologize in advance.
>>
>> It seems to me that this feature contradicts a popular programming
>> paradigm
>> of placing the braces for every for/if/while. I don't want to argue
>> about whether
>> this is a good paradigm or not, but I think using braces leads to
>> safer and less error
>> prone code (ok, that was a little bit of arguing).
>>
>> I think that Eclipse should encourage safer programming techniques,
>> and this
>> feature (IMHO) goes the opposite way. I don't think it should be
>> removed once it has
>> been implemented, but maybe a warning in the preview window will be
>> appropriate.
>>
>> Personally I'd add the word "recommended" to the relevant section of
>> the clean up dialog.
>>
>> Except that - amazing job. I had no idea Eclipse/JDT could be made
>> better, but
>> it's getting better with every new milestone...
>>
|
|
|
| Re: M5 remove {} brackets feature. [message #225381 is a reply to message #225292] |
Mon, 06 March 2006 18:08 |
| Eclipse User |
|
|
|
Commented and CC'd myself. I have no problem with the cleanup dialog.
How about adding a web link to the description/discussion for each code
style option/warning?
Genady
Martin Aeschlimann wrote:
> There's also bug 128780, see more comments there.
>
> The Quick Fix you saw in Ed's blog is not intended, we'll remove that
> one; it's making the list of QF too long. But for 'clean up' we want
> to provide symetical options. The opinions on using braces or not are
> not clear-cut as many here want to make believe. I'm a bracket fan,
> but I think I'm only one of ten here in the lab.
>
> Martin
>
>
> Genady wrote:
>> I now see that Ed wrote about it too:
>> http://blogs.zdnet.com/Burnette/?p=16
>>
>> Genady Beryozkin
>> http://www.genady.net/
>>
>>
>>
>> Genady wrote:
>>
>>> Hi JDT guys,
>>>
>>> I haven't been following 3.2 dev lately, so I might have missed the
>>> discussion
>>> on this topic. so if I did, I apologize in advance.
>>>
>>> It seems to me that this feature contradicts a popular programming
>>> paradigm
>>> of placing the braces for every for/if/while. I don't want to argue
>>> about whether
>>> this is a good paradigm or not, but I think using braces leads to
>>> safer and less error
>>> prone code (ok, that was a little bit of arguing).
>>>
>>> I think that Eclipse should encourage safer programming techniques,
>>> and this
>>> feature (IMHO) goes the opposite way. I don't think it should be
>>> removed once it has
>>> been implemented, but maybe a warning in the preview window will be
>>> appropriate.
>>>
>>> Personally I'd add the word "recommended" to the relevant section of
>>> the clean up dialog.
>>>
>>> Except that - amazing job. I had no idea Eclipse/JDT could be made
>>> better, but
>>> it's getting better with every new milestone...
>>>
|
|
|
Powered by
FUDForum. Page generated in 0.04698 seconds
Home