1170409 - ESLint: Reconfigure dot-location

Status: RESOLVED FIXED

(DevTools :: General, defect)

Description

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

The current setting of:

"dot-location": [1, "object"],

warns for the common case of XPCOM patterns:

let chromeWindow = aWindow.QueryInterface(Ci.nsIInterfaceRequestor)
                          .getInterface(Ci.nsIWebNavigation)

Either it should be set to: 

"dot-location": [1, "property"],

or just disabled.

Comment 1

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

Patrick, what do you think?

Comment 2

[Patrick Brosset <:pbro>]

I agree, I thought I had changed it already, but that might have been in a commit I haven't landed yet.

Comment 3

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

Attachment: 0001-Bug-1170409-ESLint-dot-location-property.-r-pbrosset.patch

Comment 4

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/ce72fec8fe5a

Comment 5

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

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