Skip to content

Comments

fixed: some variable is not defined#95

Open
joe223 wants to merge 1 commit intotravist:masterfrom
joe223:master
Open

fixed: some variable is not defined#95
joe223 wants to merge 1 commit intotravist:masterfrom
joe223:master

Conversation

@joe223
Copy link

@joe223 joe223 commented Feb 8, 2017

change:

  1. Use Strict Mode.
  2. It works well on Node.js & 微信小程序 enviroment.

eg:

// jsencrypt/lib/jsbn/base64.js
function b64tohex(s) {
  var ret = ""
  var i;
  var k = 0; // b64 state, 0-3
  var slop;
  var v; // new code: v is not defined
  for(i = 0; i < s.length; ++i) {
    if(s.charAt(i) == b64pad) break;
    v = b64map.indexOf(s.charAt(i));
    ...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant