Use default `max_lines` when calling `up` & `down`
Description
Both up and down only show 1 source line when called, I think it would be more useful if they provided more context. This removes the max_lines: 1 kwarg when they call show_src. max_lines in show_src will default to CONFIG[:show_src_lines]^1.
Big thanks to @st0012 for showing me and @bquorning where this behavior is defined.
I want to second this proposal as I also often need to chain whereami after moving to a different frame, which IMO is not the best user experience in most cases.
Also probably useful to note that byebug displays more lines by default making it quite a difference switching from byebug to debug:
https://github.com/ruby/debug/assets/47484308/5f2174b9-1264-4753-be71-14203977c69f
Launchable Report
:x: Test session #2907805 failed
details on CI
:bell: no issues :heavy_multiplication_x:1 test failed :heavy_check_mark:63 tests passed
:x: Test session #2907807 failed
details on CI
:bell: no issues :heavy_multiplication_x:1 test failed :heavy_check_mark:63 tests passed
Passed test sessions
:white_check_mark: Test session #2907791 passed
details on CI
:white_check_mark: Test session #2907792 passed
details on CI
:white_check_mark: Test session #2907793 passed
details on CI
:white_check_mark: Test session #2907794 passed
details on CI
:white_check_mark: Test session #2907796 passed
details on CI
:white_check_mark: Test session #2907798 passed
details on CI
:white_check_mark: Test session #2907799 passed
details on CI
:white_check_mark: Test session #2907800 passed
details on CI
:white_check_mark: Test session #2907801 passed
details on CI
:white_check_mark: Test session #2907803 passed
details on CI
:white_check_mark: Test session #2907815 passed
details on CI
:white_check_mark: Test session #2907818 passed
details on CI
:white_check_mark: Test session #2907822 passed
details on CI
:white_check_mark: Test session #2907823 passed
details on CI
:white_check_mark: Test session #2907824 passed
details on CI
:white_check_mark: Test session #2907825 passed
details on CI
:white_check_mark: Test session #2907826 passed
details on CI
:white_check_mark: Test session #2907829 passed
details on CI
:white_check_mark: Test session #2907831 passed
details on CI
:white_check_mark: Test session #2907834 passed
details on CI
:white_check_mark: Test session #2907857 passed
details on CI
Build: refs_pull_1094_merge_b1963e7b72554c9672900ba046c3088598955488
Seems the protocol tests are flaky: https://github.com/ruby/debug/pull/1060
Maybe we need to introduce another configuration.