[PATCH 2/2] Add some more casts (1/2)

Yao Qi qiyaoltc@gmail.com
Fri Sep 25 13:17:00 GMT 2015


Simon Marchi <simon.marchi@ericsson.com> writes:

> -  const struct signal_catchpoint *c = (void *) bl->owner;
> +  const struct signal_catchpoint *c =
> +    (const struct signal_catchpoint *) bl->owner;

Nit here: "=" should be put the beginning of next line, like

    const struct signal_catchpoint *c
      = (const struct signal_catchpoint *) bl->owner;

GNU Coding Standards says "When you split an expression into multiple
lines, split it before an operator, not after one."  There are multiple
instances of this problem.

OK with them fixed.

-- 
Yao (齐尧)



More information about the Gdb-patches mailing list