linux-kernel - Re: [PATCH RFC 08/15] media: replace **** with a hug

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Sat, 1 Dec 2018 00:01:04 -0800
From:   Tomasz Figa <tfiga@...omium.org>
To:     jarkko.sakkinen@...ux.intel.com
Cc:     Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        Mauro Carvalho Chehab <mchehab@...nel.org>,
        Linux Media Mailing List <linux-media@...r.kernel.org>
Subject: Re: [PATCH RFC 08/15] media: replace **** with a hug

On Fri, Nov 30, 2018 at 11:28 AM Jarkko Sakkinen
<jarkko.sakkinen@...ux.intel.com> wrote:
>
> In order to comply with the CoC, replace **** with a hug. In
> addition, fix a coding style issue (lines with over 80 chars).
>
> Signed-off-by: Jarkko Sakkinen <jarkko.sakkinen@...ux.intel.com>
> ---
>  drivers/media/i2c/bt819.c | 8 +++++---
>  1 file changed, 5 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/media/i2c/bt819.c b/drivers/media/i2c/bt819.c
> index 472e37637c8d..c0f198b764f0 100644
> --- a/drivers/media/i2c/bt819.c
> +++ b/drivers/media/i2c/bt819.c
> @@ -165,9 +165,11 @@ static int bt819_init(struct v4l2_subdev *sd)
>                 0x0f, 0x00,     /* 0x0f Hue control */
>                 0x12, 0x04,     /* 0x12 Output Format */
>                 0x13, 0x20,     /* 0x13 Vertial Scaling msb 0x00
> -                                          chroma comb OFF, line drop scaling, interlace scaling
> -                                          BUG? Why does turning the chroma comb on fuck up color?
> -                                          Bug in the bt819 stepping on my board?
> +                                          chroma comb OFF, line drop scaling,
> +                                          interlace scaling BUG? Why does
> +                                          turning the chroma comb on hug up

Putting the strong language aside, this comment could actually benefit
from rewording to make it convey more information about the problem.
For example, wouldn't

NOTE: The chroma comb causes a random color distortion on some boards
(stepping of the chip?).

be much more meaningful? I had to guess what f**king up was supposed
to mean here, which is obviously a problem with this comment. It could
be extended even more by mentioning what board and chip stepping it
was observed on.

Best regards,
Tomasz