374121 - Strict warnings in calendarWindow.js, calendar-month-view.xml, calGoogleCalendar.js, calGoogleSession.js

Status: RESOLVED FIXED

(Calendar :: General, defect)

Description

[Stefan Sitter [:ssitter]]

Attachment: rev0 - fix strict warnings

There are some strict warnings in calendarWindow.js, calendar-month-view.xml, calGoogleCalendar.js and calGoogleSession.js:

Warning: trailing comma is not legal in ECMA-262 object initializers
Source File: chrome://calendar/content/calendarWindow.js
Line: 78, Column: 3
Source Code:
   };

-> Solution: remove trailing comma

Warning: anonymous function does not always return a value
Source File: chrome://calendar/content/calendar-month-view.xml
Line: 127, Column: 12
Source Code:
            }

-> Solution: from looking at the .idl and the Thunderbird implementation there is no need to return anything from getFlavorData() -> remove return

Warning: function cGC_general_response does not always return a value
Source File: file:///C:/[...]/calGoogleCalendar.js
Line: 806, Column: 4
Source Code:
    }

-> Solution: return null in error case

Warning: test for equality (==) mistyped as assignment (=)?
Source File: file:///C:/[...]/calGoogleSession.js
Line: 386, Column: 56
Source Code:
                while (request = this.mItemQueue.shift()) {

-> Solution: add another pair of braces

Comment 1

[Matthew (lilmatt) Willis]

Comment on attachment 258707 [details] [diff] [review]
rev0 - fix strict warnings

>Index: mozilla/calendar/providers/gdata/components/calGoogleCalendar.js
>===================================================================
>             // Operation failed
>             aListener.onOperationComplete(this,
>                                           e.result,
>                                           aOperation,
>                                           null,
>                                           e.message);
>         }
>+        return null;
Add a comment here like "Returning null to avoid js strict warning"



>Index: mozilla/calendar/providers/gdata/components/calGoogleSession.js
>===================================================================
>                 // Process Items that were requested while logging in
>                 var request;
>-                while (request = this.mItemQueue.shift()) {
>+                while ((request = this.mItemQueue.shift())) {
Add a comment here like "Extra parentheses to avoid js strict warning"

r=lilmatt with those. (I'll address them on checkin for you) ;)

Comment 2

[Matthew (lilmatt) Willis]

Patch checked in on MOZILLA_1_8_BRANCH and trunk.

-> FIXED