Patchwork committed: hook signture patches (12/11): comment fixes

login
register
mail settings
Submitter Joern Rennecke
Date June 30, 2010, 6:31 p.m.
Message ID <20100630143120.jrj3lq8bokkg84o4-nzlynne@webmail.spamcop.net>
Download mbox | patch
Permalink /patch/57444/
State New
Headers show

Comments

Joern Rennecke - June 30, 2010, 6:31 p.m.
Quoting Richard Henderson <rth@redhat.com>:

> All 11 patches are ok.

Thank you for the review.

I found that some comment-only changes fell by the wayside when I chopped
up the patch into pieces dealing with each changed target hook.
As these were basically restatements of the comment that was modified
in patch #1, I've checked this in as obvious.
2010-06-30  Joern Rennecke  <joern.rennecke@embecosm.com>

	* target.def (print_operand, print_operand_address): Update comment.
	(print_operand_punct_valid_p): Likewise.

Patch

Index: target.def
===================================================================
--- target.def	(revision 161628)
+++ target.def	(working copy)
@@ -411,9 +411,8 @@  DEFHOOK
 #define HOOK_PREFIX "TARGET_"
 
 /* Emit a machine-specific insn operand.  */
-/* ??? Documenting the argument types for this hook requires a GFDL
-   license grant.  Also, the documentation documents this as a macro,
-   not a hook, and uses a different name for the hook argument FILE.  */
+/* ??? tm.texi only documents the old macro PRINT_OPERAND,
+   not this  hook, and uses a different name for the argument FILE.  */
 DEFHOOK_UNDOC
 (print_operand,
  "",
@@ -421,9 +420,8 @@  DEFHOOK_UNDOC
  default_print_operand)
 
 /* Emit a machine-specific memory address.  */
-/* ??? Documenting the argument types for this hook requires a GFDL
-   license grant.  Also, the documentation documents this as a macro,
-   not a hook, and uses different hook argument names.  */
+/* ??? tm.texi only documents the old macro PRINT_OPERAND_ADDRESS,
+   not this  hook, and uses different argument names.  */
 DEFHOOK_UNDOC
 (print_operand_address,
  "",
@@ -432,9 +430,8 @@  DEFHOOK_UNDOC
 
 /* Determine whether CODE is a valid punctuation character for the
    `print_operand' hook.  */
-/* ??? Documenting the argument types for this hook requires a GFDL
-   license grant.  Also, the documentation documents this as a macro,
-   not a hook.  */
+/* ??? tm.texi only documents the old macro PRINT_OPERAND_PUNCT_VALID_P,
+   not this  hook.  */
 DEFHOOK_UNDOC
 (print_operand_punct_valid_p,
  "",