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

RepositoryAccess: Entry occurred twice の警告がでる #134

Open
YoshikiHigo opened this issue Mar 11, 2019 · 7 comments
Open

RepositoryAccess: Entry occurred twice の警告がでる #134

YoshikiHigo opened this issue Mar 11, 2019 · 7 comments

Comments

@YoshikiHigo
Copy link
Member

No description provided.

@YoshikiHigo
Copy link
Member Author

リポジトリに保存されているソースコードがコンパイル可能であることの保証はない.
例えば,以下のようなメソッドの重複が存在する.

https://github.com/netty/netty/blob/e6eb7fa4664b18682be55745d3e0f663b661804b/codec-http/src/main/java/io/netty/handler/codec/spdy/DefaultSpdyHeaders.java#L240-L252

なので,「この警告がでること=FinerGitのバグ」というわけではない.

@sh5i
Copy link
Collaborator

sh5i commented Mar 11, 2019

現在,重複があった場合,先着したものが勝ちます(次着したものは消えます).
警告を出した上で iterator().mjava.2 とかにリネームするということもできますが,ツールの設計としてはどうしましょうか?

@YoshikiHigo
Copy link
Member Author

そうですね.2,3...とsuffixをつけていった方が多くの情報が残りますからそれのほうが良いかもしれませんね.
それと,どのコミットのどのファイルで発生したのかが簡単にわかるとありがたいです.

@sh5i
Copy link
Collaborator

sh5i commented Mar 11, 2019

ファイル名に反映させるということではなく,ログが見づらいという意味でいいですよね(私も感じています,すみません).

@YoshikiHigo
Copy link
Member Author

そうです.ファイル名に反映させるのは変ですよね.

@sh5i
Copy link
Collaborator

sh5i commented Mar 11, 2019

ログをまともにする方法は考えがありますので,少し時間を下さい.

@sh5i
Copy link
Collaborator

sh5i commented Mar 29, 2020

#141 取り込むと、ログは比較的読みやすくなると思います。

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

No branches or pull requests

2 participants