Login | Register For Free | Help
Search for: (Advanced)

Mailing List Archive: MythTV: Dev
Re: [mythtv-commits] mythtv/master commit: 784eb58dd by Daniel Kristjansson (daniel-kristjansson)
 

Index | Next | Previous | View Flat


stichnot at gmail

Jan 28, 2012, 6:40 AM


Views: 219
Permalink
Re: [mythtv-commits] mythtv/master commit: 784eb58dd by Daniel Kristjansson (daniel-kristjansson)

On Sat, Jan 28, 2012 at 5:04 AM, MythTV <noreply [at] mythtv> wrote:
>      Author:  Daniel Kristjansson <danielk [at] cuymedia>
>  Change Date:  2012-01-28T05:03:38-08:00
>   Push Date:  2012/01/28 05:03:46 -0800
>  Repository:  mythtv
>      Branch:  master
> New Revision:  784eb58dd282d8d9ca9ed4869eb8d1be93855834
>   Changeset:  https://github.com/MythTV/mythtv/commit/784eb58dd
>
> Log:
>
> Const correctness fixes, note there are unaddressed design issues, see below.
>
> The design issue I see is that we set a PlayerContext for
> the instance but many methods still take a PlayerContext
> as a param. Most likely these can just use the instance's
> PlayerContext.

http://code.mythtv.org/trac/ticket/8901#comment:3

This was my ill-advised attempt to minimize the size of the patch.

Jim
_______________________________________________
mythtv-dev mailing list
mythtv-dev [at] mythtv
http://www.mythtv.org/mailman/listinfo/mythtv-dev

Subject User Time
Re: [mythtv-commits] mythtv/master commit: 784eb58dd by Daniel Kristjansson (daniel-kristjansson) stichnot at gmail Jan 28, 2012, 6:40 AM
    Re: [mythtv-commits] mythtv/master commit: 784eb58dd by Daniel Kristjansson (daniel-kristjansson) mtdean at thirdcontact Jan 28, 2012, 8:09 AM

  Index | Next | Previous | View Flat
 
 


Interested in having your list archived? Contact Gossamer Threads
 
  Web Applications & Managed Hosting Powered by Gossamer Threads Inc.