[openwebbeans] branch master updated: adding comments and cosmetic changes

2020-06-05 Thread gerdogdu
This is an automated email from the ASF dual-hosted git repository. gerdogdu pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/openwebbeans.git The following commit(s) were added to refs/heads/master by this push: new c2b0738 adding comments and cosmetic

Re: [openwebbeans] branch master updated: adding comments and cosmetic changes

2020-06-04 Thread Gurkan Erdogdu
Hi Romain I am going over each file slowly and commit slowly :) there is no reason otherwise Regards. Gurkan On Thu, Jun 4, 2020 at 3:04 PM Romain Manni-Bucau wrote: > I would even say *it compiles in another project* (since if OWB compiles it > does not prove it is backward compat cause

Re: [openwebbeans] branch master updated: adding comments and cosmetic changes

2020-06-04 Thread Gurkan Erdogdu
I don't see any problem,it compiles perfectly :) On Thu, Jun 4, 2020 at 1:32 PM Mark Struberg wrote: > did you double check compile and runtime bytecode compatibility? > > -protected abstract Class getMarkerInterface(); > +protected abstract Class getMarkerInterface(); > > Always have

Re: [openwebbeans] branch master updated: adding comments and cosmetic changes

2020-06-04 Thread Romain Manni-Bucau
I would even say *it compiles in another project* (since if OWB compiles it does not prove it is backward compat cause i'm not sure we impl it in another module than the one we define it into ;)) BTW any reason for this kind of commit? If you wantto do a global comment+generic round trip to do

Re: [openwebbeans] branch master updated: adding comments and cosmetic changes

2020-06-04 Thread Mark Struberg
did you double check compile and runtime bytecode compatibility? -protected abstract Class getMarkerInterface(); +protected abstract Class getMarkerInterface(); Always have to think hard about those types of changes. Seen some unpleasant surprises in the past ;) txs and LieGrue, strub

[openwebbeans] branch master updated: adding comments and cosmetic changes

2020-06-03 Thread gerdogdu
This is an automated email from the ASF dual-hosted git repository. gerdogdu pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/openwebbeans.git The following commit(s) were added to refs/heads/master by this push: new 91dcbf2 adding comments and cosmetic