781348 - service.js style improvements

Status: RESOLVED FIXED

(Firefox :: Sync, defect)

Description

[Gregory Szorc [:gps]]

Attachment: Fix minor style nits in service.js, v1

I was looking at service.js and OH GOD MY EYES ARE ON FIRE. This makes some of the burning go away.

Comment 1

[Richard Newman [:rnewman]]

Comment on attachment 650319 [details] [diff] [review]
Fix minor style nits in service.js, v1

Review of attachment 650319 [details] [diff] [review]:
-----------------------------------------------------------------

::: services/sync/modules/service.js
@@ +385,5 @@
>        engines = pref.split(",");
>      }
>  
>      // Grab the actual engines and register them
> +    Engines.register(engines.map(function onItem(name) Weave[name + "Engine"]));

Also add { return }.

@@ +774,5 @@
>      }
>    },
>  
> +  changePassword: function changePassword(newpass)
> +    this._notify("changepwd", "", function onNotify() {

While you're here breaking blame: { return }.

@@ +796,5 @@
>        return true;
>      })(),
>  
> +  changePassphrase: function changePassphrase(newphrase)
> +    this._catch(this._notify("changepph", "", function onNotify() {

Likewise.

@@ +868,5 @@
>    },
>  
>    login: function login(username, password, passphrase)
>      this._catch(this._lock("service.js: login",
> +          this._notify("login", "", function onNotify() {

Oh, the indenting!

Comment 2

[Gregory Szorc [:gps]]

Attachment: Fix minor style nits in service.js, v2

I want a 2nd set of eyes on the change to login(). The code makes my head hurt.

Comment 3

[Richard Newman [:rnewman]]

Comment on attachment 650353 [details] [diff] [review]
Fix minor style nits in service.js, v2

Review of attachment 650353 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good to me. If existing tests pass will tell for sure :)

Comment 4

[Gregory Szorc [:gps]]

https://hg.mozilla.org/services/services-central/rev/e1c2f8982a9d

Comment 5

[Gregory Szorc [:gps]]

https://hg.mozilla.org/mozilla-central/rev/e1c2f8982a9d