commit | 8e278616cb3b3d6bfc5a6ca51b480e48e14d3c9c | [log] [tgz] |
---|---|---|
author | Wade Guthrie <wdg@chromium.org> | Tue Feb 26 10:32:34 2013 -0800 |
committer | ChromeBot <chrome-bot@google.com> | Thu Feb 28 13:43:24 2013 -0800 |
tree | 0ec03945673c25acd15ea7683a0b56a9ac321a2c | |
parent | 73fcc3f6560240d0d5dde15927cc0b51dc692dd5 [diff] |
shill: Improves debug output of Config80211 code. This CL does a few things to improve the Config80211 debugging output. It replaces the |GenericToString| method with |Print| for messages and attributes so that a) newlines print correctly and b) nested attributes print better; it removes anachronistic |Nl80211Message::ToString| method; and it standardizes message send and receive output. The |Nl80211Message::ToString| method was originally used to compare config80211 output with that of the 'iw' program. Unit tests and the new |Print| method have been introduced to supplant the need for |ToString|. This CL removes |ToString| and a number of helper methods that support it. BUG=None. TEST=unittest and manual test (invoked shill with a specific log level and verified that the printouts were correct). Change-Id: I7f9133a2bd63864d20ed56b588f2139126360ccd Reviewed-on: https://gerrit.chromium.org/gerrit/44057 Reviewed-by: Christopher Wiley <wiley@chromium.org> Tested-by: Wade Guthrie <wdg@chromium.org> Commit-Queue: Wade Guthrie <wdg@chromium.org>