Re: [PATCH 2/2] revision: implement --show-linear-break for --graph

2 messages, 2 authors, 2021-01-19 · open the first message on its own page

Re: [PATCH 2/2] revision: implement --show-linear-break for --graph

From: Junio C Hamano <hidden>
Date: 2021-01-18 21:10:37

Kyle Marek [off-list ref] writes:
Me too, but I think a user-defined mark needs to be a string to
support Unicode characters.
Ahh, I didn't even consider making it user-defined.

As it seems a lot safer to make this an optional feature, it does
sort-of make sense to let the letters used for root & left-root be
customizable, and it does make sense to take a multi-byte character,
but I am not sure what implications it has if we allowed any string
without ensuring that it occupies one display column.

Re: [PATCH 2/2] revision: implement --show-linear-break for --graph

From: Kyle Marek <hidden>
Date: 2021-01-19 07:48:18

On 1/18/21 4:01 PM, Junio C Hamano wrote:
Kyle Marek [off-list ref] writes:
quoted
Me too, but I think a user-defined mark needs to be a string to
support Unicode characters.
Ahh, I didn't even consider making it user-defined.

As it seems a lot safer to make this an optional feature, it does
sort-of make sense to let the letters used for root & left-root be
customizable, and it does make sense to take a multi-byte character,
but I am not sure what implications it has if we allowed any string
without ensuring that it occupies one display column.
Does git, or a dependency library, have the ability to interpret TERM 
and locale to determine on-screen character count/size?

If not, maybe let users use multi-character strings, but call it misuse 
of the option that will mess offset that row of the --graph output until 
we have something to determine on-screen size.

-- 

-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-
-                                                               -
- Kyle Marek                        PD Inc. http://www.pdinc.us -
- Jr. Developer                     10 West 24th Street #100    -
- +1 (443) 269-1555 x361            Baltimore, Maryland 21218   -
-                                                               -
-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help