Merge bitcoin#29633: log: Remove error() reference

fanquake · vijaydasmp · commit 737bd9df8399 · 2026-06-21T23:12:06.000+05:30
d0e6564 log: Remove error() reference (Fabian Jahr) Pull request description: Mini-followup to bitcoin#29236 that was just merged. Removes a reference to `error()` that was missed in a comment. ACKs for top commit: ryanofsky: Code review ACK d0e6564. Just dropped LogPrintf reference since last review stickies-v: ACK d0e6564 Empact: ACK bitcoin@d0e6564 Tree-SHA512: 8abe4895951013c2ceca9a57743aacabaf8af831d07eee9ae8372c121c16e88b7226f0e537200c3464792e19ac7e03b57ba0be31f43add8802753972b0aefc48
diff --git a/src/logging.h b/src/logging.h
@@ -257,7 +257,7 @@ std::string SafeStringFormat(const std::string& fmt, const Args&... args)
     }
 }
 
-// Be conservative when using LogPrintf/error or other things which
+// Be conservative when using functions that
 // unconditionally log to debug.log! It should not be the case that an inbound
 // peer can fill up a user's disk with debug.log entries.
 

Original file line number	Diff line number	Diff line change
`@@ -257,7 +257,7 @@ std::string SafeStringFormat(const std::string& fmt, const Args&... args)`
`257`	`257`	`}`
`258`	`258`	`}`
`259`	`259`
`260`		`-// Be conservative when using LogPrintf/error or other things which`
	`260`	`+// Be conservative when using functions that`
`261`	`261`	`// unconditionally log to debug.log! It should not be the case that an inbound`
`262`	`262`	`// peer can fill up a user's disk with debug.log entries.`
`263`	`263`