Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

bug fix & enhancements #72

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

bug fix & enhancements #72

wants to merge 7 commits into from

Conversation

orca-zhang
Copy link

  1. fix clean redundant centers error：findpat.js
  2. support more encoding
  3. grammar optimization

@orca-zhang
grammar optimization
a094653 
post-increment to pre-increment if possible, simple array declaration
@orca-zhang
fix clean redundant centers error
b53427b 
suffix change error after deleting, fix no rest centers size limit error
@orca-zhang
remove new operations of Array
51e9578 
new ops can lead to low performance
@orca-zhang
grammar optimization
1f094b9 
remove new operations of Array, quick init ops of Array
@orca-zhang
support more encoding
af87bda 
support utf8 & sjis & hanzi encoding
@orca-zhang
fix Array bugs
e3d8d74
jasj
jasj approved these changes Oct 10, 2017
View reviewed changes
Copy link

@jasj jasj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

better way to handle array, thanks for your contribution

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

@jasj jasj jasj approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@orca-zhang @jasj