FYI: coding convention at: https://cwiki.apache.org/confluence/display/CLOUDSTACK/Coding+conventions
The updated column limit is: "File content must be kept within 180 columns" --Sheng On Wed, Nov 20, 2013 at 7:46 AM, Alex Huang <alex.hu...@citrix.com> wrote: > The branch is called reformat if you like to check it. > > --Alex > > > -----Original Message----- > > From: Alex Huang [mailto:alex.hu...@citrix.com] > > Sent: Wednesday, November 20, 2013 7:15 AM > > To: dev@cloudstack.apache.org > > Subject: [MERGE] Reformat of the source code.... > > > > Taking advantage of the CCC happening and 4.3 just being branched, I did > a > > reformat of the entire source base. It converts all tabs to spaces and > rejoin > > lines that were split due to the 80 column limit. I also added > checkstyle to > > maven build. There may be things missing here and there but overall > should > > reduce the number of lines in our checkins. > > > > It created a massive amount of change of course but it all compiles and > starts > > up. Any concerns about met checking this in to master? I'll wait 24 > hours for > > this merge request since we've talked about this before. > > > > --Alex >
