On 2/28/13 10:49 AM, bscSCORM wrote:
>> @@ -76,7 +76,42 @@
>>   		
>>
>>           <script type="text/javascript">
>> -
>> +
>> +        /*
>> +         * Make JSON safe for IE6
>> +         * https://developer.mozilla.org/en/JavaScript/Reference/Global_Objects/JSON#Browser_compatibility
>> +        */
>> +        if (!window.JSON) {
>> +            window.JSON = {
>> +                parse: function (sJSON) {
>> +                    /*jslint evil: true */
>> +                    return eval("(" + sJSON + ")");
>
> Do we have to use eval() here? If so, is it worth it (might it be better just to fail in IE6?)
>

Normally I'd be inclined to avoid `eval` at all costs, but in so far as 
it is in the code linked on the mozilla.org site I'm okay with it. (Also 
like that in TinCanJS directly, which we don't load on this page.) I'm 
not sure avoiding eval in the case that window.JSON isn't defined is 
worth having complete breakage in IE6 and 7 (as it turns out that 
comment is wrong).

-- 
Brian J. Miller
Rustici Software
brian.miller@scorm.com