Re: RFR: implementation for JEP 334: JVM Constants API

2018-05-23 Thread Ali Ebrahimi
/constant.api/javadoc.07/ > overview-summary.html > > PS. We are offering a MacBook Wheel to the authors of the first 5 comments > :) > -- Best Regards, Ali Ebrahimi

Re: Fwd: RFR(m): 8140281 deprecate Optional.get()

2016-04-25 Thread Ali Ebrahimi
explained to me the other day, this >> code is later recompiled against the JDK 9 libraries, and in that case it >> does generate the warning. >> >> So for this case I think calling get() with @SuppressWarnings is the way >> to proceed. I opted to extract a local variable and put @SW on it, in order >> to minimize its scope, but if you prefer an alternative I'd be happy to >> change it. >> >> s'marks >> > > -- Best Regards, Ali Ebrahimi

Re: Signature of MethodHandleInfo.reflectAs is not specific enough

2014-03-01 Thread Ali Ebrahimi
Hi, On Sat, Mar 1, 2014 at 12:16 AM, John Rose wrote: > On Feb 25, 2014, at 3:13 AM, Ali Ebrahimi > wrote: > > I know, this is too late, but I want to share my suggestion: > > public T reflectAs(Class super T> expected, MethodHandles.Lookup lookup) > > > Isn&#x

Re: Signature of MethodHandleInfo.reflectAs is not specific enough

2014-02-25 Thread Ali Ebrahimi
sm = reflectAs(Method.class, MethodHandles.lookup());//fails Regards, Ali Ebrahimi On Mon, Nov 11, 2013 at 1:59 AM, Remi Forax wrote: > The is a stupid issue with the signature of MethodHandleInfo.reflectAs, > j.l.r.Field, Method or Constructor implement two interfaces Memb

Re: Signature of MethodHandleInfo.reflectAs is not specific enough

2013-11-11 Thread Ali Ebrahimi
AM, Peter Levart wrote: > On 11/11/2013 02:24 AM, Ali Ebrahimi wrote: > > This is another workaround: > > public R reflectAs(Class > expected, Lookup lookup); > > info.reflectAs(Member.class, lookup);//works > info.reflectAs(AnnotatedElement.class, lookup);//works

Re: Signature of MethodHandleInfo.reflectAs is not specific enough

2013-11-10 Thread Ali Ebrahimi
This is another workaround: public R reflectAs(Class expected, Lookup lookup); info.reflectAs(Member.class, lookup);//works info.reflectAs(AnnotatedElement.class, lookup);//works info.reflectAs(Member.class, lookup);//works info.reflectAs(AnnotatedElement.class, lookup);//works info.reflectAs(

Re: Review: lamda expressions and bulk data operations on collections.

2013-08-30 Thread Ali Ebrahimi
I see nobody using mangled (comparing&map) methods (even oracle guys) and this is the thing that I already have expected, just since we all lazy and this would hurt performance. This issue would be a candidate topic for next edition of Joshua Bloch's Effective Java. So to prevent this I propose fo

Re: RFR: 8023681: Fix raw type warning caused by Sink

2013-08-23 Thread Ali Ebrahimi
ould even accept this heinous cast, I haven't > tried it) > > Mike > > On Aug 23 2013, at 14:18 , Ali Ebrahimi wrote: > > > Why not to make castingIdentity method a constant? > > > > > > On Fri, Aug 23, 2013 at 11:47 PM, Henry Jen > wrote: > &

Re: RFR: 8023681: Fix raw type warning caused by Sink

2013-08-23 Thread Ali Ebrahimi
Why not to make castingIdentity method a constant? On Fri, Aug 23, 2013 at 11:47 PM, Henry Jen wrote: > Hi, > > Please kindly review the fix for eliminate some warnings in > java.util.stream package. > > Chained Sink is an internal implementation detail, add the type for > downstream is more pr

Re: Breaking Changeset: 4802647: Throw required NPEs from removeAll()/retainAll()

2013-06-04 Thread Ali Ebrahimi
, Jun 4, 2013 at 2:08 PM, Alan Bateman wrote: > On 04/06/2013 10:14, Ali Ebrahimi wrote: > >> : >> >> >> the cause of this error is this new changeset: >> >> 4802647: Throw required NPEs from removeAll()/retainAll() >> >> current code assume that

Breaking Changeset: 4802647: Throw required NPEs from removeAll()/retainAll()

2013-06-04 Thread Ali Ebrahimi
when building openjfx8 with jdk8b92 i encountered this error : Total time: 5.313 secs :buildSrc:clean :buildSrc:generateGrammarSource error(10): internal error: Can't get property indirectDelegates using method ge t/isIndirectDelegates from org.antlr.tool.Grammar instance : java.lang.NullPointerE

Re: Upgrade to JAXP 1.5 Breaks Existing Apps

2013-06-03 Thread Ali Ebrahimi
thanks. On Mon, Jun 3, 2013 at 2:59 PM, Alan Bateman wrote: > On 02/06/2013 22:05, Ali Ebrahimi wrote: > >> I update to jdk8b92 and almost all apps deals with xml parsing breaks. >> in other word, current default value for XMLConstants.ACCESS_EXTERNAL_** >> DTD >>

Upgrade to JAXP 1.5 Breaks Existing Apps

2013-06-02 Thread Ali Ebrahimi
I update to jdk8b92 and almost all apps deals with xml parsing breaks. in other word, current default value for XMLConstants.ACCESS_EXTERNAL_DTD property is empty string. This should be at least change to "file".

Re: CFR - updated 8001667: Comparator combinators and extension methods

2013-03-07 Thread Ali Ebrahimi
Hi, for me this is more readable than current naming. Comparator cmp1 = compareWith(People::getFirstName). thenCompareWith(People::getLastName); Ali Ebrahimi On Thu, Mar 7, 2013 at 12:22 AM, Henry Jen wrote: > On 03/06/2013 03:28 AM, Ali Ebrahimi wr

Re: CFR - updated 8001667: Comparator combinators and extension methods

2013-03-06 Thread Ali Ebrahimi
Hi, just one suggestion: rename comparing with compareWith 1) public static > Comparator compareWith(Function keyExtractor) { 2) default Comparator thenCompareWith(Comparator other) Best Regards, Ali Ebrahimi On Wed, Mar 6, 2013 at 12:16 AM, Henry Jen wrote: > Hi, > > Anoth