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.

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

긴 텍스트 (>9K) 버그 수정 #5

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

Conversation

yunhojeon
Copy link

맞춤법 검사 서버가 텍스트 길이가 9천자 좀 넘을 때 500 에러를 리턴합니다. 이를 해결하기 위해, 검사 대상 텍스트를 최대 8천자 이내에서 문장 boundary를 고려하여 split하고, 순차적으로 서버에 검사 요청하도록 수정했습니다.
아울러 검사 결과의 error offset 값이 text내 offset과 맞지 않는 문제를 수정했습니다. 윈도우스의 경우 텍스트 줄바꿈이 \r\n인데, 맞춤법 서버는 내부적으로 모두 \r로 바꾼 후 맞춤법 검사를 수행하고, 바뀐 텍스트 기준으로 error offset을 돌려줍니다. 여기서 발생하는 오차를 해결하기 위해 전송 전에 \n를 ' ' (space)로 치환합니다. 이로써 error descriptor를 적용하는 방식을 더 간단하게 수정할 수 있었습니다.

@yunhojeon yunhojeon requested a review from fallroot January 26, 2020 02:31
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