LinkedInAttic/inject

Safari "Parse error"

jimmyhchan opened this issue · 3 comments

jimmyhchan commented

Safari throws a "parse error"

<!DOCTYPE html>

<html lang="en">
  <head>
    <title>Page Title</title>
    <meta charset="utf-8">
    <script src="inject.js"></script>
  </head>

  <body>
    <div id="container">

    </div>
  </body>
</html>
jakobo commented

What version are you running? (check the Inject.version value) 0.4.0 and earlier have a trivial error used to record error offsets from the old debugging system. 0.4.1 RC1 and newer shouldn't be exhibiting this anymore.

jimmyhchan commented

Sorry
Inject 0.4 final
Safari 5.0.5 safari 5.1.7
On Jan 25, 2013 7:08 PM, "Jakob Heuser" notifications@github.com wrote:

What version are you running? (check the Inject.version value) 0.4.0 and
earlier have a trivial error used to record error offsets from the old
debugging system. 0.4.1 RC1 and newer shouldn't be exhibiting this anymore.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/223#issuecomment-12730099.

jakobo commented

This is definitely fixed in 0.4.1 for all RCs. Root cause was a method to simulate an error in order to improve debugging accuracy. (Suffice to say, it wasn't perfect). If you need to manually edit the file, it's at the top of the Executor.js closure.

The error itself is innocuous, but for people trying to debug, it's certainly a red herring.