[
Date Prev][
Date Next][
Thread Prev][
Thread Next][
Date Index][
Thread Index]
[
List Home]
|
RE: [cdt-dev] Warnings and SuppressWarnings
|
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
