linux-kernel - Re: [PATCH] trivial: use (C) instead of \251

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 for Android: free password hash cracker in your pocket

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

Message-ID: <alpine.LNX.2.00.1004300957130.9849@pobox.suse.cz>
Date:	Fri, 30 Apr 2010 10:01:42 +0200 (CEST)
From:	Jiri Kosina <jkosina@...e.cz>
To:	Ingo Molnar <mingo@...e.hu>
Cc:	Tejun Heo <tj@...nel.org>, lkml <linux-kernel@...r.kernel.org>,
	Paul Mackerras <paulus@....ibm.com>
Subject: Re: [PATCH] trivial: use (C) instead of \251

On Fri, 30 Apr 2010, Ingo Molnar wrote:

> > > The latin-1 copyright character \251 is invalid in ASCII or UTF-8 and
> > > causes annoying problems with some editors and tools.  Use (C)
> > > instead.
> > > 
> > > Signed-off-by: Tejun Heo <tj@...nel.org>
> > > Cc: Paul Mackerras <paulus@....ibm.com>
> > > ---
> > > I'm not really sure this would survive transport over mail in
> > > appliable form, attaching the patch too just in case.
> > 
> > It didn't survive :) But it was easy enough to fix by hand. Applied, 
> > thanks Tejun.
> 
> NAK.
> 
> A similar patch was sent a few weeks ago, it was objected to and rejected - 
> the copyright character was used for legal reasons.

Well, Paul himself stated that according to their lawyer department, 
"Copyright" (which is there) is enough. So it made sense to remove invalid 
(at least in some sense) character.

Also zillion of other files copyrighted by Paul contain "Copyright (C)".

If we really want this character there then, why not rather in UTF-8, so 
that it works flawlessly?

> Jiri, in the future please dont apply 'trivial' patches that involve 
> subsystems that i co-maintain and which is not acked by any of the subsystem 
> maintainers.

OK, as you wish.

-- 
Jiri Kosina
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/