[mythtv] [mythtv-commits] mythtv commit: r16256 by nigel

Michael T. Dean mtdean at thirdcontact.com
Wed Feb 27 02:55:43 UTC 2008


On 02/25/2008 10:11 PM, mythtv at cvs.mythtv.org wrote:
>       Author: nigel
>         Date: 2008-02-26 03:11:50 +0000 (Tue, 26 Feb 2008)
> New Revision: 16256
>    Changeset: http://cvs.mythtv.org/trac/changeset/16256
>
> Modified:
>
>    branches/release-0-21-fixes/mythtv/contrib/misc_status_info/README
>
> Log:
>
> Correct typo. (was corrected in trunk as [15982])

Heh.  Didn't see the change in [15982], but technically, this wasn't a
typo.  The code checks for "<p>" tags or the "<br" part of a tag to
catch either "<br>" or "<br />".  It doesn't make a distinction between
"<p>" and "</p>" as the presence of "</p>" implies "<p>".  See
mythtv/programs/mythbackend/httpstatus.cpp around lines 1256 - 1262.  Of
course, it completely ignores capitalized tags and won't work, i.e. with
style info attributes in the "<p>" tag, but should be good enough as it
does as the README suggests (allowing the user to write the HTML to fit
how it works).

That being said, I don't think it's worth changing the README back as it
probably makes more sense to the user in its current "corrected" form,
even if it's not as precise as before.  As long as we don't "correct" it
in httpstatus.cpp, it should be fine.  :)

Mike


More information about the mythtv-dev mailing list