RFR: 8258754: Gracefully fallback to the OpenGL rendering pipeline if Metal rendering pipeline initialization fails [v6]

Philip Race philip.race at oracle.com
Mon Jan 18 20:39:42 UTC 2021


Alexey,

On 1/11/21 2:11 AM, Alexey Ushakov wrote:
> So, we can get rid of (CGraphicsDevice.usingMetalPipeline()) ? … : … checks
>   
> Best Regards,
> Alexey

I think this exactly what I raised on Jan 8th :-

On 1/8/21 11:44 AM, Phil Race wrote:
> 52: Object context)
>> 53:     {
>> 54:         return CGraphicsDevice.useMetalPipeline() ? new MTLVolatileSurfaceManager(vImg, context) :
> similar to my other naming comments this is more "using" than "use"  - or enabled again.
> I'd like consistent naming where it makes sense.
> But also this looks like another case where I'd prefer to see us have a pattern that retrieves  the installed pipeline.
> So this entire method body is something like
> getInstalledPipeline().createVolatileSurfaceManager(vImg, context);

Our conclusion was to address it in another bug/PR.

-phil.


More information about the lanai-dev mailing list