[Lldb-commits] [PATCH] D128069: [lldb] add SBSection.alignment to python bindings

Jonas Devlieghere via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Fri Jun 17 12:32:10 PDT 2022


JDevlieghere added a comment.

In D128069#3592733 <https://reviews.llvm.org/D128069#3592733>, @dmlary wrote:

> I went through the existing tests for SBSection, and there is only one test case for all the getters & props, and that is for `target_byte_size`.  Based on that lack, and the simplicity of the getter, I didn't think a test case was warranted here.

I'm less concerned about the getter and the property and more about the underlying functionality. It doesn't look like it's being tested, and adding it to the SB API allows to change that.

> If the reviewers feel a test is necessary here, I can definitely add one.

Yes, unless there's a good reason not too, every change should be accompanied by a test.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D128069/new/

https://reviews.llvm.org/D128069



More information about the lldb-commits mailing list