Open dcb314 opened 4 months ago
@llvm/issue-subscribers-lldb
Author: None (dcb314)
Hi!
This issue may be a good introductory issue for people new to working on LLVM. If you would like to work on this issue, your first steps are:
test/
create fine-grained testing targets, so you can e.g. use make check-clang-ast
to only run Clang's AST tests.git clang-format HEAD~1
to format your changes.If you have any further questions about this issue, don't hesitate to ask via a comment in the thread below.
@llvm/issue-subscribers-good-first-issue
Author: None (dcb314)
@JOE1994 can you please assign this task to me?
@vedantbarve I've assigned this to you.
Hi @vedantbarve Are you currently working on it?
I wanted to make my first contribution to LLVM through this PR. Since @aabhinavg had self-assigned it, I took up some other work. I will get free in 3-4 days. @JOE1994 @aabhinavg If you guys are okay with it, can I please complete this issue in a few days?
Extremely sorry for the late reply.
No idea what happened here. I did not actively interact with the linked PR, not is the linked commit there from me. either some weird GitHub issue or a hash collision?
No idea what happened here. I did not actively interact with the linked PR, not is the linked commit there from me. either some weird GitHub issue or a hash collision?
Github indicates that you merged the relevant PR.
No idea what happened here. I did not actively interact with the linked PR, not is the linked commit there from me. either some weird GitHub issue or a hash collision?
Github indicates that you merged the relevant PR.
I pushed a commit directly to master at that point in time. If you look at the referred commit in the linked PR, it's visible that someone else is the author. Odd.
Static analyser cppcheck says:
lldb/source/Core/Debugger.cpp:2069:15: performance: Ineffective call of function 'substr' because a prefix of the string is assigned to itself. Use resize() or pop_back() instead. [uselessCallsSubstr]
Source code is