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

Fix 120 #121

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

Fix 120 #121

wants to merge 2 commits into from

Conversation

agrexgh
Copy link

@agrexgh agrexgh commented Jun 3, 2020

インライン表示できてなかった箇所を修正いたしました。
漏れをなくす方法が当方では思い浮かばず、
できる限り拾って修正をさせていただく形をとりました。
修正点のレビューをお願いいたします。


なお、手順の検証ができるように、
検出の手順を記載しておきます。

  • インラインの前後に半角スペースが日本語訳文に埋め込まれてなかった
    (半角スペースがなくても、とかなどの記号の場合は問題なかった)。
    問題箇所を拾うために使った正規表現

    (.(?<![. ,()*\[\]:。「」、()])`[^ ][^`]*?[^ ]`|`[^ ][^`]*?[^ ]`(?![. ,()*\[\]:。「」、()]))
    

    ※正規表現がうまく作れておらず、
    拾った箇所がすべて問題のある個所というわけではなく、問題ない箇所も拾います。
    その中から問題になりそうなところを目視確認、修正していきました。

  • インラインの ` が漏れていた箇所があった。
    行単位で奇数個になっている行を拾って修正。
    上記を拾うために使った正規表現は以下

    '^[^`]*?([^`]*`[^`]*`[^`]*)*?`[^`]*?($)'
    

@agrexgh
Copy link
Author

agrexgh commented Jun 3, 2020

紐づけを失念しておりましたので紐づけいたします。
#120

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.

2 participants