logo       

Re: LYMainLoop.c [PATCH]: msg#00026

web.lynx.devel

Subject: Re: LYMainLoop.c [PATCH]

On Sat, 12 Aug 2006, Doug Kaufman wrote:

There was a change made to LYMainLoop.c with 2.8.6dev.1 (which wasn't
included in the patch file for that version) which broke the formatting

I'm confused - perhaps you mean to say that one of the patches suggested around that time had a \r%s in it. I have a local copy of the dev.x stuff and don't see that string in any of the revisions of LYMainLoop.c

Here's a grep through the differences for SPAWNING_MSG for instance
(the leading +/- of course reflect changes):

printf("%s\r\n", SPAWNING_MSG);
+ printf("%s\r\n", SPAWNING_MSG);
- printf("%s\r\n", SPAWNING_MSG);
printf("%s\r\n", SPAWNING_MSG);
printf("%s\r\n", SPAWNING_MSG);
- printf(SPAWNING_MSG);
+ printf("%s\r\n", SPAWNING_MSG);
printf(SPAWNING_MSG);
printf(SPAWNING_MSG);
printf(SPAWNING_MSG);
printf(SPAWNING_MSG);
printf(SPAWNING_MSG);

of the spawning message. I suspect that this was done accidentally. At
least with DOS and MingW the spawning message now starts at the right
side of the screen and wraps. The message used to start with \r\n, but
probably just a \r is sufficient.
Doug


--- lynx2.8.6dev.18/lynx2-8-6/src/LYMainLoop.c.ori 2006-05-29
14:44:54.000000000 -0800
+++ lynx2.8.6dev.18/lynx2-8-6/src/LYMainLoop.c 2006-08-10 23:36:40.000000000
-0800
@@ -4183,7 +4183,7 @@
{
if (!no_shell) {
stop_curses();
- printf("%s\r\n", SPAWNING_MSG);
+ printf("\r%s\r\n", SPAWNING_MSG);
#if defined(__CYGWIN__)
/* handling "exec $SHELL" does not work if $SHELL is null */
if (LYGetEnv("SHELL") == NULL) {

--
Doug Kaufman
Internet: dkaufman@xxxxxxxxx



_______________________________________________
Lynx-dev mailing list
Lynx-dev@xxxxxxxxxx
http://lists.nongnu.org/mailman/listinfo/lynx-dev


--
Thomas E. Dickey
http://invisible-island.net
ftp://invisible-island.net


_______________________________________________
Lynx-dev mailing list
Lynx-dev@xxxxxxxxxx
http://lists.nongnu.org/mailman/listinfo/lynx-dev



<Prev in Thread] Current Thread [Next in Thread>
Google Custom Search

News | FAQ | advertise