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

Protect i64 #13

Merged
merged 1 commit into from
Mar 9, 2017
Merged

Protect i64 #13

merged 1 commit into from
Mar 9, 2017

Conversation

YanagiEiichi
Copy link
Contributor

@lujjjh
Copy link
Contributor

lujjjh commented Mar 9, 2017

i64 确实会有问题……

不过这种解决方案可能会导致字符串里的 \uXXXX 出问题，我先合了发个版本，到时候再想想怎么处理好了。

@lujjjh lujjjh merged commit 8767bd4 into ElemeFE:master Mar 9, 2017
@lujjjh
Copy link
Contributor

lujjjh commented Mar 9, 2017

可能可以换成 \d{5,}，不太确定会不会有别的副作用。

@lujjjh
Copy link
Contributor

lujjjh commented Mar 9, 2017

想了想也可能会有，还是老老实实排除掉字符串好了。

@YanagiEiichi
Copy link
Contributor Author

转义字符的处理确实有坑。。改一下正则应该就能解决。。

@YanagiEiichi
Copy link
Contributor Author

@lujjjh #14

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@YanagiEiichi @lujjjh