Re: RFR: 8210765: Remove finalize method in CStrike.java [v2]

2025-08-04 Thread Alexander Zvegintsev
On Mon, 21 Jul 2025 16:52:54 GMT, Phil Race wrote: >> Remove finalize method from CStrike.java >> >> This one is a bit odd in that there's already a Disposer used - and in fact >> it involves two classes >> CStrikeDisposer.java and its subclass - the nested class >> CStrike.GlyphInfoCache >> C

Re: RFR: 8210765: Remove finalize method in CStrike.java [v2]

2025-08-02 Thread Prasanta Sadhukhan
On Mon, 21 Jul 2025 16:52:54 GMT, Phil Race wrote: >> Remove finalize method from CStrike.java >> >> This one is a bit odd in that there's already a Disposer used - and in fact >> it involves two classes >> CStrikeDisposer.java and its subclass - the nested class >> CStrike.GlyphInfoCache >> C

Re: RFR: 8210765: Remove finalize method in CStrike.java [v2]

2025-08-01 Thread Alisen Chung
On Mon, 21 Jul 2025 16:52:54 GMT, Phil Race wrote: >> Remove finalize method from CStrike.java >> >> This one is a bit odd in that there's already a Disposer used - and in fact >> it involves two classes >> CStrikeDisposer.java and its subclass - the nested class >> CStrike.GlyphInfoCache >> C

Re: RFR: 8210765: Remove finalize method in CStrike.java [v2]

2025-08-01 Thread Phil Race
On Fri, 1 Aug 2025 03:18:16 GMT, Prasanta Sadhukhan wrote: >> And if it is 0, what would you do ? >> Previously 0 meant it hadn't yet been initialized. Now it is always >> initialized. >> >> You'd need to return it anyway even it was zero. > > Then I guess whoever is using the `nativeStrikePtr

Re: RFR: 8210765: Remove finalize method in CStrike.java [v2]

2025-08-01 Thread Phil Race
On Thu, 31 Jul 2025 04:12:07 GMT, Prasanta Sadhukhan wrote: > Guess wildcard import and copyright year fix can be done.. I'm not going looking for clean up here. - PR Comment: https://git.openjdk.org/jdk/pull/26397#issuecomment-3145861648

Re: RFR: 8210765: Remove finalize method in CStrike.java [v2]

2025-07-31 Thread Prasanta Sadhukhan
On Thu, 31 Jul 2025 19:47:29 GMT, Phil Race wrote: >> src/java.desktop/macosx/classes/sun/font/CStrike.java line 99: >> >>> 97: >>> 98: public long getNativeStrikePtr() { >>> 99: return nativeStrikePtr; >> >> Any reason this check was removed? >> I see that initNativeStrikePtr will

Re: RFR: 8210765: Remove finalize method in CStrike.java [v2]

2025-07-31 Thread Phil Race
On Thu, 31 Jul 2025 04:10:37 GMT, Prasanta Sadhukhan wrote: >> Phil Race has updated the pull request incrementally with one additional >> commit since the last revision: >> >> 8210765 > > src/java.desktop/macosx/classes/sun/font/CStrike.java line 99: > >> 97: >> 98: public long getNat

Re: RFR: 8210765: Remove finalize method in CStrike.java [v2]

2025-07-30 Thread Prasanta Sadhukhan
On Mon, 21 Jul 2025 16:52:54 GMT, Phil Race wrote: >> Remove finalize method from CStrike.java >> >> This one is a bit odd in that there's already a Disposer used - and in fact >> it involves two classes >> CStrikeDisposer.java and its subclass - the nested class >> CStrike.GlyphInfoCache >> C

Re: RFR: 8210765: Remove finalize method in CStrike.java

2025-07-21 Thread Phil Race
On Fri, 18 Jul 2025 20:51:04 GMT, Phil Race wrote: > Remove finalize method from CStrike.java > > This one is a bit odd in that there's already a Disposer used - and in fact > it involves two classes > CStrikeDisposer.java and its subclass - the nested class > CStrike.GlyphInfoCache > CStrike.

Re: RFR: 8210765: Remove finalize method in CStrike.java [v2]

2025-07-21 Thread Phil Race
> Remove finalize method from CStrike.java > > This one is a bit odd in that there's already a Disposer used - and in fact > it involves two classes > CStrikeDisposer.java and its subclass - the nested class > CStrike.GlyphInfoCache > CStrike.GlyphInfoCache is tracking all the glyph image refere

RFR: 8210765: Remove finalize method in CStrike.java

2025-07-18 Thread Phil Race
Remove finalize method from CStrike.java This one is a bit odd in that there's already a Disposer used - and in fact it involves two classes CStrikeDisposer.java and its subclass - the nested class CStrike.GlyphInfoCache CStrike.GlyphInfoCache is tracking all the glyph image references. CStrikeD