getting a handle on build warnings

Jonathan Gibbons Jonathan.Gibbons at Sun.COM
Fri Jul 11 22:20:24 UTC 2008


Well, here's an unexpected initial result.

I just ran a build of langtools+jdk on my ubuntu laptop.   I got a  
whopping 2658 warnings!!  [Those who volunteered to help get rid of  
all the warnings, don't all step back at once!]  But surprisingly,  
after a quick "sort -u", only 625 of them are unique. So that means we  
get on average a 4x bang for the buck for each warning we fix :-)

-- Jon


On Jul 11, 2008, at 11:02 AM, Jonathan Gibbons wrote:

> Depending on what lint options you use, deprecation warnings are  
> typically reported as a single "Note:" at the end of the  
> compilation, rather than as individual warning messages.  Ideally,  
> they should go too, but for now, I'd settle for removing messages  
> that show up as diagnostics in IDEs, emacs, etc.
>
> For my part, I work mostly in the langtools area,  and we are having  
> a general background effort to clean up the code there. The goal is  
> to clean the code and then use -Werror to treat any reappearance of  
> warnings as errors.
>
> -- Jon
>
>
> On Jul 11, 2008, at 10:52 AM, Rob Ross wrote:
>
>> Just curious, would part of this revision process entail removing  
>> calls to deprecated methods and replacing them with their  
>> documented replacement methods?
>>
>> There are many warnings about calls to deprecated methods in the  
>> OpenJDK code.
>>
>>
>> Rob Ross, Lead Software Engineer
>> E! Networks
>>
>> ---------------------------------------------------
>> "Beware of he who would deny you access to information, for in his  
>> heart he dreams himself your master." -- Commissioner Pravin Lal
>>
>>
>>
>> On Jul 11, 2008, at 10:43 AM, Jonathan Gibbons wrote:
>>
>>>
>>> On Jul 11, 2008, at 10:00 AM, Thorbjørn Ravn Andersen wrote:
>>>
>>>> Jonathan Gibbons skrev  den 11-07-2008 13:52:
>>>>>
>>>>> Yes, that technique can work well. But either way, the next step  
>>>>> is
>>>>> to try writing the code to analyze the build log, to see how far  
>>>>> the general
>>>>> idea can be taken, and how much interest there is to track/fix  
>>>>> warnings.
>>>>>
>>>> Personally I'd like there to be no warnings at all, and would  
>>>> like to contribute work to get there
>>>>
>>>
>>> Thank you.
>>>
>>> Getting rid of the actual warnings will require cooperation from  
>>> the teams who are
>>> responsible for the various parts of the code. In some cases,  
>>> there may be resistance,
>>> since stability is sometimes preferred over "no warnings".  That  
>>> being said, what I'm
>>> hoping to achieve with this discussion and any related effort is a  
>>> non-intrusive way
>>> of gathering information about warnings, so that we have a better  
>>> understanding of
>>> where the warnings are, in what areas of the code and in which  
>>> source files.  That way,
>>> we can work with the teams involved to see if there is interest in  
>>> reducing their
>>> warning count in as safe a way as possible.  Any assistance in  
>>> getting to that point
>>> would be welcome.
>>>
>>> (Note that anyone contributing to OpenJDK must first sign the Sun  
>>> Contributor Agreement;
>>> you can find details at http://sca.dev.java.net.)
>>>
>>> -- Jon
>>>
>>>
>>
>




More information about the build-dev mailing list