RE: [cdt-dev] Warnings in ui-plugin

[Chris Recoskie <recoskie@xxxxxxxxxx>]

> The main problem is that people use different compiler settings, so they
> just don't see the warnings you are getting. Maybe we should come up
> with a minimal set of the compiler settings.

Agreed.  No reason why we can't store these in the prefs for each project
and then unless someone changes the default, when they check out the
projects from CVS and build them they'll get the same settings.  I would
suggest changing the build settings for those projects to fail on warnings
as well.

Part of the problem as well as there is a sea of warnings that occur when
you build other CDT plugins, or the platform plugins.  When you have
thousands of errors, new ones you introduce can get lost, or don't even
show up in the problems view because it's set by default to only display
100 warnings.  Not that we shouldn't manually go inspect the UI project
when we build it to make sure there are no warnings there, but it is a task
easily forgotten.

===========================

Chris Recoskie
Team Lead, IBM CDT Team
IBM Toronto
http://www.eclipse.org/cdt



                                                                           
  From:   "Mikhail Khodjaiants" <Mikhail.Khodjaiants@xxxxxxx>              
                                                                           
  To:     "CDT General developers list." <cdt-dev@xxxxxxxxxxx>             
                                                                           
  Date:   21/02/2007 08:50 AM                                              
                                                                           
  Subject RE: [cdt-dev] Warnings in ui-plugin                              
  :                                                                        
                                                                           




Markus,

> I think it is a good habit to keep the code clean

I absolutely agree with you and this applies to all plugins. I had been
trying to keep the debug plugins clean before 3.1, but now it's almost
impossible.
The main problem is that people use different compiler settings, so they
just don't see the warnings you are getting. Maybe we should come up
with a minimal set of the compiler settings.

Mikhail

-----Original Message-----
From: cdt-dev-bounces@xxxxxxxxxxx [mailto:cdt-dev-bounces@xxxxxxxxxxx]
On Behalf Of Schorn, Markus
Sent: 21 February 2007 13:21
To: CDT General developers list.
Subject: [cdt-dev] Warnings in ui-plugin

Hi,
Half a year ago, I invested into fixing all the warnings in the
ui-plugin. Recently by adding the language-mappings and the new project
model new warnings were introduced.

I think it is a good habit to keep the code clean and once more I ask
for your support in doing this. Especially new functionality added
should be free of warnings.

Please fix the warnings or explain to me why I should not care.

Markus.
_______________________________________________
cdt-dev mailing list
cdt-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/cdt-dev

--
IMPORTANT NOTICE: The contents of this email and any attachments are
confidential and may also be privileged. If you are not the intended
recipient, please notify the sender immediately and do not disclose the
contents to any other person, use it for any purpose, or store or copy the
information in any medium.  Thank you.


_______________________________________________
cdt-dev mailing list
cdt-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/cdt-dev