Subject: Re: lib/35489: Build fails with -DNDEBUG
To: None <lib-bug-people@netbsd.org, gnats-admin@netbsd.org,>
From: Christian Biere <christianbiere@gmx.de>
List: netbsd-bugs
Date: 01/26/2007 19:55:02
The following reply was made to PR lib/35489; it has been noted by GNATS.

From: Christian Biere <christianbiere@gmx.de>
To: gnats-bugs@NetBSD.org
Cc: 
Subject: Re: lib/35489: Build fails with -DNDEBUG
Date: Fri, 26 Jan 2007 21:00:53 +0100

 Andreas Wiese wrote:
 > #v+
 >    757          case EVLIST_TIMEOUT: {
 >    758                  struct event *tmp =3D RB_INSERT(event_tree, &base->=
 > timetree, ev);
 >    759                  assert(tmp =3D=3D NULL);
 >    760                  break;
 >    761          }
 > #v-
 
 Thanks, I've committed a similar fix.
 
 > (BTW: what do these braces do there?)
 
 They introduce a new block scope allowing to declare variables. Otherwise,
 you'd have to put it after 'switch (x) {' where it's unnecessarily visible
 beyond the necessary scope.
 
 -- 
 Christian