Skrypt commitFixer

[GoogleCodeExporter]

Na początek przytoczę emaile z RT:

WOJTEK:

Proponuję zrobić masowy porządek z wcięciami w kodzie.

Chodzi o to, że w wielu plikach na zmianę używane są spacje i taby do 
wcięć. Czyni to te pliki nieprzyjemne w czytaniu i edycji. (Ja 
wprawdzie sie nie dzielam za bardzo w edycji plików OC, ale jak czasem 
rzucę okiem na te diffy, to szczena opada ;))

W związku z tym proponuję, że odplalę na całym kodzie skrypt, który
zrobi porządek. Standard PSR-2 (takie zalecenia jak pisać w PHP)
zaleca użycie spacji zmiast tabów, więc pewnie tak właśnie warto
zrobić (OKAPI używa tabów, po tej zmianie będzie używać spacji).

Przy okazji poprawione zostałyby rzeczy typu: białe znaki na końcach
linii, mieszane znaki końca linii (unix/windows, tzn. LF/CRLF), itp.

Co Wy na to?

ANDRZEJ Z.:

Całkiem, całkiem ten standard, a kod trzeba uporządkować bo jest masakra - 
aby się zorientować o co chodzi w kodzie trzeba go sformatować bo bez tego 
jest ciężko.

ŁZA:

Jestem za, zresztą jakiś czas temu proponowałem, żeby przyjąć ten 
standard formatowania (PSR-2) - moim zdaniem jest czytelny i przyjemny dla oka. 
(te spacje zamiast taba prywatnie mi sie nie podobają, ale jak standard to 
standard nie ma co się wdawać w osobiste preferencje)

Proponuję od razu ustawić w konfiguracji swoich edytorów, żeby używały 
spacji w momencie wciśnięcia klawisza tab - większość znanych mi narzędzi 
ma taką opcję w ustawieniach.

WOJTEK:

Ok, zatem commituję taką zmianę dziś wieczorem.

Ponieważ zmieni się praktycznie *każda linia w każdym pliku w
projekcie*, więc proszę wszystkich o commitowanie swoich ewentualnych
lokalnych zmian do wieczora. W przeciwnym razie będziecie mieć bardzo
dużo konfliktów po uruchomieniu "svn up"!

Original issue reported on code.google.com by rygielski on 23 Jan 2014 at 1:41

[GoogleCodeExporter]

Pierwszy wielki reformat poszedł w r5676.

Na razie wątek zamykam, choć może jeszcze do niego wrócę.

Original comment by rygielski on 24 Jan 2014 at 8:44

[GoogleCodeExporter]

Original comment by rygielski on 24 Jan 2014 at 8:44

• Changed state: Done