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

Refactor string operation #125

Merged
merged 6 commits into from Sep 29, 2019
Merged

Conversation

ma8ma
Copy link
Collaborator

@ma8ma ma8ma commented Sep 28, 2019

玉葱版からのパッチです。
文字列操作の細かなリファクタリングを行います。

@ma8ma ma8ma added the refactoring コードの整理 (バグ修正でも機能変更でもない) label Sep 28, 2019
@ma8ma ma8ma added this to In progress in State of JDim-v0.2.0+ via automation Sep 28, 2019
JD Project added 6 commits September 29, 2019 16:46
URL判定にstd::stringの一時変数を使わないように修正する。
連続したヌル文字をmemsetを使って置き換えるよう修正する。
リダイレクト判定にstd::stringの一時変数を使わないように修正する。
初期化時に代入ではなく条件を満たした場合に代入するように修正する。
whileループからbreakで抜けるように修正する。
@ma8ma ma8ma merged commit 8a34e78 into JDimproved:master Sep 29, 2019
State of JDim-v0.2.0+ automation moved this from In progress to Done Sep 29, 2019
@ma8ma ma8ma deleted the refactor-string-operation branch September 29, 2019 10:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactoring コードの整理 (バグ修正でも機能変更でもない)
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

1 participant