Merge pull request #4 from krolaw/master
Fixes binaryStringToUtf8 and compiler type mismatch warning in utf8ToBinaryString
This commit is contained in:
		
						commit
						670ac1314b
					
				
							
								
								
									
										4
									
								
								index.js
									
									
									
									
									
								
							
							
						
						
									
										4
									
								
								index.js
									
									
									
									
									
								
							| @ -6,7 +6,7 @@ function utf8ToBinaryString(str) { | |||||||
|   // replaces any uri escape sequence, such as %0A,
 |   // replaces any uri escape sequence, such as %0A,
 | ||||||
|   // with binary escape, such as 0x0A
 |   // with binary escape, such as 0x0A
 | ||||||
|   var binstr = escstr.replace(/%([0-9A-F]{2})/g, function(match, p1) { |   var binstr = escstr.replace(/%([0-9A-F]{2})/g, function(match, p1) { | ||||||
|     return String.fromCharCode('0x' + p1); |     return String.fromCharCode(parseInt(p1,16)); | ||||||
|   }); |   }); | ||||||
| 
 | 
 | ||||||
|   return binstr; |   return binstr; | ||||||
| @ -25,7 +25,7 @@ function utf8ToBase64(str) { | |||||||
| 
 | 
 | ||||||
| function binaryStringToUtf8(binstr) { | function binaryStringToUtf8(binstr) { | ||||||
|   var escstr = binstr.replace(/(.)/g, function (m, p) { |   var escstr = binstr.replace(/(.)/g, function (m, p) { | ||||||
|     var code = p.charCodeAt(p).toString(16).toUpperCase(); |     var code = p.charCodeAt(0).toString(16).toUpperCase(); | ||||||
|     if (code.length < 2) { |     if (code.length < 2) { | ||||||
|       code = '0' + code; |       code = '0' + code; | ||||||
|     } |     } | ||||||
|  | |||||||
		Loading…
	
	
			
			x
			
			
		
	
		Reference in New Issue
	
	Block a user