You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@harmony.apache.org by Mark Hindess <ma...@googlemail.com> on 2006/10/18 09:56:47 UTC
[classlib] Build failure messages
These messages aren't getting through on the commits list on due to the
large number of warnings since switching to ecj. I've modified the
summary process so that it ignores these for the moment.
Hopefully this means that the failure messages that haven't been making
the commits list will start getting through.
Regards,
Mark.
---------------------------------------------------------------------
Terms of use : http://incubator.apache.org/harmony/mailing.html
To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
For additional commands, e-mail: harmony-dev-help@incubator.apache.org
Re: [classlib] Build failure messages
Posted by Mikhail Fursov <mi...@gmail.com>.
+1
I never know if it's something important or not.
On 10/19/06, Denis Kishenko <dk...@gmail.com> wrote:
>
> I also worried about a lot of warnings and I think warnings like these
> can be fixed
>
>
> last\modules\concurrent\standard\src\main\java\java\util\concurrent\ConcurrentHashMap.java
> [javac] (at line 26)
> [javac] import java.io.ObjectInputStream;
> [javac] ^^^^^^^^^^^^^^^^^^^^^^^^^
> [javac] The import java.io.ObjectInputStream is never used
>
> [javac] 34. WARNING in
>
> C:\Work\Projects\Harmony-last\modules\concurrent\standard\src\main\java\java\util\concurrent\CountDownLatch.java
> [javac] (at line 130)
> [javac] private static final class Sync extends
> AbstractQueuedSynchronizer {
> [javac] ^^^^
> [javac] The serializable class Sync does not declare a static
> final serialVersionUID field of type long
>
> 2006/10/19, Nathan Beyer <nb...@gmail.com>:
> > I added a compiler arg to the main 'build-java.xml' compile to disable
> > all of the warnings.
> >
> > This should take care of the warnings.
> >
> > -Nathan
> >
> > On 10/18/06, Geir Magnusson Jr. <ge...@pobox.com> wrote:
> > > Can we just shut them off for now? it's a good project for someone to
> > > go and clean those up, but there's no need for all of us to see them.
> > >
> > > geir
> > >
> > > Mark Hindess wrote:
> > > > These messages aren't getting through on the commits list on due to
> the
> > > > large number of warnings since switching to ecj. I've modified the
> > > > summary process so that it ignores these for the moment.
> > > >
> > > > Hopefully this means that the failure messages that haven't been
> making
> > > > the commits list will start getting through.
> > > >
> > > > Regards,
> > > > Mark.
> > > >
> > > >
> > > >
> > > >
> > > >
> ---------------------------------------------------------------------
> > > > Terms of use : http://incubator.apache.org/harmony/mailing.html
> > > > To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> > > > For additional commands, e-mail:
> harmony-dev-help@incubator.apache.org
> > > >
> > >
> > > ---------------------------------------------------------------------
> > > Terms of use : http://incubator.apache.org/harmony/mailing.html
> > > To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> > > For additional commands, e-mail: harmony-dev-help@incubator.apache.org
> > >
> > >
> >
> > ---------------------------------------------------------------------
> > Terms of use : http://incubator.apache.org/harmony/mailing.html
> > To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> > For additional commands, e-mail: harmony-dev-help@incubator.apache.org
> >
> >
>
>
> --
> Denis M. Kishenko
> Enterprise Solutions Software Division
>
> ---------------------------------------------------------------------
> Terms of use : http://incubator.apache.org/harmony/mailing.html
> To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> For additional commands, e-mail: harmony-dev-help@incubator.apache.org
>
>
--
Mikhail Fursov
Re: [classlib] Build failure messages
Posted by Tim Ellison <t....@gmail.com>.
Nathan Beyer wrote:
> * unused - unused private fields, methods; these are tough as some of
> them are seemingly used by native code
yes, as you say some of these slots are used by native code to stash
state; and some are there for serialization compatibility even though
our impl doesn't use them.
Each case will have to be determined (and documented in the code!) then
neutralized with the suppress warnings annotation.
Regards,
Tim
--
Tim Ellison (t.p.ellison@gmail.com)
IBM Java technology centre, UK.
---------------------------------------------------------------------
Terms of use : http://incubator.apache.org/harmony/mailing.html
To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
For additional commands, e-mail: harmony-dev-help@incubator.apache.org
Re: [classlib] Build failure messages
Posted by "Geir Magnusson Jr." <ge...@pobox.com>.
You have patience I can only dream about...
geir
Nathan Beyer wrote:
> I've been trying to work through many of the warnings, module by
> module. Most of the warnings fall into the following categories.
> * serial - Serializable classes that don't have explicit
> serialVersionUID fields
> * unchecked - missing type variables, mostly collection references
> * unused - unused private fields, methods; these are tough as some of
> them are seemingly used by native code
> * missing @Override - methods that override other methods are
> annotated for safety
>
> -Nathan
>
> On 10/19/06, Tim Ellison <t....@gmail.com> wrote:
>> Nathan has turned off all warnings. I suggest that we start to turn
>> them back on incrementally and fix the warnings we see.
>>
>> Regards,
>> Tim
>>
>> Denis Kishenko wrote:
>> > I also worried about a lot of warnings and I think warnings like these
>> > can be fixed
>> >
>> >
>> last\modules\concurrent\standard\src\main\java\java\util\concurrent\ConcurrentHashMap.java
>>
>> >
>> > [javac] (at line 26)
>> > [javac] import java.io.ObjectInputStream;
>> > [javac] ^^^^^^^^^^^^^^^^^^^^^^^^^
>> > [javac] The import java.io.ObjectInputStream is never used
>> >
>> > [javac] 34. WARNING in
>> >
>> C:\Work\Projects\Harmony-last\modules\concurrent\standard\src\main\java\java\util\concurrent\CountDownLatch.java
>>
>> >
>> > [javac] (at line 130)
>> > [javac] private static final class Sync extends
>> > AbstractQueuedSynchronizer {
>> > [javac] ^^^^
>> > [javac] The serializable class Sync does not declare a static
>> > final serialVersionUID field of type long
>> >
>> > 2006/10/19, Nathan Beyer <nb...@gmail.com>:
>> >> I added a compiler arg to the main 'build-java.xml' compile to disable
>> >> all of the warnings.
>> >>
>> >> This should take care of the warnings.
>> >>
>> >> -Nathan
>> >>
>> >> On 10/18/06, Geir Magnusson Jr. <ge...@pobox.com> wrote:
>> >> > Can we just shut them off for now? it's a good project for
>> someone to
>> >> > go and clean those up, but there's no need for all of us to see
>> them.
>> >> >
>> >> > geir
>> >> >
>> >> > Mark Hindess wrote:
>> >> > > These messages aren't getting through on the commits list on due
>> >> to the
>> >> > > large number of warnings since switching to ecj. I've modified
>> the
>> >> > > summary process so that it ignores these for the moment.
>> >> > >
>> >> > > Hopefully this means that the failure messages that haven't been
>> >> making
>> >> > > the commits list will start getting through.
>> >> > >
>> >> > > Regards,
>> >> > > Mark.
>> >> > >
>> >> > >
>> >> > >
>> >> > >
>> >> > >
>> ---------------------------------------------------------------------
>> >> > > Terms of use : http://incubator.apache.org/harmony/mailing.html
>> >> > > To unsubscribe, e-mail:
>> harmony-dev-unsubscribe@incubator.apache.org
>> >> > > For additional commands, e-mail:
>> >> harmony-dev-help@incubator.apache.org
>> >> > >
>> >> >
>> >> >
>> ---------------------------------------------------------------------
>> >> > Terms of use : http://incubator.apache.org/harmony/mailing.html
>> >> > To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
>> >> > For additional commands, e-mail:
>> harmony-dev-help@incubator.apache.org
>> >> >
>> >> >
>> >>
>> >> ---------------------------------------------------------------------
>> >> Terms of use : http://incubator.apache.org/harmony/mailing.html
>> >> To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
>> >> For additional commands, e-mail: harmony-dev-help@incubator.apache.org
>> >>
>> >>
>> >
>> >
>>
>> --
>>
>> Tim Ellison (t.p.ellison@gmail.com)
>> IBM Java technology centre, UK.
>>
>> ---------------------------------------------------------------------
>> Terms of use : http://incubator.apache.org/harmony/mailing.html
>> To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
>> For additional commands, e-mail: harmony-dev-help@incubator.apache.org
>>
>>
>
> ---------------------------------------------------------------------
> Terms of use : http://incubator.apache.org/harmony/mailing.html
> To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> For additional commands, e-mail: harmony-dev-help@incubator.apache.org
>
---------------------------------------------------------------------
Terms of use : http://incubator.apache.org/harmony/mailing.html
To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
For additional commands, e-mail: harmony-dev-help@incubator.apache.org
Re: [classlib] Build failure messages
Posted by Ilya Okomin <il...@gmail.com>.
On 10/20/06, Tim Ellison <t....@gmail.com> wrote:
>
> Denis Kishenko wrote:
> > I have made fix[1] for "unused" category (only java code). If nobody
> > has objections I could fix other caterogies.
> >
> > [1] http://issues.apache.org/jira/browse/HARMONY-1926
>
> See Nathan's comments about unused vars -- they cannot be fixed safely
> with the Java wizard. Unused imports should be ok to fix that way.
I see one step to be added to the unused vars removing: to check if 'unused
var' is unused in the native code:)
However I think it is related only for unused fields, local variables from
subroutines and methods can be deleted without any checks.
Regards,
Ilya.
Regards,
> Tim
>
> --
>
> Tim Ellison (t.p.ellison@gmail.com)
> IBM Java technology centre, UK.
>
> ---------------------------------------------------------------------
> Terms of use : http://incubator.apache.org/harmony/mailing.html
> To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> For additional commands, e-mail: harmony-dev-help@incubator.apache.org
>
>
--
--
Ilya Okomin
Intel Middleware Products Division
Re: [classlib] Build failure messages
Posted by Denis Kishenko <dk...@gmail.com>.
Added patch for SUID wanings to the same issue.H-1926
2006/10/20, Tim Ellison <t....@gmail.com>:
> Denis Kishenko wrote:
> > Almost all unused variables are local. Only two of them are class
> > fields and they are not using in native code. Build and all tests are
> > successful.
>
> Thanks Denis!
>
> Regards,
> Tim
>
> --
>
> Tim Ellison (t.p.ellison@gmail.com)
> IBM Java technology centre, UK.
>
> ---------------------------------------------------------------------
> Terms of use : http://incubator.apache.org/harmony/mailing.html
> To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> For additional commands, e-mail: harmony-dev-help@incubator.apache.org
>
>
--
Denis M. Kishenko
Intel Middleware Products Division
---------------------------------------------------------------------
Terms of use : http://incubator.apache.org/harmony/mailing.html
To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
For additional commands, e-mail: harmony-dev-help@incubator.apache.org
Re: [classlib] Build failure messages
Posted by Tim Ellison <t....@gmail.com>.
Denis Kishenko wrote:
> Almost all unused variables are local. Only two of them are class
> fields and they are not using in native code. Build and all tests are
> successful.
Thanks Denis!
Regards,
Tim
--
Tim Ellison (t.p.ellison@gmail.com)
IBM Java technology centre, UK.
---------------------------------------------------------------------
Terms of use : http://incubator.apache.org/harmony/mailing.html
To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
For additional commands, e-mail: harmony-dev-help@incubator.apache.org
Re: [classlib] Build failure messages
Posted by Denis Kishenko <dk...@gmail.com>.
Almost all unused variables are local. Only two of them are class
fields and they are not using in native code. Build and all tests are
successful.
2006/10/20, Tim Ellison <t....@gmail.com>:
> Denis Kishenko wrote:
> > I have made fix[1] for "unused" category (only java code). If nobody
> > has objections I could fix other caterogies.
> >
> > [1] http://issues.apache.org/jira/browse/HARMONY-1926
>
> See Nathan's comments about unused vars -- they cannot be fixed safely
> with the Java wizard. Unused imports should be ok to fix that way.
>
> Regards,
> Tim
>
> --
>
> Tim Ellison (t.p.ellison@gmail.com)
> IBM Java technology centre, UK.
>
> ---------------------------------------------------------------------
> Terms of use : http://incubator.apache.org/harmony/mailing.html
> To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> For additional commands, e-mail: harmony-dev-help@incubator.apache.org
>
>
--
Denis M. Kishenko
Intel Middleware Products Division
---------------------------------------------------------------------
Terms of use : http://incubator.apache.org/harmony/mailing.html
To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
For additional commands, e-mail: harmony-dev-help@incubator.apache.org
Re: [classlib] Build failure messages
Posted by Tim Ellison <t....@gmail.com>.
Denis Kishenko wrote:
> I have made fix[1] for "unused" category (only java code). If nobody
> has objections I could fix other caterogies.
>
> [1] http://issues.apache.org/jira/browse/HARMONY-1926
See Nathan's comments about unused vars -- they cannot be fixed safely
with the Java wizard. Unused imports should be ok to fix that way.
Regards,
Tim
--
Tim Ellison (t.p.ellison@gmail.com)
IBM Java technology centre, UK.
---------------------------------------------------------------------
Terms of use : http://incubator.apache.org/harmony/mailing.html
To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
For additional commands, e-mail: harmony-dev-help@incubator.apache.org
Re: [classlib] Build failure messages
Posted by Denis Kishenko <dk...@gmail.com>.
I have made fix[1] for "unused" category (only java code). If nobody
has objections I could fix other caterogies.
[1] http://issues.apache.org/jira/browse/HARMONY-1926
2006/10/20, Nathan Beyer <nb...@gmail.com>:
> I've been trying to work through many of the warnings, module by
> module. Most of the warnings fall into the following categories.
> * serial - Serializable classes that don't have explicit serialVersionUID fields
> * unchecked - missing type variables, mostly collection references
> * unused - unused private fields, methods; these are tough as some of
> them are seemingly used by native code
> * missing @Override - methods that override other methods are
> annotated for safety
>
> -Nathan
>
> On 10/19/06, Tim Ellison <t....@gmail.com> wrote:
> > Nathan has turned off all warnings. I suggest that we start to turn
> > them back on incrementally and fix the warnings we see.
> >
> > Regards,
> > Tim
> >
> > Denis Kishenko wrote:
> > > I also worried about a lot of warnings and I think warnings like these
> > > can be fixed
> > >
> > > last\modules\concurrent\standard\src\main\java\java\util\concurrent\ConcurrentHashMap.java
> > >
> > > [javac] (at line 26)
> > > [javac] import java.io.ObjectInputStream;
> > > [javac] ^^^^^^^^^^^^^^^^^^^^^^^^^
> > > [javac] The import java.io.ObjectInputStream is never used
> > >
> > > [javac] 34. WARNING in
> > > C:\Work\Projects\Harmony-last\modules\concurrent\standard\src\main\java\java\util\concurrent\CountDownLatch.java
> > >
> > > [javac] (at line 130)
> > > [javac] private static final class Sync extends
> > > AbstractQueuedSynchronizer {
> > > [javac] ^^^^
> > > [javac] The serializable class Sync does not declare a static
> > > final serialVersionUID field of type long
> > >
> > > 2006/10/19, Nathan Beyer <nb...@gmail.com>:
> > >> I added a compiler arg to the main 'build-java.xml' compile to disable
> > >> all of the warnings.
> > >>
> > >> This should take care of the warnings.
> > >>
> > >> -Nathan
> > >>
> > >> On 10/18/06, Geir Magnusson Jr. <ge...@pobox.com> wrote:
> > >> > Can we just shut them off for now? it's a good project for someone to
> > >> > go and clean those up, but there's no need for all of us to see them.
> > >> >
> > >> > geir
> > >> >
> > >> > Mark Hindess wrote:
> > >> > > These messages aren't getting through on the commits list on due
> > >> to the
> > >> > > large number of warnings since switching to ecj. I've modified the
> > >> > > summary process so that it ignores these for the moment.
> > >> > >
> > >> > > Hopefully this means that the failure messages that haven't been
> > >> making
> > >> > > the commits list will start getting through.
> > >> > >
> > >> > > Regards,
> > >> > > Mark.
> > >> > >
> > >> > >
> > >> > >
> > >> > >
> > >> > > ---------------------------------------------------------------------
> > >> > > Terms of use : http://incubator.apache.org/harmony/mailing.html
> > >> > > To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> > >> > > For additional commands, e-mail:
> > >> harmony-dev-help@incubator.apache.org
> > >> > >
> > >> >
> > >> > ---------------------------------------------------------------------
> > >> > Terms of use : http://incubator.apache.org/harmony/mailing.html
> > >> > To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> > >> > For additional commands, e-mail: harmony-dev-help@incubator.apache.org
> > >> >
> > >> >
> > >>
> > >> ---------------------------------------------------------------------
> > >> Terms of use : http://incubator.apache.org/harmony/mailing.html
> > >> To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> > >> For additional commands, e-mail: harmony-dev-help@incubator.apache.org
> > >>
> > >>
> > >
> > >
> >
> > --
> >
> > Tim Ellison (t.p.ellison@gmail.com)
> > IBM Java technology centre, UK.
> >
> > ---------------------------------------------------------------------
> > Terms of use : http://incubator.apache.org/harmony/mailing.html
> > To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> > For additional commands, e-mail: harmony-dev-help@incubator.apache.org
> >
> >
>
> ---------------------------------------------------------------------
> Terms of use : http://incubator.apache.org/harmony/mailing.html
> To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> For additional commands, e-mail: harmony-dev-help@incubator.apache.org
>
>
--
Denis M. Kishenko
Enterprise Solutions Software Division
---------------------------------------------------------------------
Terms of use : http://incubator.apache.org/harmony/mailing.html
To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
For additional commands, e-mail: harmony-dev-help@incubator.apache.org
Re: [classlib] Build failure messages
Posted by Nathan Beyer <nb...@gmail.com>.
I've been trying to work through many of the warnings, module by
module. Most of the warnings fall into the following categories.
* serial - Serializable classes that don't have explicit serialVersionUID fields
* unchecked - missing type variables, mostly collection references
* unused - unused private fields, methods; these are tough as some of
them are seemingly used by native code
* missing @Override - methods that override other methods are
annotated for safety
-Nathan
On 10/19/06, Tim Ellison <t....@gmail.com> wrote:
> Nathan has turned off all warnings. I suggest that we start to turn
> them back on incrementally and fix the warnings we see.
>
> Regards,
> Tim
>
> Denis Kishenko wrote:
> > I also worried about a lot of warnings and I think warnings like these
> > can be fixed
> >
> > last\modules\concurrent\standard\src\main\java\java\util\concurrent\ConcurrentHashMap.java
> >
> > [javac] (at line 26)
> > [javac] import java.io.ObjectInputStream;
> > [javac] ^^^^^^^^^^^^^^^^^^^^^^^^^
> > [javac] The import java.io.ObjectInputStream is never used
> >
> > [javac] 34. WARNING in
> > C:\Work\Projects\Harmony-last\modules\concurrent\standard\src\main\java\java\util\concurrent\CountDownLatch.java
> >
> > [javac] (at line 130)
> > [javac] private static final class Sync extends
> > AbstractQueuedSynchronizer {
> > [javac] ^^^^
> > [javac] The serializable class Sync does not declare a static
> > final serialVersionUID field of type long
> >
> > 2006/10/19, Nathan Beyer <nb...@gmail.com>:
> >> I added a compiler arg to the main 'build-java.xml' compile to disable
> >> all of the warnings.
> >>
> >> This should take care of the warnings.
> >>
> >> -Nathan
> >>
> >> On 10/18/06, Geir Magnusson Jr. <ge...@pobox.com> wrote:
> >> > Can we just shut them off for now? it's a good project for someone to
> >> > go and clean those up, but there's no need for all of us to see them.
> >> >
> >> > geir
> >> >
> >> > Mark Hindess wrote:
> >> > > These messages aren't getting through on the commits list on due
> >> to the
> >> > > large number of warnings since switching to ecj. I've modified the
> >> > > summary process so that it ignores these for the moment.
> >> > >
> >> > > Hopefully this means that the failure messages that haven't been
> >> making
> >> > > the commits list will start getting through.
> >> > >
> >> > > Regards,
> >> > > Mark.
> >> > >
> >> > >
> >> > >
> >> > >
> >> > > ---------------------------------------------------------------------
> >> > > Terms of use : http://incubator.apache.org/harmony/mailing.html
> >> > > To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> >> > > For additional commands, e-mail:
> >> harmony-dev-help@incubator.apache.org
> >> > >
> >> >
> >> > ---------------------------------------------------------------------
> >> > Terms of use : http://incubator.apache.org/harmony/mailing.html
> >> > To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> >> > For additional commands, e-mail: harmony-dev-help@incubator.apache.org
> >> >
> >> >
> >>
> >> ---------------------------------------------------------------------
> >> Terms of use : http://incubator.apache.org/harmony/mailing.html
> >> To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> >> For additional commands, e-mail: harmony-dev-help@incubator.apache.org
> >>
> >>
> >
> >
>
> --
>
> Tim Ellison (t.p.ellison@gmail.com)
> IBM Java technology centre, UK.
>
> ---------------------------------------------------------------------
> Terms of use : http://incubator.apache.org/harmony/mailing.html
> To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> For additional commands, e-mail: harmony-dev-help@incubator.apache.org
>
>
---------------------------------------------------------------------
Terms of use : http://incubator.apache.org/harmony/mailing.html
To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
For additional commands, e-mail: harmony-dev-help@incubator.apache.org
Re: [classlib] Build failure messages
Posted by Tim Ellison <t....@gmail.com>.
Nathan has turned off all warnings. I suggest that we start to turn
them back on incrementally and fix the warnings we see.
Regards,
Tim
Denis Kishenko wrote:
> I also worried about a lot of warnings and I think warnings like these
> can be fixed
>
> last\modules\concurrent\standard\src\main\java\java\util\concurrent\ConcurrentHashMap.java
>
> [javac] (at line 26)
> [javac] import java.io.ObjectInputStream;
> [javac] ^^^^^^^^^^^^^^^^^^^^^^^^^
> [javac] The import java.io.ObjectInputStream is never used
>
> [javac] 34. WARNING in
> C:\Work\Projects\Harmony-last\modules\concurrent\standard\src\main\java\java\util\concurrent\CountDownLatch.java
>
> [javac] (at line 130)
> [javac] private static final class Sync extends
> AbstractQueuedSynchronizer {
> [javac] ^^^^
> [javac] The serializable class Sync does not declare a static
> final serialVersionUID field of type long
>
> 2006/10/19, Nathan Beyer <nb...@gmail.com>:
>> I added a compiler arg to the main 'build-java.xml' compile to disable
>> all of the warnings.
>>
>> This should take care of the warnings.
>>
>> -Nathan
>>
>> On 10/18/06, Geir Magnusson Jr. <ge...@pobox.com> wrote:
>> > Can we just shut them off for now? it's a good project for someone to
>> > go and clean those up, but there's no need for all of us to see them.
>> >
>> > geir
>> >
>> > Mark Hindess wrote:
>> > > These messages aren't getting through on the commits list on due
>> to the
>> > > large number of warnings since switching to ecj. I've modified the
>> > > summary process so that it ignores these for the moment.
>> > >
>> > > Hopefully this means that the failure messages that haven't been
>> making
>> > > the commits list will start getting through.
>> > >
>> > > Regards,
>> > > Mark.
>> > >
>> > >
>> > >
>> > >
>> > > ---------------------------------------------------------------------
>> > > Terms of use : http://incubator.apache.org/harmony/mailing.html
>> > > To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
>> > > For additional commands, e-mail:
>> harmony-dev-help@incubator.apache.org
>> > >
>> >
>> > ---------------------------------------------------------------------
>> > Terms of use : http://incubator.apache.org/harmony/mailing.html
>> > To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
>> > For additional commands, e-mail: harmony-dev-help@incubator.apache.org
>> >
>> >
>>
>> ---------------------------------------------------------------------
>> Terms of use : http://incubator.apache.org/harmony/mailing.html
>> To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
>> For additional commands, e-mail: harmony-dev-help@incubator.apache.org
>>
>>
>
>
--
Tim Ellison (t.p.ellison@gmail.com)
IBM Java technology centre, UK.
---------------------------------------------------------------------
Terms of use : http://incubator.apache.org/harmony/mailing.html
To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
For additional commands, e-mail: harmony-dev-help@incubator.apache.org
Re: [classlib] Build failure messages
Posted by Denis Kishenko <dk...@gmail.com>.
I also worried about a lot of warnings and I think warnings like these
can be fixed
last\modules\concurrent\standard\src\main\java\java\util\concurrent\ConcurrentHashMap.java
[javac] (at line 26)
[javac] import java.io.ObjectInputStream;
[javac] ^^^^^^^^^^^^^^^^^^^^^^^^^
[javac] The import java.io.ObjectInputStream is never used
[javac] 34. WARNING in
C:\Work\Projects\Harmony-last\modules\concurrent\standard\src\main\java\java\util\concurrent\CountDownLatch.java
[javac] (at line 130)
[javac] private static final class Sync extends
AbstractQueuedSynchronizer {
[javac] ^^^^
[javac] The serializable class Sync does not declare a static
final serialVersionUID field of type long
2006/10/19, Nathan Beyer <nb...@gmail.com>:
> I added a compiler arg to the main 'build-java.xml' compile to disable
> all of the warnings.
>
> This should take care of the warnings.
>
> -Nathan
>
> On 10/18/06, Geir Magnusson Jr. <ge...@pobox.com> wrote:
> > Can we just shut them off for now? it's a good project for someone to
> > go and clean those up, but there's no need for all of us to see them.
> >
> > geir
> >
> > Mark Hindess wrote:
> > > These messages aren't getting through on the commits list on due to the
> > > large number of warnings since switching to ecj. I've modified the
> > > summary process so that it ignores these for the moment.
> > >
> > > Hopefully this means that the failure messages that haven't been making
> > > the commits list will start getting through.
> > >
> > > Regards,
> > > Mark.
> > >
> > >
> > >
> > >
> > > ---------------------------------------------------------------------
> > > Terms of use : http://incubator.apache.org/harmony/mailing.html
> > > To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> > > For additional commands, e-mail: harmony-dev-help@incubator.apache.org
> > >
> >
> > ---------------------------------------------------------------------
> > Terms of use : http://incubator.apache.org/harmony/mailing.html
> > To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> > For additional commands, e-mail: harmony-dev-help@incubator.apache.org
> >
> >
>
> ---------------------------------------------------------------------
> Terms of use : http://incubator.apache.org/harmony/mailing.html
> To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> For additional commands, e-mail: harmony-dev-help@incubator.apache.org
>
>
--
Denis M. Kishenko
Enterprise Solutions Software Division
---------------------------------------------------------------------
Terms of use : http://incubator.apache.org/harmony/mailing.html
To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
For additional commands, e-mail: harmony-dev-help@incubator.apache.org
Re: [classlib] Build failure messages
Posted by Paulex Yang <pa...@gmail.com>.
Nathan Beyer wrote:
> I added a compiler arg to the main 'build-java.xml' compile to disable
> all of the warnings.
Thanks a lot, Nathan, but how about also add that arg to build-test.xml?
I think the ECJ probably feels more unhappy with the test cases:).
>
> This should take care of the warnings.
>
> -Nathan
>
> On 10/18/06, Geir Magnusson Jr. <ge...@pobox.com> wrote:
>> Can we just shut them off for now? it's a good project for someone to
>> go and clean those up, but there's no need for all of us to see them.
>>
>> geir
>>
>> Mark Hindess wrote:
>> > These messages aren't getting through on the commits list on due to
>> the
>> > large number of warnings since switching to ecj. I've modified the
>> > summary process so that it ignores these for the moment.
>> >
>> > Hopefully this means that the failure messages that haven't been
>> making
>> > the commits list will start getting through.
>> >
>> > Regards,
>> > Mark.
>> >
>> >
>> >
>> >
>> > ---------------------------------------------------------------------
>> > Terms of use : http://incubator.apache.org/harmony/mailing.html
>> > To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
>> > For additional commands, e-mail: harmony-dev-help@incubator.apache.org
>> >
>>
>> ---------------------------------------------------------------------
>> Terms of use : http://incubator.apache.org/harmony/mailing.html
>> To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
>> For additional commands, e-mail: harmony-dev-help@incubator.apache.org
>>
>>
>
> ---------------------------------------------------------------------
> Terms of use : http://incubator.apache.org/harmony/mailing.html
> To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> For additional commands, e-mail: harmony-dev-help@incubator.apache.org
>
>
--
Paulex Yang
China Software Development Lab
IBM
---------------------------------------------------------------------
Terms of use : http://incubator.apache.org/harmony/mailing.html
To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
For additional commands, e-mail: harmony-dev-help@incubator.apache.org
Re: [classlib] Build failure messages
Posted by "Geir Magnusson Jr." <ge...@pobox.com>.
coolio - thx
Nathan Beyer wrote:
> I added a compiler arg to the main 'build-java.xml' compile to disable
> all of the warnings.
>
> This should take care of the warnings.
>
> -Nathan
>
> On 10/18/06, Geir Magnusson Jr. <ge...@pobox.com> wrote:
>> Can we just shut them off for now? it's a good project for someone to
>> go and clean those up, but there's no need for all of us to see them.
>>
>> geir
>>
>> Mark Hindess wrote:
>> > These messages aren't getting through on the commits list on due to the
>> > large number of warnings since switching to ecj. I've modified the
>> > summary process so that it ignores these for the moment.
>> >
>> > Hopefully this means that the failure messages that haven't been making
>> > the commits list will start getting through.
>> >
>> > Regards,
>> > Mark.
>> >
>> >
>> >
>> >
>> > ---------------------------------------------------------------------
>> > Terms of use : http://incubator.apache.org/harmony/mailing.html
>> > To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
>> > For additional commands, e-mail: harmony-dev-help@incubator.apache.org
>> >
>>
>> ---------------------------------------------------------------------
>> Terms of use : http://incubator.apache.org/harmony/mailing.html
>> To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
>> For additional commands, e-mail: harmony-dev-help@incubator.apache.org
>>
>>
>
> ---------------------------------------------------------------------
> Terms of use : http://incubator.apache.org/harmony/mailing.html
> To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> For additional commands, e-mail: harmony-dev-help@incubator.apache.org
>
---------------------------------------------------------------------
Terms of use : http://incubator.apache.org/harmony/mailing.html
To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
For additional commands, e-mail: harmony-dev-help@incubator.apache.org
Re: [classlib] Build failure messages
Posted by Nathan Beyer <nb...@gmail.com>.
I added a compiler arg to the main 'build-java.xml' compile to disable
all of the warnings.
This should take care of the warnings.
-Nathan
On 10/18/06, Geir Magnusson Jr. <ge...@pobox.com> wrote:
> Can we just shut them off for now? it's a good project for someone to
> go and clean those up, but there's no need for all of us to see them.
>
> geir
>
> Mark Hindess wrote:
> > These messages aren't getting through on the commits list on due to the
> > large number of warnings since switching to ecj. I've modified the
> > summary process so that it ignores these for the moment.
> >
> > Hopefully this means that the failure messages that haven't been making
> > the commits list will start getting through.
> >
> > Regards,
> > Mark.
> >
> >
> >
> >
> > ---------------------------------------------------------------------
> > Terms of use : http://incubator.apache.org/harmony/mailing.html
> > To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> > For additional commands, e-mail: harmony-dev-help@incubator.apache.org
> >
>
> ---------------------------------------------------------------------
> Terms of use : http://incubator.apache.org/harmony/mailing.html
> To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> For additional commands, e-mail: harmony-dev-help@incubator.apache.org
>
>
---------------------------------------------------------------------
Terms of use : http://incubator.apache.org/harmony/mailing.html
To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
For additional commands, e-mail: harmony-dev-help@incubator.apache.org
Re: [classlib] Build failure messages
Posted by "Geir Magnusson Jr." <ge...@pobox.com>.
Can we just shut them off for now? it's a good project for someone to
go and clean those up, but there's no need for all of us to see them.
geir
Mark Hindess wrote:
> These messages aren't getting through on the commits list on due to the
> large number of warnings since switching to ecj. I've modified the
> summary process so that it ignores these for the moment.
>
> Hopefully this means that the failure messages that haven't been making
> the commits list will start getting through.
>
> Regards,
> Mark.
>
>
>
>
> ---------------------------------------------------------------------
> Terms of use : http://incubator.apache.org/harmony/mailing.html
> To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
> For additional commands, e-mail: harmony-dev-help@incubator.apache.org
>
---------------------------------------------------------------------
Terms of use : http://incubator.apache.org/harmony/mailing.html
To unsubscribe, e-mail: harmony-dev-unsubscribe@incubator.apache.org
For additional commands, e-mail: harmony-dev-help@incubator.apache.org