YahooArchive / boomerang

End user oriented web performance testing and beaconing
http://lognormal.github.com/boomerang/doc/
Other
1.18k stars 448 forks source link

Console output doesn't work for iPhone/iPad #44

Closed solarice closed 12 years ago

solarice commented 12 years ago

I get a warning here:

else if(typeof console !== "undefined" && typeof console.log !== "undefined") { boomr.log = function(m,l,s) { console.log(s + ": [" + l + "] ", m); }; }

patched temporary by replacing the last comma with a plus operator within the console.log function reducing the number or args to 1. I don't know the best way to fix. Any suggestions?

bluesmoon commented 12 years ago

That fix is probably sufficient. console.log is supposed to concatenate multiple arguments with a space between them, and coerce all of them into strings. Since all arguments are strings anyway, this is the same as just adding a +

solarice commented 12 years ago

Ok great! Thanks.