Skip to content

pr-517/derrickstolee/graph-assert-fix-v2

This is a possible fix for the bug reported in [1].

The first commit fixes the runtime failure due to the assert() statement.

The second commit replaces the assert() statements with a macro that
triggers a BUG().

The third commit adds another test that shows a more complicated example and
how the new code in v2.25.0-rc1 has a behavior change that is not
necessarily wanted.

Thanks, -Stolee

[1]
https://lore.kernel.org/git/CAHt=fUXTHc4JPsapvHKnw5vHhp2cBOYRNfdaSDWBUnKt8fWfeA@mail.gmail.com/

Derrick Stolee (2):
  graph: fix case that hit assert()
  graph: fix lack of color in horizontal lines

 graph.c                      | 17 +++++----
 t/t4215-log-skewed-merges.sh | 71 ++++++++++++++++++++++++++++++++++++
 2 files changed, 80 insertions(+), 8 deletions(-)

base-commit: 8679ef24ed64018bb62170c43ce73e0261c0600a

Submitted-As: https://lore.kernel.org/git/pull.517.v2.git.1578432422.gitgitgadget@gmail.com
In-Reply-To: https://lore.kernel.org/git/pull.517.git.1578408947.gitgitgadget@gmail.com
Assets 2