> Subject: RE: [cdt-dev] Warnings and SuppressWarnings
> Importance: Low
>
> I would never veto something that people clearly want.
>
> May I make a few more suggestions (based on the current
> settings in the UI
> plugin) :
>
> redundant null check -> warning
> unnecessary else statement -> warning
> redundant super interface -> warning
> usage of raw type -> warning
> missing @override annotation -> warning
> missing @deprecated annotation -> warning unused import -> error
>
>
> Mike Kucera
> Software Developer
> IBM CDT Team, Toronto
>
mkucera@xxxxxxxxxx
>
>
>
>
> "Schorn, Markus"
>
> <Markus.Schorn@wi
>
>
ndriver.com>
> To
> Sent by: "CDT General
> developers list."
> cdt-dev-bounces@e <
cdt-dev@xxxxxxxxxxx>
>
>
clipse.org
> cc
>
>
>
> Subject
> 04/08/2008 04:56 RE: [cdt-dev] Warnings
> and
> AM SuppressWarnings
>
>
>
>
>
> Please respond to
>
> "CDT General
>
> developers list."
>
> <cdt-dev@eclipse.
>
> org>
>
>
>
>
>
>
>
>
>
> Ok, I'd like to increase the warning levels:
> "Enum type constant not covered on switch": none -> warning
> "Null pointer access": warning -> error
> "Potential null pointer access": none -> warning
> "switch case fall-through":
> none -> warning
>
> Please reply, if you have an objection.
>
>
> Mike, if you don't explicitly veto this, I'll enable the
> shared warning-settings for the core-plugin, also.
>
> Markus.
>
>
> > -----Original Message-----
> > From:
cdt-dev-bounces@xxxxxxxxxxx
> > [mailto:
cdt-dev-bounces@xxxxxxxxxxx] On Behalf Of Mike Kucera
> > Sent: Tuesday, April 08, 2008 1:59 AM
> > To: CDT General developers list.
> > Subject: Re: [cdt-dev] Warnings and SuppressWarnings
> > Importance: Low
> >
> > +1
> >
> > I like the idea of turning some warnings into errors.
> >
> > Mike Kucera
> > Software Developer
> > IBM CDT Team, Toronto
> >
mkucera@xxxxxxxxxx
> >
> >
> >
> >
> > Elena Laskavaia
> >
> > <ELaskavaia@qnx.c
> >
> > om>
> > To
> > Sent by: "CDT General
> > developers list."
> > cdt-dev-bounces@e <
cdt-dev@xxxxxxxxxxx>
> >
> >
clipse.org
> > cc
> >
> >
> >
> > Subject
> > 04/07/2008 12:38 Re: [cdt-dev] Warnings
> > and
> > PM SuppressWarnings
> >
> >
> >
> >
> >
> > Please respond to
> >
> > "CDT General
> >
> > developers list."
> >
> > <cdt-dev@eclipse.
> >
> > org>
> >
> >
> >
> >
> >
> >
> >
> >
> >
> > I think it is good idea. But we should enable as much as we can and
> > turn some warning into errors, such as unused import.
> >
> > Schorn, Markus wrote:
> > >
> > > Hi,
> > > I am getting a bunch of warnings in the core-plugin of
> the following
> > > kind: Unnecessary @SuppressWarnings. I assume this is because my
> > > compiler warning settings differ from the ones of others.
> > >
> > > Therefore I suggest to use a common set of warnings for all
> > of us and
> > > check it into CVS (like we are doing this in the cdt.ui plugin).
> > >
> > > Has anyone objections against this? I'd start out with the same
> > > settings as we are using for the ui-plugin.
> > >
> > > Markus.
> > > _______________________________________________
> > > cdt-dev mailing list
> > >
cdt-dev@xxxxxxxxxxx
> > >
https://dev.eclipse.org/mailman/listinfo/cdt-dev
> > >
> > _______________________________________________
> > cdt-dev mailing list
> >
cdt-dev@xxxxxxxxxxx
> >
https://dev.eclipse.org/mailman/listinfo/cdt-dev
> >
> >
> > _______________________________________________
> > cdt-dev mailing list
> >
cdt-dev@xxxxxxxxxxx
> >
https://dev.eclipse.org/mailman/listinfo/cdt-dev
> >
> _______________________________________________
> cdt-dev mailing list
>
cdt-dev@xxxxxxxxxxx
>
https://dev.eclipse.org/mailman/listinfo/cdt-dev
>
>
> _______________________________________________
> cdt-dev mailing list
>
cdt-dev@xxxxxxxxxxx
>
https://dev.eclipse.org/mailman/listinfo/cdt-dev
>
_______________________________________________
cdt-dev mailing list
cdt-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/cdt-dev
