ggreif edited PR #1989 from correct-dir-index
to main
:
<!--
Please ensure that the following steps are all taken care of before submitting
the PR.
[ ] This has been discussed in issue #..., or if not, please tell us why
here.[ ] A short description of what this does, why it is needed; if the
description becomes long, the matter should probably be discussed in an issue
first.[ ] This PR contains test cases, if meaningful.
- [ ] A reviewer from the core maintainer team has been assigned for this PR.
If you don't know who could review this, please indicate so. The list of
suggested reviewers on the right can help you.Please ensure all communication adheres to the code of conduct.
-->
ggreif edited PR #1989 from correct-dir-index
to main
:
<!--
Please ensure that the following steps are all taken care of before submitting
the PR.
[ ] This has been discussed in issue #..., or if not, please tell us why
here.[ ] A short description of what this does, why it is needed; if the
description becomes long, the matter should probably be discussed in an issue
first.[ ] This PR contains test cases, if meaningful.
- [ ] A reviewer from the core maintainer team has been assigned for this PR.
If you don't know who could review this, please indicate so. The list of
suggested reviewers on the right can help you.Please ensure all communication adheres to the code of conduct.
-->Fixes #1988.
yurydelendik submitted PR Review.
yurydelendik submitted PR Review.
yurydelendik created PR Review Comment:
Based on the comment above, the
directory_index_base
is a better name.nit: can you also make the comment as a sentence: capitalize the "S" and add the period?
yurydelendik edited PR Review Comment.
ggreif submitted PR Review.
ggreif created PR Review Comment:
I was hesitant naming it
_base
, because we have two index bases here. Output has base1
and input can have base0
or1
.
ggreif submitted PR Review.
ggreif created PR Review Comment:
// Since we are outputting DWARF-4, perform base change.
ggreif updated PR #1989 from correct-dir-index
to main
:
<!--
Please ensure that the following steps are all taken care of before submitting
the PR.
[ ] This has been discussed in issue #..., or if not, please tell us why
here.[ ] A short description of what this does, why it is needed; if the
description becomes long, the matter should probably be discussed in an issue
first.[ ] This PR contains test cases, if meaningful.
- [ ] A reviewer from the core maintainer team has been assigned for this PR.
If you don't know who could review this, please indicate so. The list of
suggested reviewers on the right can help you.Please ensure all communication adheres to the code of conduct.
-->Fixes #1988.
yurydelendik has marked PR #1989 as ready for review.
yurydelendik merged PR #1989.
Last updated: Dec 23 2024 at 13:07 UTC