Hi Masahiro,
> This commit splits out the special E_OR handling ('-' instead of '||')
> into a dedicated helper expr_print_revdev().
> Restore the original expr_print() prior to commit 1ccb27143360
> ("kconfig: make "Selected by:" and "Implied by:" readable").
> This makes sense because:
> - We need to chop those expressions only when printing the reverse
> dependency, and only when E_OR is encountered
> - Otherwise, it should be printed as before, so fall back to
> expr_print()
> This also improves the behavior; for a single line, it was previously
> displayed in the same line as "Selected by", like this:
> Selected by: A [=n] && B [=n]
> This will be displayed in a new line, consistently:
> Selected by:
> - A [=n] && B [=n]
> Signed-off-by: Masahiro Yamada <[email protected]>
Reviewed-by: Petr Vorel <[email protected]>
> ---
<snip>
> get_symbol_props_str(r, sym, P_SELECT, _(" Selects: "));
> if (sym->rev_dep.expr) {
> - str_append(r, _(" Selected by: "));
> + str_append(r, _(" Selected by: \n"));
^
I'd remove unnecessary whitespace here ^.
str_append(r, _(" Selected by:\n"));
> expr_gstr_print_revdep(sym->rev_dep.expr, r);
> - str_append(r, "\n");
> }
> get_symbol_props_str(r, sym, P_IMPLY, _(" Implies: "));
> if (sym->implied.expr) {
> - str_append(r, _(" Implied by: "));
> + str_append(r, _(" Implied by: \n"));
The same here.
> expr_gstr_print_revdep(sym->implied.expr, r);
> - str_append(r, "\n");
> }
> str_append(r, "\n\n");
Kind regards,
Petr
