[Lxr-dev] [ lxr-Bugs-3204285 ] HTML code not enough XML compliant (cosmetic)

[SourceForge.net <no...@so...>]

Bugs item #3204285, was opened at 2011-03-09 15:51
Message generated for change (Comment added) made by ajlittoz
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=390117&aid=3204285&group_id=27350

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Browsing
Group: None
>Status: Closed
Resolution: Works For Me
Priority: 2
Private: No
Submitted By: Andre-Littoz (ajlittoz)
Assigned to: Andre-Littoz (ajlittoz)
Summary: HTML code not enough XML compliant (cosmetic)

Initial Comment:
LXR v0.9.8

File source generates HTML page which may contain informative data structured in paragraphs. These paragraphs are opened with <p> tag but never closed. This is OK for HTML but is not in the spirit of XML.

The proposed patch add a </p> tag at the end of the concerned paragraphs.

File: source

110225 direxpand

!-154,154
!              . " does not exist.</i>\n</p>\n");
!-156,156
!            "\<p align=\"center\">\n<i>This directory might exist in other versions, try 'Show attic files' or select a different Version.</i>\n</p>\n"
-------------------------- and of patch ------------------------

110225 printfile
!-293,293
!                "\<p align=\"center\">\n<i>The file $pathname does not exist.</i>\n</p>\n"
!-296,296
!                "\<p align=\"center\">\n<i>This file might exist in other versions, try 'Show attic files' or select a different Version.</i>\n</p>\n"
---------------- end of patch -----------------------

110225 source init code
!-308,308
!    print("\<p align=\"center\">\n<i>The file $pathname does not exist.</i>\n</p>\n");
------------------ end of patch ----------------------


----------------------------------------------------------------------

>Comment By: Andre-Littoz (ajlittoz)
Date: 2011-03-12 14:38

Message:
Incorporated a slightly modified version of the fix: use name of variable
'v' instead of Version to make intent more clear.

----------------------------------------------------------------------

Comment By: Andre-Littoz (ajlittoz)
Date: 2011-03-12 14:38

Message:
This has now been fixed in CVS.

If you can install the new version and check that it solves your
problem, then it would be very useful.

Thanks for reporting this defect and helping to make LXR better.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=390117&aid=3204285&group_id=27350