Re: [new patch] Re: [PATCH] remove redundant NULL checks before kfree() in drivers/video/w100fb.c and add if()+comment back in drivers/video/console/bitblit.c

2005-03-20 Thread Antonino A. Daplas
On Monday 21 March 2005 06:45, Jesper Juhl wrote: > On Sun, 20 Mar 2005, Jesper Juhl wrote: > > On Mon, 21 Mar 2005, Antonino A. Daplas wrote: > > > On Monday 21 March 2005 06:02, Jesper Juhl wrote: > > > > On Mon, 21 Mar 2005, Antonino A. Daplas wrote: > > > > > On Sunday 20 March 2005 06:59,

[new patch] Re: [PATCH] remove redundant NULL checks before kfree() in drivers/video/w100fb.c and add if()+comment back in drivers/video/console/bitblit.c

2005-03-20 Thread Jesper Juhl
On Sun, 20 Mar 2005, Jesper Juhl wrote: > > On Mon, 21 Mar 2005, Antonino A. Daplas wrote: > > On Monday 21 March 2005 06:02, Jesper Juhl wrote: > > > On Mon, 21 Mar 2005, Antonino A. Daplas wrote: > > > > On Sunday 20 March 2005 06:59, Jesper Juhl wrote: > > > > > Checking a pointer for NULL

Re: [PATCH] remove redundant NULL checks before kfree() in drivers/video/

2005-03-20 Thread Jesper Juhl
On Mon, 21 Mar 2005, Antonino A. Daplas wrote: > On Monday 21 March 2005 06:02, Jesper Juhl wrote: > > On Mon, 21 Mar 2005, Antonino A. Daplas wrote: > > > On Sunday 20 March 2005 06:59, Jesper Juhl wrote: > > > > Checking a pointer for NULL before calling kfree() on it is redundant, > > > >

Re: [PATCH] remove redundant NULL checks before kfree() in drivers/video/

2005-03-20 Thread Antonino A. Daplas
On Monday 21 March 2005 05:49, Geert Uytterhoeven wrote: > On Mon, 21 Mar 2005, Antonino A. Daplas wrote: > > On Sunday 20 March 2005 06:59, Jesper Juhl wrote: > > > Checking a pointer for NULL before calling kfree() on it is redundant, > > > kfree() deals with NULL pointers just fine. > > > This

Re: [PATCH] remove redundant NULL checks before kfree() in drivers/video/

2005-03-20 Thread Antonino A. Daplas
On Monday 21 March 2005 06:02, Jesper Juhl wrote: > On Mon, 21 Mar 2005, Antonino A. Daplas wrote: > > On Sunday 20 March 2005 06:59, Jesper Juhl wrote: > > > Checking a pointer for NULL before calling kfree() on it is redundant, > > > kfree() deals with NULL pointers just fine. > > > This patch

Re: [PATCH] remove redundant NULL checks before kfree() in drivers/video/

2005-03-20 Thread Jan Engelhardt
>>... > >This is performance critical, so I would like the check to remain. A comment >may be added in this section. Hm, if we used Yoshifuji's inline kfree(), you'd get both. A check and a clean code. (Though I would not move kfree to __kfree, but make the inline variant explicitly different

Re: [PATCH] remove redundant NULL checks before kfree() in drivers/video/

2005-03-20 Thread Jesper Juhl
On Mon, 21 Mar 2005, Antonino A. Daplas wrote: > On Sunday 20 March 2005 06:59, Jesper Juhl wrote: > > Checking a pointer for NULL before calling kfree() on it is redundant, > > kfree() deals with NULL pointers just fine. > > This patch removes such checks from files in drivers/video/ > > > >

Re: [PATCH] remove redundant NULL checks before kfree() in drivers/video/

2005-03-20 Thread Geert Uytterhoeven
On Mon, 21 Mar 2005, Antonino A. Daplas wrote: > On Sunday 20 March 2005 06:59, Jesper Juhl wrote: > > Checking a pointer for NULL before calling kfree() on it is redundant, > > kfree() deals with NULL pointers just fine. > > This patch removes such checks from files in drivers/video/ > > > >

Re: [PATCH] remove redundant NULL checks before kfree() in drivers/video/

2005-03-20 Thread Antonino A. Daplas
On Sunday 20 March 2005 06:59, Jesper Juhl wrote: > Checking a pointer for NULL before calling kfree() on it is redundant, > kfree() deals with NULL pointers just fine. > This patch removes such checks from files in drivers/video/ > > Since this is a fairly trivial change (and the same change made

Re: [PATCH] remove redundant NULL checks before kfree() in drivers/video/

2005-03-20 Thread Antonino A. Daplas
On Sunday 20 March 2005 06:59, Jesper Juhl wrote: Checking a pointer for NULL before calling kfree() on it is redundant, kfree() deals with NULL pointers just fine. This patch removes such checks from files in drivers/video/ Since this is a fairly trivial change (and the same change made

Re: [PATCH] remove redundant NULL checks before kfree() in drivers/video/

2005-03-20 Thread Geert Uytterhoeven
On Mon, 21 Mar 2005, Antonino A. Daplas wrote: On Sunday 20 March 2005 06:59, Jesper Juhl wrote: Checking a pointer for NULL before calling kfree() on it is redundant, kfree() deals with NULL pointers just fine. This patch removes such checks from files in drivers/video/ Since this is a

Re: [PATCH] remove redundant NULL checks before kfree() in drivers/video/

2005-03-20 Thread Jesper Juhl
On Mon, 21 Mar 2005, Antonino A. Daplas wrote: On Sunday 20 March 2005 06:59, Jesper Juhl wrote: Checking a pointer for NULL before calling kfree() on it is redundant, kfree() deals with NULL pointers just fine. This patch removes such checks from files in drivers/video/ Since this is

Re: [PATCH] remove redundant NULL checks before kfree() in drivers/video/

2005-03-20 Thread Jan Engelhardt
... This is performance critical, so I would like the check to remain. A comment may be added in this section. Hm, if we used Yoshifuji's inline kfree(), you'd get both. A check and a clean code. (Though I would not move kfree to __kfree, but make the inline variant explicitly different

Re: [PATCH] remove redundant NULL checks before kfree() in drivers/video/

2005-03-20 Thread Antonino A. Daplas
On Monday 21 March 2005 06:02, Jesper Juhl wrote: On Mon, 21 Mar 2005, Antonino A. Daplas wrote: On Sunday 20 March 2005 06:59, Jesper Juhl wrote: Checking a pointer for NULL before calling kfree() on it is redundant, kfree() deals with NULL pointers just fine. This patch removes such

Re: [PATCH] remove redundant NULL checks before kfree() in drivers/video/

2005-03-20 Thread Antonino A. Daplas
On Monday 21 March 2005 05:49, Geert Uytterhoeven wrote: On Mon, 21 Mar 2005, Antonino A. Daplas wrote: On Sunday 20 March 2005 06:59, Jesper Juhl wrote: Checking a pointer for NULL before calling kfree() on it is redundant, kfree() deals with NULL pointers just fine. This patch

Re: [PATCH] remove redundant NULL checks before kfree() in drivers/video/

2005-03-20 Thread Jesper Juhl
On Mon, 21 Mar 2005, Antonino A. Daplas wrote: On Monday 21 March 2005 06:02, Jesper Juhl wrote: On Mon, 21 Mar 2005, Antonino A. Daplas wrote: On Sunday 20 March 2005 06:59, Jesper Juhl wrote: Checking a pointer for NULL before calling kfree() on it is redundant, kfree() deals with

[new patch] Re: [PATCH] remove redundant NULL checks before kfree() in drivers/video/w100fb.c and add if()+comment back in drivers/video/console/bitblit.c

2005-03-20 Thread Jesper Juhl
On Sun, 20 Mar 2005, Jesper Juhl wrote: On Mon, 21 Mar 2005, Antonino A. Daplas wrote: On Monday 21 March 2005 06:02, Jesper Juhl wrote: On Mon, 21 Mar 2005, Antonino A. Daplas wrote: On Sunday 20 March 2005 06:59, Jesper Juhl wrote: Checking a pointer for NULL before calling

Re: [new patch] Re: [PATCH] remove redundant NULL checks before kfree() in drivers/video/w100fb.c and add if()+comment back in drivers/video/console/bitblit.c

2005-03-20 Thread Antonino A. Daplas
On Monday 21 March 2005 06:45, Jesper Juhl wrote: On Sun, 20 Mar 2005, Jesper Juhl wrote: On Mon, 21 Mar 2005, Antonino A. Daplas wrote: On Monday 21 March 2005 06:02, Jesper Juhl wrote: On Mon, 21 Mar 2005, Antonino A. Daplas wrote: On Sunday 20 March 2005 06:59, Jesper Juhl wrote:

[PATCH] remove redundant NULL checks before kfree() in drivers/video/

2005-03-19 Thread Jesper Juhl
Checking a pointer for NULL before calling kfree() on it is redundant, kfree() deals with NULL pointers just fine. This patch removes such checks from files in drivers/video/ Since this is a fairly trivial change (and the same change made everywhere) I've just made a single patch for all the

[PATCH] remove redundant NULL checks before kfree() in drivers/video/

2005-03-19 Thread Jesper Juhl
Checking a pointer for NULL before calling kfree() on it is redundant, kfree() deals with NULL pointers just fine. This patch removes such checks from files in drivers/video/ Since this is a fairly trivial change (and the same change made everywhere) I've just made a single patch for all the