That would work fine. The <!-- works fine within the standard document because the js enabled browsers discard it. Dynamically loaded scripts are regex'ed and evaled so they have to be valid JS.

I could strip them out, but I'd rather not add the overhead if it isn't needed. You may want to override that method and add in the 1 liner to strip the "<!--" if you can't remove it from the JS source.