Feature/#342 #344
Feature/#342 #344
Conversation
|
テストについてですがaclがどこかに存在していないと書きづらいなと思いました。 ・サブモジュールとして登録する のどちらかがよさそうでしょうか。 最近すこし忙しくテストの追加はもしかしたら遅くなってしまうかもです。 |
|
@kjnh10 プルリクありがとうございます。しかし先に #342 (comment) のコメントへの返事がほしいです。 |
|
いや、すでにコメントの編集という形で返事があったみたいですね。編集だとまったく通知が来ないので見逃していました。 |
|
なるほど。上にあった方が分かりやすいかなと思ったのですがその場合でも編集した旨を新規メッセージに書いた方がよさそうということですね。 |
if this option is set, the lines like below is also expanded. #include <atcoder/....> #include "atcoder/....."
|
@kjnh10 ひとまず先に issue で議論をしましょう。 |
|
(コミュニケーションミス連発って感じで申し訳ない 🙇) |
|
いえいえ。こちらこそです。 |
|
AtCoder Library 側で解決してもらったのでこのプルリクは閉じます。せっかく書いてもらったのに申し訳ないです。 今回は「外部のライブラリを直す選択肢がある」という複雑なケース (設計判断として難しい + 相手との折衝が必要となる) のが難しさで、このために色々な混乱があったように思います。次からはもう少しうまくやろうと思います。 |
|
いえいえ。対応、修正いただきありがとうございました🎉 |
@kmyk
pull requestを作成してみました。
oj-bundle -I [path to acl] --expand_acl
と渡すと#include <atcoder/****>形式のものは再帰的に展開するように変更しました。
フラグなしの実行の場合は挙動が変化しないようにしたつもりです。
フラグをつけた場合は下記の①、②、③、④全てに対して正しく展開されるはずです。(mathを例にしてますが特に意味はないです。）
①#include <atcoder/math>
②#include "atcoder/math"
③#include <atcoder/math.hpp>
④#include "atcoder/math.hpp"
（①、②、③に関してはフラグなしだと展開されない。④に関してはフラグなしだと部分的に展開される。）