strange compilation warning

[Andy Fish]

The following code:

enum Foo { one, two };
int Bar(Foo foo)
{
switch (foo)
{
case Foo.one:
return 7;
case Foo.two:
return 4;
}
}

gives a compilation warning "not all paths return a value"

surely this is spurious though? - there is no path I can see that does not
return a value

Andy

[Marc Gravell]

there is no path I can see that does not return a value
How about:
Foo foo = (Foo) 17; // this works just fine
Bar(foo); //boom

Best option: add a
default: throw new ArgumentExcepti on();
line...

Marc

[Andy Fish]

"Marc Gravell" <ma**********@g mail.comwrote in message
news:u3******** ******@TK2MSFTN GP04.phx.gbl...

>there is no path I can see that does not return a value

How about:
Foo foo = (Foo) 17; // this works just fine
Bar(foo); //boom

Best option: add a
default: throw new ArgumentExcepti on();
line...

Marc

Ooh - I didn't know you could cast an int to an enum even if it was out of
range

Thanks Marc

[=?ISO-8859-1?Q?Arne_Vajh=F8j?=]

Andy Fish wrote:

"Marc Gravell" <ma**********@g mail.comwrote in message
news:u3******** ******@TK2MSFTN GP04.phx.gbl...

>>there is no path I can see that does not return a value

How about:
Foo foo = (Foo) 17; // this works just fine
Bar(foo); //boom

Best option: add a
default: throw new ArgumentExcepti on();
line...

Ooh - I didn't know you could cast an int to an enum even if it was out of
range

C# enum is almost just an int.

If I were to guess at why, then I would say that it is necesarry
to support using multiple values in one variable (flags).

Arne