Re: RFR: 8186958: Need method to create pre-sized HashMap [v12]

2022-04-10 Thread XenoAmess
On Sun, 10 Apr 2022 07:41:05 GMT, Alan Bateman wrote: >> XenoAmess has updated the pull request incrementally with one additional >> commit since the last revision: >> >> use (double) DEFAULT_LOAD_FACTOR instead of 0.75 > > src/java.base/share/classes/java/lang/module/Resolver.java line 501:

Re: RFR: 8186958: Need method to create pre-sized HashMap [v12]

2022-04-10 Thread Alan Bateman
On Wed, 6 Apr 2022 16:02:31 GMT, XenoAmess wrote: >> 8186958: Need method to create pre-sized HashMap > > XenoAmess has updated the pull request incrementally with one additional > commit since the last revision: > > use (double) DEFAULT_LOAD_FACTOR instead of 0.75

Re: RFR: 8186958: Need method to create pre-sized HashMap [v12]

2022-04-09 Thread XenoAmess
On Sat, 9 Apr 2022 16:02:17 GMT, liach wrote: > > > Quick question: If the maps are intended to be fixed-size, can't the > > > users just call `new HashMap<>(size, 1)`, increasing the growth factor to > > > prevent growth? > > > > > > @liach this questions equals question : "why default load

Re: RFR: 8186958: Need method to create pre-sized HashMap [v12]

2022-04-09 Thread liach
On Sat, 9 Apr 2022 15:25:47 GMT, XenoAmess wrote: > > Quick question: If the maps are intended to be fixed-size, can't the users > > just call `new HashMap<>(size, 1)`, increasing the growth factor to prevent > > growth? > > @liach this questions equals question : "why default load factor be

Re: RFR: 8186958: Need method to create pre-sized HashMap [v12]

2022-04-09 Thread XenoAmess
On Sat, 9 Apr 2022 04:33:00 GMT, liach wrote: > Quick question: If the maps are intended to be fixed-size, can't the users > just call `new HashMap<>(size, 1)`, increasing the growth factor to prevent > growth? @liach this questions equals question : "why default load factor be 0.75 not 1" In

Re: RFR: 8186958: Need method to create pre-sized HashMap [v12]

2022-04-08 Thread liach
On Wed, 6 Apr 2022 16:02:31 GMT, XenoAmess wrote: >> 8186958: Need method to create pre-sized HashMap > > XenoAmess has updated the pull request incrementally with one additional > commit since the last revision: > > use (double) DEFAULT_LOAD_FACTOR instead of 0.75 Quick question: If the

Re: RFR: 8186958: Need method to create pre-sized HashMap [v12]

2022-04-08 Thread Stuart Marks
On Wed, 6 Apr 2022 16:02:31 GMT, XenoAmess wrote: >> 8186958: Need method to create pre-sized HashMap > > XenoAmess has updated the pull request incrementally with one additional > commit since the last revision: > > use (double) DEFAULT_LOAD_FACTOR instead of 0.75 Regarding the scope of

Re: RFR: 8186958: Need method to create pre-sized HashMap [v12]

2022-04-06 Thread XenoAmess
On Wed, 6 Apr 2022 16:02:31 GMT, XenoAmess wrote: >> 8186958: Need method to create pre-sized HashMap > > XenoAmess has updated the pull request incrementally with one additional > commit since the last revision: > > use (double) DEFAULT_LOAD_FACTOR instead of 0.75 There be another

Re: RFR: 8186958: Need method to create pre-sized HashMap [v12]

2022-04-06 Thread XenoAmess
> 8186958: Need method to create pre-sized HashMap XenoAmess has updated the pull request incrementally with one additional commit since the last revision: use (double) DEFAULT_LOAD_FACTOR instead of 0.75 - Changes: - all: https://git.openjdk.java.net/jdk/pull/7928/files -