Re: lynx-dev [PATCH 2.8.3.dev6] fixes for psrc mode

[Vlad Harchev]

On Sat, 14 Aug 1999, Vlad Harchev wrote:

> On Mon, 16 Aug 1999, Leonid Pauzner wrote:
>[...] 
>  As for 'NCSAMosaic' - this problem seems to be caused by the fact, that 
> GridTextc:split_line removes trailing whitespaces (but why it didn't happen in
> plain src mode? - I thought it won't remove anything trailing if
> LY_SOFT_NEWLINE is at the end.. ). I'll inspect this.

 I found where it was. Here is a patch to 'split_line' to fix this. You can
assure in that in the following way:
 Compose the following file:
<ul>
<li><a href="http://www.ncsa.uiuc.edu/SDG/Software/Mosaic/MetaIndex.html";>NCSA 
Mosaic</a>
</ul>

 Try to print this before applying the patch. The space between NCSA and
Mosaic will be lost.
 Now try printing the following file:

<ul>
<li>|a href="http://www.ncsa.uiuc.edu/SDG/Software/Mosaic/MetaIndex.html";|NCSA 
Mosaic</a>
</ul>

 The space won't be lost, since the 'text->firstanchor' is NULL - there are
no anchors in the 2nd file.
 So, this patch really fixes the problem (at least for me). Can you try it
with your lynx?

>[...] 

 Best regards,
  -Vlad

diff -ru old/src/GridText.c fixed/src/GridText.c
--- old/src/GridText.c  Fri Aug 13 07:13:22 1999
+++ fixed/src/GridText.c        Sat Aug 14 18:39:18 1999
@@ -2256,6 +2256,11 @@
 #else
           (previous->data[previous->size-1] == ' ') &&
 #endif
+#ifdef USE_PSRC
+           !psrc_view && /*don't strip trailing whites - since next line can 
+               start with LY_SOFT_NEWLINE - so we don't loose spaces when
+               'p'rinting this text to file -VH */
+#endif
           (ctrl_chars_on_this_line || HeadTrim || text->first_anchor ||
            underline_on || bold_on ||
            text->style->alignment != HT_LEFT ||