Bug 3921 - smbd/trans2.c: "lock_blocking" is set but never used
Summary: smbd/trans2.c: "lock_blocking" is set but never used
Status: RESOLVED FIXED
Alias: None
Product: Samba 3.0
Classification: Unclassified
Component: Build environment (show other bugs)
Version: 3.0.23
Hardware: SGI IRIX
: P3 minor
Target Milestone: none
Assignee: Tim Potter
QA Contact: Samba QA Contact
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-07-12 09:48 UTC by Jason Mader (mail bounces back)
Modified: 2006-07-12 14:13 UTC (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jason Mader (mail bounces back) 2006-07-12 09:48:01 UTC
( revision 16982 )

--- smbd/trans2.o ---
cc-1552 c99: WARNING File = smbd/trans2.c, Line = 4506
  The variable "lock_blocking" is set but never used.

                        BOOL lock_blocking;
                             ^
Comment 1 Volker Lendecke 2006-07-12 09:52:47 UTC
Hi, Jason!

Before you start a new stream of these: Do your messages somehow match what we get from the build farm? We recently have fixed quite a few IRIX warnings that came from http://build.samba.org/?function=View+Build;host=us4;tree=samba_3_0;compiler=cc

Does this somehow match your findings? It might be less effort to just put Jeremy's nose there :-)

Volker
Comment 2 Jason Mader (mail bounces back) 2006-07-12 10:42:16 UTC
Rest easy, I'm done adding bugs for 3.0.23 and svn; there aren't really any good compiler warnings left in 3.0.23 that haven't been fixed in svn (just a couple Won't Fixes).  Today I added three that looked interesting from latest svn revision.

The Irix build farm was at revision 16973, and so I have compiled that.  The build farm doesn't show any compiler warnings, but I see Bug 2995, Bug 3921, Bug 3923, and Bug 3922.  So it might not yet be a good place for Jeremy to look for the warnings.

Two differences that I see between what I do and the build farm are, --with-quotas, and libiconv support.
Comment 3 Jeremy Allison 2006-07-12 14:13:18 UTC
Fixed thanks ! We should be looking at this...
Jeremy.