{"comment":{"url":"https://api.github.com/repos/readium/readium-cfi-js/comments/13116777","html_url":"https://github.com/readium/readium-cfi-js/commit/44b70a57bd058b81f2cb38e4ed0495c089737f8a#commitcomment-13116777","id":13116777,"user":{"login":"JCCR","id":5132652,"avatar_url":"https://avatars.githubusercontent.com/u/5132652?v=3","gravatar_id":"","url":"https://api.github.com/users/JCCR","html_url":"https://github.com/JCCR","followers_url":"https://api.github.com/users/JCCR/followers","following_url":"https://api.github.com/users/JCCR/following{/other_user}","gists_url":"https://api.github.com/users/JCCR/gists{/gist_id}","starred_url":"https://api.github.com/users/JCCR/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/JCCR/subscriptions","organizations_url":"https://api.github.com/users/JCCR/orgs","repos_url":"https://api.github.com/users/JCCR/repos","events_url":"https://api.github.com/users/JCCR/events{/privacy}","received_events_url":"https://api.github.com/users/JCCR/received_events","type":"User","site_admin":false},"position":2,"line":1,"path":"gen/cfi_parser_gen.js","commit_id":"44b70a57bd058b81f2cb38e4ed0495c089737f8a","created_at":"2015-09-08T16:39:12Z","updated_at":"2015-09-08T16:39:12Z","body":"@danielweck \r\nOh yes, I forgot this was auto generated. Thank you for taking a look at this. \r\nI added this when I was integrating a new test framework for readium-js: https://github.com/readium/readium-js/blob/feature/intel-contrib/spec/reader/reader.html\r\nIn that bootstrapper it uses a \"no optimize\" configuration. I ran into an issue that when this file is shimmed by requirejs it wasn't capturing the exported variable properly, adding it as an explicit window property fixed that.\r\n\r\nIn retrospect I might not have hit that issue if I had the same script block from what you pointed out here: https://github.com/readium/readium-cfi-js/blob/master/dev/index_NO-RequireJS.html#L7"}}