[mythtv] [mythtv-commits] mythtv commit: r25714 by cpinkham
David Blain
MythTv at TheBlains.net
Tue Aug 17 21:00:18 UTC 2010
> -----Original Message-----
> From: mythtv-dev-bounces at mythtv.org [mailto:mythtv-dev-
> bounces at mythtv.org] On Behalf Of Chris Pinkham
> Sent: Tuesday, August 17, 2010 4:30 PM
> To: Development of mythtv
> Subject: Re: [mythtv] [mythtv-commits] mythtv commit: r25714 by
> cpinkham
>
> * On Tue Aug 17, 2010 at 10:16:45AM -0400, David Blain wrote:
> > The only issue I see with this commit is that one of the methods
> > returns html instead of xml. This breaks what the intention of the
> > design behind MythXml's functionality and also hinders future reuse
> > of
> the methods.
>
> GetFileList?? I'll gladly change it. I was under the impression that
> there were other non-xml items in there (other than the binary files),
> but I admit not looking over all of the code.
>
> --
> Chris
Looks like "GetInternetSearch" sneaked in without me noticing. If I ever
get my changes merged with the latest trunk, the HTML format will be lost
(unless someone writes a custom formatter for it which is possible with my
new design, but not very useful).
I just wanted to keep MythXML as a clean api that will render data in
multiple formats, and if people rely on a method returning formatted html,
it will break.
Thanks,
David.
More information about the mythtv-dev
mailing list