clean up editor configuration lines in a11y files

RESOLVED FIXED

Status

()

defect
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: tbsaunde, Assigned: tbsaunde)

Tracking

(Blocks 1 bug)

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

Assignee

Description

8 years ago
User-Agent:       Mozilla/5.0 (X11; Linux x86_64; rv:6.0a1) Gecko/20110425 Firefox/6.0a1
Build Identifier: 

before we tart reindenting atk/ to use 2 space t indents instead of 4 we should change the editor configuration lines to use 2 spaces instead of 4.  Also while doing this outside of atk/ there where a few files that had no such lines and many others that only  had emacs ones, make sure all these files have ones for both vim and emacs.

Reproducible: Always
Comment on attachment 528559 [details] [diff] [review]
MAKE SURE EACH FILE IN ATK/ HAS BOTH VIM AND EMACS LINES SETUP FOR 2 SPACE INDENTS

Review of attachment 528559 [details] [diff] [review]:

r=me
Attachment #528559 - Flags: review?(surkov.alexander) → review+
landed - http://hg.mozilla.org/mozilla-central/rev/f9a978db8102
Assignee: nobody → trev.saunders
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla6

Comment 5

8 years ago
It doesn't work for me.
I got
Error detected while processing modelines:
line    2:
E518: Unknown option: */

It works if I change
/* vim:expandtab:shiftwidth=2:tabstop=2: */
to
/* vim:expandtab:shiftwidth=2:tabstop=2:
 */
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Trevor, could you address comment #5 please?
Assignee

Comment 7

8 years ago
(In reply to comment #6)
> Trevor, could you address comment #5 please?

Yeah, I notice the same issue too, I  took a quick look and we had this issue  in some files before, and I ended up just spreading it around all of atk/ :(    https://developer.mozilla.org/En/Mozilla_Coding_Style_Guide suggests /* vim: set ts=2 et sw=2 tw=80: * which works for me, does it work for Ginn?
Blocks: cleana11y
Target Milestone: mozilla6 → ---
Assignee

Comment 8

8 years ago
Posted patch patchSplinter Review
I used the vim configuration suggested in 
https://developer.mozilla.org/En/Developer_Guide/Coding_Style
Ginn Fernando does this work for you? it seems to work for me.
Attachment #535766 - Flags: review?(fherrera)

Updated

8 years ago
Attachment #535766 - Flags: review?(fherrera) → review+
Assignee

Comment 9

8 years ago
landed http://hg.mozilla.org/mozilla-central/rev/78ad95eaf1f1
Status: REOPENED → RESOLVED
Last Resolved: 8 years ago8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.