Changeset 19591 for platform

Show
Ignore:
Timestamp:
09/19/08 13:00:07 (6 years ago)
Author:
drry
Message:
  • fixed regexes.
  • fixed missing vars.
Location:
platform/firefox/FireMobileSimulator/trunk/src
Files:
2 modified

Legend:

Unmodified
Added
Removed
  • platform/firefox/FireMobileSimulator/trunk/src/chrome/content/common/util.js

    r19579 r19591  
    131131 */ 
    132132function dms2degree(dms){ 
    133         dms.match(/[+-]?(\d+)\.(\d+)\.(\d+\.\d+)/); 
    134         dms1 = parseInt(RegExp.$1); 
    135         dms2 = parseInt(RegExp.$2); 
    136         dms3 = parseFloat(RegExp.$3); 
     133        if (!/[-+]?(\d+)\.(\d+)\.(\d+\.\d+)/.test(dms)) { 
     134                return null; 
     135        var dms1 = parseInt(RegExp.$1); 
     136        var dms2 = parseInt(RegExp.$2); 
     137        var dms3 = parseFloat(RegExp.$3); 
    137138        var degree = dms1+dms2/60+dms3/3600; 
    138139        return degree; 
  • platform/firefox/FireMobileSimulator/trunk/src/components/msimStreamConverter.js

    r19583 r19591  
    136136        var data = si.read(aCount); 
    137137 
    138         if ((this.charset == undefined || this.charset == '') 
    139                         && data 
    140                                         .match(/^<\?xml(?:\s[^>]*?)?\sencoding\s*=\s*["']([^"']*)["']|<meta(?:\s[^>]*?)?\s(?:http-equiv\s*=\s*["']content-type['"](?:\s[^>]*?)?\scontent\s*=\s*["']([^"']*)["']|content\s*=\s*["']([^"']*)["'](?:\s[^>]*?)?(?:\shttp-equiv\s*=\s*["']content-type['"]))/i)) { 
    141                 var m = RegExp.$1 || RegExp.$2 || RegExp.$3; 
    142                 dump("[msim]data match regexp:" + m + "\n"); 
    143                 if (m.match(/;\s*charset=([^\s;]+)/i)) { 
    144                         var charset = RegExp.$1; 
    145                         this.charset = charset; 
    146                         dump("[msim]guessed charset is " + charset + "\n"); 
    147                 } else { 
     138        var m; 
     139        if (this.charset == undefined || this.charset == '') { 
     140                if (/^<\?xml(?:\s[^>]*?)?\sencoding\s*=\s*["']([^"']*)["']|<meta(?:\s[^>]*?)?\s(?:http-equiv\s*=\s*(["']?)content-type\2(?:\s[^>]*?)?\scontent\s*=\s*["']?[^;]+(?:;[^;=]+(?:=\s*[^\s;]*)?)*?;\s*charset\s*=\s*([^"'\s;<>]+)|content\s*=\s*(["']?)[^;]+(?:;[^;=]+(?:=\s*[^\s;]*)?)*?;\s*charset\s*=\s*([^"'\s;<>]+)[^"']*?\4(?:\s[^>]*?)?\shttp-equiv\s*=\s*(["']?)content-type\6)/i.test(data)) { 
     141                        m = RegExp.$1 || RegExp.$3 || RegExp.$5; 
    148142                        this.charset = m; 
    149143                        dump("[msim]guessed charset is " + m + "\n"); 
     144                } else { 
     145                        dump("[msim]No encoding match found"); 
    150146                } 
    151147        } else { 
    152                 dump("[msim]No encoding match found (or already got charset: " + this.charset 
     148                dump("[msim]Already got charset: " + this.charset 
    153149                                + ")\n"); 
    154150        }