Indenting code?
Phil Race
philip.race at oracle.com
Fri Sep 14 22:43:07 UTC 2012
On 9/14/2012 3:22 PM, Jim Graham wrote:
> I think tabs and spaces at ends of lines are fine rules for automated
> enforcement, but the rest of the formatting guidelines are guidelines,
> not rules. They are intended to make the code more readable as a
> whole by promoting consistency, but their individual mandates aren't
> always 100% the best technique. I'd say that 99.9% of the code I
> write could follow the guidelines just fine, but there are some cases
> where I play with indentation or combining or splitting lines to
> highlight the parallel nature or similarities of several lines of code
> (this probably comes up more in graphics where you deal with x,y and
> r,g,b,a computations that are all mostly identical than in more
> generic code).
>
> In contrast, I'm not sure I know of any code that requires tabs or
> spaces at the ends of lines to look good or be readable/maintainable...
Agreed. And as best I can recall most of the open source libs I've dealt
with try to varying
degrees to eliminate those so its not been much of an issue.
-phil.
>
> ...jim
>
> On 9/14/12 1:51 PM, Kelly O'Hair wrote:
>>
>> On Sep 14, 2012, at 1:30 PM, Phil Race wrote:
>>
>>> On 9/14/2012 12:34 PM, Kelly O'Hair wrote:
>>>> If we had a well defined formatter tool that we told people they
>>>> must use, I suppose jcheck could enforce it's use
>>>> by refusing any changeset where the source hasn't been "properly"
>>>> formatted.
>>>> But getting a whole bunch of different tools to do the same
>>>> formatting? That seems like an exercise in futility.
>>>>
>>>
>>> Although I suppose we already do this for the tabs and extra spaces
>>> at the end of lines for jcheck,
>>> we also need to remember all the 3rd party code we have and the pain
>>> that would be when we
>>> update it. I like to make zero to few changes to such code and live
>>> with their preferences - even
>>> those that irk me.
>>>
>>> -phil.
>>
>> A very valid point, we would need to know which files we would
>> enforce these rules, and that could be
>> very tricky.
>>
>> Seems like we have lots of better things to do and maybe this just
>> needs to be visited each time a source
>> file or set of sources gets a major change.
>>
>> -kto
More information about the discuss
mailing list