Ethereal-dev: RE: [Ethereal-dev] [Patch] iostat win32 build

Note: This archive is from the project's previous web site, ethereal.com. This list is no longer active.

From: "Graham Bloice" <graham.bloice@xxxxxxxxxxxxx>
Date: Mon, 2 Dec 2002 16:09:42 -0000
This is a duplicate of my earlier message
http://www.ethereal.com/lists/ethereal-dev/200212/msg00013.html

As the developers guide notes:

Don't use a label without a statement following it.  For example,
something such as

	if (...) {

		...

	done:
	}

will not work with all compilers - you have to do

	if (...) {

		...

	done:
		;
	}

with some statement, even if it's a null statement, after the label.

My patch just added null statements, I was too lazy to go the extra mile for
break as well.

Graham Bloice

> -----Original Message-----
> From: ethereal-dev-admin@xxxxxxxxxxxx
> [mailto:ethereal-dev-admin@xxxxxxxxxxxx]On Behalf Of Jason House
> Sent: 02 December 2002 15:31
> To: Ethereal Development
> Subject: [Ethereal-dev] [Patch] iostat win32 build
>
>
> Well, I just tried to build the latest io_stat and found that it didn't
> compile...  I looked at the code and it was fine... but for whatever
> reason, visual studio doesn't like the "default:" followed by "}" with
> nothing in between.  I simply added "break;" and it compiles fine now.