Re: RFR: 8263363: Minor cleanup of Lanai code - unused code removal and comments correction [v3]

2021-04-27 Thread Ajit Ghaisas
On Fri, 9 Apr 2021 07:21:57 GMT, Sergey Bylokhov wrote: >> This is the only place where we use MTLContext.invalidateCurrentContext() - >> which when processed in MTLRenderQueue - clears some native stuff and nulls >> out both mtlc and dstOps pointers maintained in MTLRenderQueue.m. I think,

Re: RFR: 8263363: Minor cleanup of Lanai code - unused code removal and comments correction [v3]

2021-04-09 Thread Sergey Bylokhov
On Thu, 8 Apr 2021 10:39:38 GMT, Ajit Ghaisas wrote: >> src/java.desktop/macosx/classes/sun/java2d/metal/MTLGraphicsConfig.java line >> 149: >> >>> 147: try { >>> 148: // getMTLConfigInfo() creates new MTLContext, so we should >>> first >>> 149: // invalidate

Re: RFR: 8263363: Minor cleanup of Lanai code - unused code removal and comments correction [v3]

2021-04-08 Thread Ajit Ghaisas
> Refer JBS for 3 issues that this PR addresses. > In addition, I have corrected an erroneous free() call in the same method I > was cleaning up. Ajit Ghaisas has updated the pull request incrementally with one additional commit since the last revision: log message correction -