* [patch] Expand comment about complain() replacement
@ 2002-11-15 12:57 Andrew Cagney
2002-11-15 13:09 ` Andrew Cagney
0 siblings, 1 reply; 2+ messages in thread
From: Andrew Cagney @ 2002-11-15 12:57 UTC (permalink / raw)
To: gdb-patches
[-- Attachment #1: Type: text/plain, Size: 106 bytes --]
Hello,
This patch just expands the description of what replaced complain() a
little.
committed,
Andrew
[-- Attachment #2: diffs --]
[-- Type: text/plain, Size: 1500 bytes --]
2002-11-15 Andrew Cagney <ac131313@redhat.com>
* complaints.h: Add comment explaining how to eliminate a
deprecated_complain call.
Index: complaints.h
===================================================================
RCS file: /cvs/src/src/gdb/complaints.h,v
retrieving revision 1.5
diff -u -r1.5 complaints.h
--- complaints.h 25 Oct 2002 22:25:55 -0000 1.5
+++ complaints.h 15 Nov 2002 20:54:02 -0000
@@ -53,12 +53,24 @@
/* Deprecated interfaces to keep the old code working (until it is all
converted to the above). Existing code such as:
- struct deprecated_complaint msg = { "msg", 0, 0 };
- complaint (&msg);
+ struct deprecated_complaint msg = { "msg 0x%08x[sic]", 0, 0 };
+ deprecated_complain (&msg, addr);
- should be replaced by:
+ should be replaced by either the new call (for the singular case):
- complaint (&symtab_complaints, __FILE__, __LINE__, "msg");
+ complaint (&symtab_complaints, "msg 0x%s", paddr (addr));
+
+ or with a wrapper function (for the many-of case):
+
+ msg_complaint (CORE_ADDR addr)
+ { complaint (&symtab_complaints, "msg 0x%s", paddr (addr)); }
+ ...
+ msg_complaint (addr);
+
+ Yes, the typo is intentional. The motivation behind this interface
+ change is to eliminate all possability of this problem re-occuring
+ (it has occured in the past and no one is sure that it isn't
+ present now).
Support for complaining about things in the symbol file that aren't
catastrophic.
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [patch] Expand comment about complain() replacement
2002-11-15 12:57 [patch] Expand comment about complain() replacement Andrew Cagney
@ 2002-11-15 13:09 ` Andrew Cagney
0 siblings, 0 replies; 2+ messages in thread
From: Andrew Cagney @ 2002-11-15 13:09 UTC (permalink / raw)
To: Andrew Cagney; +Cc: gdb-patches
[-- Attachment #1: Type: text/plain, Size: 31 bytes --]
[as was pointed out privately]
[-- Attachment #2: diffs --]
[-- Type: text/plain, Size: 799 bytes --]
* complaints.h: Fix spelling errors.
Index: complaints.h
===================================================================
RCS file: /cvs/src/src/gdb/complaints.h,v
retrieving revision 1.6
diff -u -r1.6 complaints.h
--- complaints.h 15 Nov 2002 20:55:45 -0000 1.6
+++ complaints.h 15 Nov 2002 21:07:12 -0000
@@ -68,8 +68,8 @@
msg_complaint (addr);
Yes, the typo is intentional. The motivation behind this interface
- change is to eliminate all possability of this problem re-occuring
- (it has occured in the past and no one is sure that it isn't
+ change is to eliminate all possibility of this problem re-occurring
+ (it has occurred in the past and no one is sure that it isn't
present now).
Support for complaining about things in the symbol file that aren't
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2002-11-15 21:09 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2002-11-15 12:57 [patch] Expand comment about complain() replacement Andrew Cagney
2002-11-15 13:09 ` Andrew Cagney
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox