libosmocore[master]: vty: fix "everything" logging option
gerrit-no-reply at lists.osmocom.org
Sat Feb 11 14:56:05 UTC 2017
Patch Set 4: Code-Review-1
I think given we have configs in the wild that include everything.. where it might or might not be intended.. we should have a new name.. "all" maybe?
git grep everything -- openbsc/doc/ | wc -l
Line 7: void logging_vty_add_cmds(const struct log_info *unused_info);
then remove it... as long as we don't write (void) the compiler should not complain..
Line 142: "Log everything (use with extreme care)",
What about existing configs that write "everything"? Maybe introduce "all" and have everything as an alias for "XYZ???"
Line 356: static inline bool log_to_target(struct log_target *tar, int subsys, int level)
But it doesn't look. it is a rather long predicate function.. should_log_to_target?
To view, visit https://gerrit.osmocom.org/1582
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-Owner: Max <msuraev at sysmocom.de>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Holger Freyther <holger at freyther.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Max <msuraev at sysmocom.de>
More information about the gerrit-log