From dfbeb0610efc06efbbf071c0a05bde9361ca8ed7 Mon Sep 17 00:00:00 2001 From: Gabriel Dos Reis Date: Sat, 21 Jun 2003 13:09:05 +0000 Subject: re PR c++/10915 (a not-useful non-avoidable warning: conversion to a reference to the same type will never use a type conversion operator) testsuite/ * g++.old-deja/g++.jason/conversion5.C: Adjust option. * g++.old-deja/g++.bugs/900215_01.C: Likewise. cp/ PR c++/10915 * decl.c (grok_op_properties): Warn possible confusing conversion only if -Wconversion. From-SVN: r68303 --- gcc/cp/ChangeLog | 6 ++++++ gcc/cp/decl.c | 2 +- gcc/testsuite/ChangeLog | 5 +++++ gcc/testsuite/g++.dg/warn/conversion-function-1.C | 8 ++++++++ gcc/testsuite/g++.old-deja/g++.bugs/900215_01.C | 2 ++ gcc/testsuite/g++.old-deja/g++.jason/conversion5.C | 1 + 6 files changed, 23 insertions(+), 1 deletion(-) create mode 100644 gcc/testsuite/g++.dg/warn/conversion-function-1.C (limited to 'gcc') diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog index 88c91db..7f0c4ed 100644 --- a/gcc/cp/ChangeLog +++ b/gcc/cp/ChangeLog @@ -1,3 +1,9 @@ +2003-06-21 Gabriel Dos Reis + + PR c++/10915 + * decl.c (grok_op_properties): Warn possible confusing conversion + only if -Wconversion. + 2003-06-20 Mark Mitchell PR c++/10749 diff --git a/gcc/cp/decl.c b/gcc/cp/decl.c index d73c0d0..f539842 100644 --- a/gcc/cp/decl.c +++ b/gcc/cp/decl.c @@ -12371,7 +12371,7 @@ grok_op_properties (tree decl, int friendp) && DERIVED_FROM_P (t, current_class_type)) what = "a base class"; - if (what) + if (what && warn_conversion) warning ("conversion to %s%s will never use a type conversion operator", ref ? "a reference to " : "", what); } diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index a0b9777..a947a8f 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2003-06-21 Gabriel Dos Reis + + * g++.old-deja/g++.jason/conversion5.C: Adjust option. + * g++.old-deja/g++.bugs/900215_01.C: Likewise. + 2003-06-20 Mark Mitchell PR c++/10888 diff --git a/gcc/testsuite/g++.dg/warn/conversion-function-1.C b/gcc/testsuite/g++.dg/warn/conversion-function-1.C new file mode 100644 index 0000000..878011c --- /dev/null +++ b/gcc/testsuite/g++.dg/warn/conversion-function-1.C @@ -0,0 +1,8 @@ +// Copyright (C) 2003 Free Software Foundation +// Contributed by Gabriel Dos Reis +// { dg-options "-Wno-conversion" } + +struct A { + operator A&(); +}; + diff --git a/gcc/testsuite/g++.old-deja/g++.bugs/900215_01.C b/gcc/testsuite/g++.old-deja/g++.bugs/900215_01.C index 7681057..c79a703 100644 --- a/gcc/testsuite/g++.old-deja/g++.bugs/900215_01.C +++ b/gcc/testsuite/g++.old-deja/g++.bugs/900215_01.C @@ -20,6 +20,8 @@ // 8/3/2000 (nathan): The std allows you to define such an op, but // it will never be called. [class.conv.fct]. Make it an unconditional warning. +// { dg-options "-Wconversion" } + struct struct0 { operator void (); // { dg-warning "" } operator void diff --git a/gcc/testsuite/g++.old-deja/g++.jason/conversion5.C b/gcc/testsuite/g++.old-deja/g++.jason/conversion5.C index e409fae..0031084 100644 --- a/gcc/testsuite/g++.old-deja/g++.jason/conversion5.C +++ b/gcc/testsuite/g++.old-deja/g++.jason/conversion5.C @@ -1,4 +1,5 @@ // { dg-do assemble } +// { dg-options "-Wconversion" } struct A { }; struct B: public A { A a; -- cgit v1.1