Don't split up error messages in your source code
Every so often, developers come up with really clever ways to frustrate system administrators and other people who want to go look at their code to diagnose problems. The one that I ran into today looks like this:
if (rval != IDM_STATUS_SUCCESS) { cmn_err(CE_NOTE, "iscsi connection(%u) unable to " "connect to target %s", icp->conn_oid, icp->conn_sess->sess_name); idm_conn_rele(icp->conn_ic); }
In the name of keeping the source lines under 80 characters wide, the developer here has split the error message into two parts, using modern C's constant string concatenation to have the compiler put them back together.
Perhaps it is not obvious why this is at least really annoying. Suppose that you start with the following error message in your logs:
iscsi connection(60) unable to connect to target <tgtname>
You (the bystander, who is not a developer) would like find the
code that produces this error message, so that you can understand
the surrounding context. If this error message was on one line in
the code, it would be very easy to search for; even if you need to
wild-card some stuff with grep, the core string 'unable to connect
to target
' ought to be both relatively unique and easy to find.
But because the message has been split onto multiple source lines,
it's not; your initial search will fail. In fact a lot of substrings
will fail to find the correct source of this message (eg 'unable
to connect
'). You're left to search for various substrings of the
message, hoping both that they are unique enough that you are not
going to be drowned in hits and that you have correctly guessed how
the developer decided to split things up or parameterize their
message.
(I don't blame developers for parameterizing their messages, but it does make searching for them in the code much harder. Clearly some parts of this message are generated on the fly, but are 'connect' or 'target' among them instead of being constant part of the message? You don't know and have to guess. 'Unable to <X> to <Y> <Z>' is not necessarily an irrational message format string, or you equally might guess 'unable to <X> to target <Z>'.)
The developers doing this are not making life impossible for people, of course. But they are making it harder and I wish they wouldn't. It is worth long lines to be able to find things in source code with common tools.
(Messages aren't the only example of this, of course, just the one that got to me today.)
Comments on this page:
|
|