#175 HTML code not enough XML compliant (cosmetic)

closed-works-for-me
Browsing (93)
2
2011-03-12
2011-03-09
No

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
! "\&lt;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
! "\&lt;p align=\"center\">\n<i>The file $pathname does not exist.</i>\n</p>\n"
!-296,296
! "\&lt;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("\&lt;p align=\"center\">\n<i>The file $pathname does not exist.</i>\n</p>\n");
------------------ end of patch ----------------------

Discussion

  • Andre-Littoz

    Andre-Littoz - 2011-03-09
    • priority: 5 --> 2
    • status: open --> open-works-for-me
     
  • Andre-Littoz

    Andre-Littoz - 2011-03-11
    • assigned_to: nobody --> ajlittoz
     
  • Andre-Littoz

    Andre-Littoz - 2011-03-12

    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.

     
  • Andre-Littoz

    Andre-Littoz - 2011-03-12

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

     
  • Andre-Littoz

    Andre-Littoz - 2011-03-12
    • status: open-works-for-me --> closed-works-for-me
     

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:





No, thanks