trivial patch for compiler errors

120 views
Skip to first unread message

John Munro

unread,
May 24, 2013, 3:01:03 PM5/24/13
to closure-lib...@googlegroups.com
I'm getting these errors from the compiler (with VERBOSE):
_closure-library\closure\goog\events\eventhandler.js:180: WARNING - Bad type annotation. Unknown type goog.events.EventWrapper
 * @param {goog.events.EventWrapper} wrapper Event wrapper to use.
           ^
_closure-library\closure\goog\events\eventhandler.js:248: WARNING - Bad type annotation. Unknown type goog.events.EventWrapper
 * @param {goog.events.EventWrapper} wrapper Event wrapper to use.
           ^
_closure-library\closure\goog\events\events.js:367: WARNING - Bad type annotation. Unknown type goog.events.EventWrapper
 * @param {goog.events.EventWrapper} wrapper Event wrapper to use.
           ^
_closure-library\closure\goog\events\events.js:507: WARNING - Bad type annotation. Unknown type goog.events.EventWrapper
 * @param {goog.events.EventWrapper} wrapper Event wrapper to use.
           ^
_closure-library\closure\goog\events\events.js:959: WARNING - Bad type annotation. Unknown type goog.debug.ErrorHandler
 * @param {goog.debug.ErrorHandler} errorHandler Error handler with which to
           ^
_closure-library\closure\goog\net\xhrio.js:327: WARNING - Bad type annotation. Unknown type goog.debug.ErrorHandler
 * @param {goog.debug.ErrorHandler} errorHandler Error handler with which to
           ^
To fix them I just added goog.require('goog.events.EventWrapper'); to eventhandler.js and goog.require('goog.debug.ErrorHandler'); to events.js - diff attached.
missing requires.diff

Hochhaus, Andrew

unread,
May 24, 2013, 3:05:26 PM5/24/13
to closure-lib...@googlegroups.com

John Munro

unread,
May 24, 2013, 3:27:29 PM5/24/13
to closure-lib...@googlegroups.com
Oh, sorry about that - just trying to be a good citizen.  Thanks for the pointer.

Hochhaus, Andrew

unread,
May 24, 2013, 3:36:05 PM5/24/13
to closure-lib...@googlegroups.com
I was confused about the error the first time I saw it too. :)

-Andy
Reply all
Reply to author
Forward
0 new messages