Reformat existing code. There should be no functional effects.

webrtc/system_wrappers/interface/aligned_array.h

 /*
  *  Copyright (c) 2014 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
@@ skipping 50 matching lines @@
   T& At(int row, int col) {
     CHECK_LE(col, cols_);
     return Row(row)[col];
   }
 
   const T& At(int row, int col) const {
     CHECK_LE(col, cols_);
     return Row(row)[col];
   }
 
-  int rows() const {

[Ben Chan, 2015/06/17 14:48:56]

As a helper class, the rows() and cols() getters seem quite useful. This class
doesn't provide any iterator interface, which means a user of this class likely
needs to memorize the number of rows and columns separately. It seems the
benefit of keeping these two getters outweighs the cost, if any.

[Peter Kasting, 2015/06/17 19:47:29]

No one called these.  I have no opposition to adding them in the future if
someone wants to call them, but if they're simply unused I don't think they
should be publicly visible.

[kjellander_webrtc, 2015/06/18 08:22:46]

Since it seems there's code that uses them outside the WebRTC repo, I think we
should add them back. I made https://codereview.webrtc.org/1178043005/ for that.

-    return rows_;
-  }
-
-  int cols() const {
-    return cols_;
-  }
-
  private:
   int rows_;
   int cols_;
   int alignment_;
   T** head_row_;
 };
 
 }  // namespace webrtc
 
 #endif  // WEBRTC_SYSTEM_WRAPPERS_INTERFACE_ALIGNED_ARRAY_