Re: [PATCH 1/8]reiserfs:stree.c Fix variable set but not used.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 06/14/2010 01:48 PM, Nick Bowler wrote:
On 13:26 Mon 14 Jun     , Justin P. Mattock wrote:
@@ -617,8 +616,7 @@ int search_by_key(struct super_block *sb, const struct cpu_key *key,	/* Key to s

  	pathrelse(search_path);

-	right_neighbor_of_leaf_node = 0;
-
+	

This hunk introduces whitespace on the empty line, which is not cool.

I can resend!!(biggest problem is working
through these warnings)


  	/* With each iteration of this loop we search through the items in the
  	   current node, and calculate the next current node(next path element)
  	   for the next iteration of this loop.. */
@@ -695,8 +693,7 @@ int search_by_key(struct super_block *sb, const struct cpu_key *key,	/* Key to s
  			   starting from the root. */
  			block_number = SB_ROOT_BLOCK(sb);
  			expected_level = -1;
-			right_neighbor_of_leaf_node = 0;
-
+			

Here, too.

Most of the patches in this series have similar issues.


main thing now(for me atleast)is,
is this actual dead code or what?
if not then something else needs to
be done, if yes then I guess I can
resend this, with out the whitespace
issue.

Justin P. Mattock
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux