mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "J.A. Magallon" <jamagallon@able.es>
To: Roman Zippel <zippel@linux-m68k.org>
Cc: Pierre Ossman <drzeus-list@drzeus.cx>,
	kbuild-devel@lists.sourceforge.net,
	LKML <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] Pointer cast warnings in scripts/
Date: Tue, 21 Jun 2005 14:19:26 +0000	[thread overview]
Message-ID: <1119363566l.11384l.0l@werewolf.able.es> (raw)
In-Reply-To: <Pine.LNX.4.61.0506211515210.3728@scrub.home> (from zippel@linux-m68k.org on Tue Jun 21 15:20:01 2005)


On 06.21, Roman Zippel wrote:
> Hi,
> 
> On Tue, 21 Jun 2005, Pierre Ossman wrote:
> 
> > Should I extract the changes for bkbits and make a reversed patch?
> 
> No, go through the warnings, analyze each one and choose an appropriate 
> solution. You might want to keep notes, which you can post with the 
> changelogs, so one can reproduce, why a certain change was done.
> 

I started doing that, and the more warnings are related to strxxx functions.
They are defined as receiving 'char *' pointers, I do not know why someone
started to fill the sources with 'signed char *' for text strings...

I did not finish the cleaup due to lack of time, but I was planning to
continue...

--
J.A. Magallon <jamagallon()able!es>     \               Software is like sex:
werewolf!able!es                         \         It's better when it's free
Mandriva Linux release 2006.0 (Cooker) for i586
Linux 2.6.12-jam1 (gcc 4.0.1 (4.0.1-0.2mdk for Mandriva Linux release 2006.0))



  parent reply	other threads:[~2005-06-21 14:21 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-06-21 11:17 Pierre Ossman
2005-06-21 12:06 ` Mikael Pettersson
2005-06-21 12:18 ` Roman Zippel
2005-06-21 12:41   ` Pierre Ossman
2005-06-21 12:51     ` Roman Zippel
2005-06-21 12:59       ` Pierre Ossman
2005-06-21 13:14         ` Bernd Petrovitsch
2005-06-22  9:21           ` Pierre Ossman
2005-06-22 10:04             ` Bernd Petrovitsch
2005-06-22 11:12               ` Pierre Ossman
2005-06-22 11:25                 ` Russell King
2005-06-21 13:20         ` Roman Zippel
2005-06-21 14:06           ` Pierre Ossman
2005-06-21 14:14             ` Roman Zippel
2005-06-21 21:31               ` J.A. Magallon
2005-06-21 14:19           ` J.A. Magallon [this message]
2005-06-21 13:25     ` Andreas Schwab

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1119363566l.11384l.0l@werewolf.able.es \
    --to=jamagallon@able.es \
    --cc=drzeus-list@drzeus.cx \
    --cc=kbuild-devel@lists.sourceforge.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=zippel@linux-m68k.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®