Skip to content
New issue

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

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

Already on GitHub? # to your account

fix fPBr() regex #81

Closed
wants to merge 1 commit into from
Closed

fix fPBr() regex #81

wants to merge 1 commit into from

Conversation

mkdgs
Copy link

@mkdgs mkdgs commented Sep 28, 2012

a very big text on one line (~10 000 chars) causes internal server error. It will be significantly faster

a very big text on one line (~10 000 chars) causes internal server error. It will be significantly faster
@netcarver netcarver closed this in f2a98cd Oct 9, 2012
@netcarver
Copy link
Contributor

Thanks Mickael. This is in the new dev branch and will be part of the next release.

@netcarver
Copy link
Contributor

Hello Mickael. Although this issue is now closed, I wonder if you might be able to supply me with a file of the text that was causing your original parse failure? The reason I'm asking is because I have another change to make to the fPBr() regex and I want to make sure it doesn't re-introduce the problem.

@netcarver
Copy link
Contributor

Hang on, missed the "on one line" part -- I can generate that myself, sorry to have bothered you.

@netcarver
Copy link
Contributor

I can't reproduce this even with ~50 000 characters on a line.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants