search

stratosphere / incubator-systemml

Mirror of Apache SystemML (Incubating)
Apache License 2.0
1 stars 4 forks source link

indentation #20

Closed FelixNeutatz closed 8 years ago

FelixNeutatz commented 8 years ago

I recognized that some of the classes use tabs for indentation and other spaces. We should use tabs according to the standard :)

fschueler commented 8 years ago

The formatting in the whole project is pretty inconsistent in general. I think we should leave it how it is for the existing classes and for files that we create, decide whether we want tabs or spaces. I am a big fan of spaces compared to tabs but we can discuss that ;)

Or we just go with the inconsistency that exists...

FelixNeutatz commented 8 years ago

let's discuss it on Monday :)

akunft commented 8 years ago

Lets decide this when we have the PR candidate and then do one pass over all our code.

fschueler commented 8 years ago

So it is settled: Tabs it is! :unamused: