Re: [classlib] Build failure messages

[Geir Magnusson Jr.]

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 <[EMAIL PROTECTED]> 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 <[EMAIL PROTECTED]>:
>> 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. <[EMAIL PROTECTED]> 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: [EMAIL PROTECTED]

>> > > For additional commands, e-mail:
>> [EMAIL PROTECTED]
>> > >
>> >

>> > ---------------------------------------------------------------------

>> > Terms of use : http://incubator.apache.org/harmony/mailing.html
>> > To unsubscribe, e-mail: [EMAIL PROTECTED]

>> > For additional commands, e-mail: [EMAIL PROTECTED]

>> >
>> >
>>
>> ---------------------------------------------------------------------
>> Terms of use : http://incubator.apache.org/harmony/mailing.html
>> To unsubscribe, e-mail: [EMAIL PROTECTED]
>> For additional commands, e-mail: [EMAIL PROTECTED]
>>
>>
>
>

--

Tim Ellison ([EMAIL PROTECTED])
IBM Java technology centre, UK.

---------------------------------------------------------------------
Terms of use : http://incubator.apache.org/harmony/mailing.html
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

---------------------------------------------------------------------
Terms of use : http://incubator.apache.org/harmony/mailing.html
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

---------------------------------------------------------------------
Terms of use : http://incubator.apache.org/harmony/mailing.html
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]