[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-openjfx-dev
Subject:    Re: [8u40] Review request: RT-38074: [macosx] Separate QTKit platform code from core media code so i
From:       David DeHaven <david.dehaven () oracle ! com>
Date:       2014-07-30 15:49:44
Message-ID: A52F4927-0B31-4BC5-BE31-ABA322567A12 () oracle ! com
[Download RAW message or body]


JIRA Issue:
https://javafx-jira.kenai.com/browse/RT-38074

Latest webrev:
http://cr.openjdk.java.net/~ddehaven/RT-38074/rt.2/

Removed new makefile (eyesore), cleaned up/enhanced existing Makefile, fixed a \
compiler warning.

Last iteration hopefully, I let it bake for 12 hours and haven't had the urge to \
change anything ;)

-DrD-

> 
> Kirill, Alexander, Kevin:
> 
> New version up for review, please take a look:
> http://cr.openjdk.java.net/~ddehaven/RT-38074/rt.1/
> 
> I moved OSXPlatform and OSXMediaPlayer code back to jfxmedia, since it was meant to \
> be an abstraction point for using either QTKit or AVFoundation in the first place. 
> -DrD-
> 
> > 
> > Belay that review.. I have some (significant) changes to make, in preparation for \
> > the larger task of implementing the AVFoundation based code. 
> > -DrD-
> > 
> > > JIRA:
> > > https://javafx-jira.kenai.com/browse/RT-38074
> > > 
> > > Webrev:
> > > http://cr.openjdk.java.net/~ddehaven/RT-38074/rt.0/
> > > 
> > > This change moves the QTKit based media platform code into it's own dylib. \
> > > NativeMediaManager had to be modified to allow detection of the new library to \
> > > determine if the platform was available or not. There may be a slight \
> > > performance impact due to loading the native libs sooner, but the bulk of the \
> > > initialization is still done at a later time. 
> > > -DrD-
> > > 
> > 
> 


[prev in list] [next in list] [prev in thread] [next in thread]