Update ktlint to 0.31.0.

build.gradle

@@ -232,7 +232,7 @@ configurations {
 }
 
 dependencies {
-    ktlint "com.github.shyiko:ktlint:0.29.0"
+    ktlint "com.github.shyiko:ktlint:0.31.0"
 }
 
 task ktlint(type: JavaExec, group: "verification") {

components/browser/engine-system/src/main/java/mozilla/components/browser/engine/system/matcher/ReversibleString.kt

@@ -41,9 +41,11 @@ abstract class ReversibleString private constructor(
             ReverseString(string, offsetStart, offsetEnd)
         }
 
-    private class ForwardString(string: String, offsetStart: Int, offsetEnd: Int)
-        : ReversibleString(string, offsetStart, offsetEnd
-    ) {
+    private class ForwardString(
+        string: String,
+        offsetStart: Int,
+        offsetEnd: Int
+    ) : ReversibleString(string, offsetStart, offsetEnd) {
         override val isReversed: Boolean = false
 
         override fun charAt(position: Int): Char {
@@ -58,9 +60,11 @@ abstract class ReversibleString private constructor(
         }
     }
 
-    private class ReverseString(string: String, offsetStart: Int, offsetEnd: Int)
-        : ReversibleString(string, offsetStart, offsetEnd
-    ) {
+    private class ReverseString(
+        string: String,
+        offsetStart: Int,
+        offsetEnd: Int
+    ) : ReversibleString(string, offsetStart, offsetEnd) {
         override val isReversed: Boolean = true
 
         override fun charAt(position: Int): Char {

components/browser/icons/src/test/java/mozilla/components/browser/icons/decoder/ICOIconDecoderTest.kt

@@ -87,7 +87,7 @@ class ICOIconDecoderTest {
 
         entries
             .sortedBy { entry -> entry.width }
-            . forEachIndexed { index, entry -> assertEquals(expectedEntries[index], entry) }
+            .forEachIndexed { index, entry -> assertEquals(expectedEntries[index], entry) }
     }
 
     @Test
@@ -114,7 +114,7 @@ class ICOIconDecoderTest {
 
         entries
             .sortedByDescending { entry -> entry.width }
-            . forEachIndexed { index, entry -> assertEquals(expectedEntries[index], entry) }
+            .forEachIndexed { index, entry -> assertEquals(expectedEntries[index], entry) }
     }
 
     private fun loadIcon(fileName: String): ByteArray =

components/feature/sync/src/main/java/mozilla/components/feature/sync/BackgroundSyncManager.kt

@@ -78,8 +78,7 @@ private val registry = ObserverRegistry<SyncStatusObserver>()
  * - periodic sync is requested to stop
  */
 @SuppressWarnings("TooManyFunctions")
-abstract class GeneralSyncManager
-: SyncManager, Observable<SyncStatusObserver> by registry, SyncStatusObserver {
+abstract class GeneralSyncManager : SyncManager, Observable<SyncStatusObserver> by registry, SyncStatusObserver {
     companion object {
         // Periodically sync in the background, to make our syncs a little more incremental.
         // This isn't strictly necessary, and could be considered an optimization.

components/service/firefox-accounts/src/main/java/mozilla/components/service/fxa/FirefoxAccount.kt

@@ -42,8 +42,10 @@ class FirefoxAccount internal constructor(private val inner: InternalFxAcct) : O
      * Note that it is not necessary to `close` the Config if this constructor is used (however
      * doing so will not cause an error).
      */
-    constructor(config: Config, persistCallback: PersistCallback? = null)
-            : this(InternalFxAcct(config, persistCallback))
+    constructor(
+        config: Config,
+        persistCallback: PersistCallback? = null
+    ) : this(InternalFxAcct(config, persistCallback))
 
     override fun close() {
         job.cancel()