275936 - Indent code

Status: RESOLVED FIXED

(addons.mozilla.org Graveyard :: Developer Pages, defect)

Description

[alanjstr]

The code needs to be indented to be more readable.

Comment 1

[Ben Fowler]

Googling for 'Mozilla indent code' produced this page on the first 
hit: http://www.mozilla.org/hacking/mozilla-style-guide.html , which
may be relevant in relation to C++ source code.

The section on 'Naming and formatting' actually specificies a tab width
of 2 (and no tabs) presumably on the grounds of readability. This 
document is over two years old, and has probably found favour.

You may find that this bug us resolved as WORKSFORME.

I am fairly sure that patches for deviations from the specifications
in the document about are appropropriate, but speaking for myself
I don't normally fix up whitespace changes, spelling or profanity
unless I am also making a substantive change (improvement) to a file.

Comment 2

[alanjstr]

This is for us to indent the php code for update.mozilla.org.  In other words,
to apply the style guide to the code.  This is more of a reminder to use the
style guide as we rewrite code than something that is expected to be patched.

Comment 3

[alanjstr]

We'll let Zend make it pretty.  Need this before we can start patching bugs, so
I'd like it soon.

Comment 4

[alanjstr]

Ok, mconnor did a big checkin on trunk.  This is now open to anyone who wants to
pick a file and clean it up.

Comment 5

[alanjstr]

Attachment: Root dir

I'll take this directory by directory.	I was able to run this one through html
tidy.

Comment 6

[alanjstr]

Attachment: Root dir

Comment 7

[alanjstr]

Attachment: About

Comment 8

[alanjstr]

Attachment: Plugins

Used tidy

Comment 9

[alanjstr]

AMO 2.0 is pretty