Opened 5 years ago
Closed 5 years ago
#10 closed defect (fixed)
'help' word wrapping problem
| Reported by: | hawke@… | Owned by: | oxygene |
|---|---|---|---|
| Priority: | minor | Milestone: | |
| Component: | FILO | Version: | |
| Keywords: | Cc: | ||
| Dependencies: | Patch Status: | patch has been committed |
Description
The help system seems to have slight problems with word wrapping.
For example, 'help terminal':
terminal: terminal [--no-echo] [--no-edit] [--timeout=SECS] [--lines=LINES] [--
s
...that is, the line beginning 'terminal:' goes out to column 79, and then on column 80 of the next line, the character 's' is printed. It looks like filo is maybe expecting CR or LF to do both?
Attachments (1)
Change History (5)
comment:1 Changed 5 years ago by oxygene
- Owner changed from somebody to oxygene
- Status changed from new to assigned
Changed 5 years ago by oxygene
comment:2 Changed 5 years ago by oxygene
- Patch Status changed from there is no patch to patch needs review
Patch attached. I'll need to figure out why grub_printf("const string") is different than grub_putstr("const string"), but putstr is the faster choice anyway (not that this really matters ;-) )
comment:3 Changed 5 years ago by oxygene
- Patch Status changed from patch needs review to patch has been committed
committed in rev 76
comment:4 Changed 5 years ago by oxygene
- Resolution set to fixed
- Status changed from assigned to closed
Note: See
TracTickets for help on using
tickets.

You're on the right track, I think. Following fixes it for me:
--- main/grub/char_io.c (revision 2062)
+++ main/grub/char_io.c (local)
@@ -935,6 +935,7 @@
+ console_putchar('\r');
There is another issue (the four spaces on the long description lines are not always clean). Once I figured that out, I'll post a patch