[bercedev]

Summary

A $ at the end of a logo line was treated as the start of a $N color code. The index check failed, the $ was printed, and then the newline after it was copied into the line, so the logo lost a line break and the padding of the following lines went off. A $ as the very last character also skipped past the terminating \0 and kept reading whatever was after the string. Now a $ followed by a line end or the end of the logo is printed as a plain dollar and the line end is left to the normal handling.

Related issue (required for new logos for new distros)

Fixes #2666

Changes

• src/logo/logo.c: in the $ handling, a following \n, \r or \0 is not consumed.
• tests/logo.c: ab$\ncd keeps its line break (height 1, width 3), ab$ stops at the terminator (width 3), and a$1b$$c is unchanged.

Screenshots

No screenshot; checked through the logo test. Before the fix the same inputs gave height 0 / width 4 for ab$\ncd and width 5 for ab$.

Checklist

• I have tested my changes locally.