public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [C++ Patch] Couple of minor fixes
@ 2014-05-12 10:33 Paolo Carlini
  2014-05-12 14:04 ` Jason Merrill
  0 siblings, 1 reply; 2+ messages in thread
From: Paolo Carlini @ 2014-05-12 10:33 UTC (permalink / raw)
  To: gcc-patches; +Cc: Jason Merrill

[-- Attachment #1: Type: text/plain, Size: 89 bytes --]

Hi,

almost obvious, I would say. Tested x86_64-linux.

Thanks,
Paolo.

////////////////

[-- Attachment #2: CL --]
[-- Type: text/plain, Size: 234 bytes --]

2014-05-12  Paolo Carlini  <paolo.carlini@oracle.com>

	* cvt.c (cp_convert_to_pointer): Don't call error_at if
	complain & tf_error is false.

	* decl.c (make_unbound_class_template): Prefer inform for
	"declared here"-type message.

[-- Attachment #3: patchlet --]
[-- Type: text/plain, Size: 973 bytes --]

Index: cvt.c
===================================================================
--- cvt.c	(revision 210320)
+++ cvt.c	(working copy)
@@ -198,8 +198,9 @@ cp_convert_to_pointer (tree type, tree expr, tsubs
 						       complain);
 	    }
 	}
-      error_at (loc, "cannot convert %qE from type %qT to type %qT",
-		expr, intype, type);
+      if (complain & tf_error)
+	error_at (loc, "cannot convert %qE from type %qT to type %qT",
+		  expr, intype, type);
       return error_mark_node;
     }
 
Index: decl.c
===================================================================
--- decl.c	(revision 210320)
+++ decl.c	(working copy)
@@ -3491,7 +3491,8 @@ make_unbound_class_template (tree context, tree na
 	  if (complain & tf_error)
 	    {
 	      error ("template parameters do not match template");
-	      error ("%q+D declared here", tmpl);
+	      inform (DECL_SOURCE_LOCATION (tmpl),
+		      "%qD declared here", tmpl);
 	    }
 	  return error_mark_node;
 	}

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [C++ Patch] Couple of minor fixes
  2014-05-12 10:33 [C++ Patch] Couple of minor fixes Paolo Carlini
@ 2014-05-12 14:04 ` Jason Merrill
  0 siblings, 0 replies; 2+ messages in thread
From: Jason Merrill @ 2014-05-12 14:04 UTC (permalink / raw)
  To: Paolo Carlini, gcc-patches

Yes, those seem obvious.

Jason

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2014-05-12 14:04 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-05-12 10:33 [C++ Patch] Couple of minor fixes Paolo Carlini
2014-05-12 14:04 ` Jason Merrill

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).