widelands-dev team mailing list archive

Thread
Date

Re: [Merge] lp:~widelands-dev/widelands/bug-1422452 into lp:widelands

To: mp+250232@xxxxxxxxxxxxxxxxxx
From: SirVer <SirVer@xxxxxx>
Date: Thu, 19 Feb 2015 06:41:53 -0000
In-reply-to: <20150218224126.8887.56978.launchpad@ackee.canonical.com>
Reply-to: mp+250232@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

The one liner is fine. 

An enum class is internally always represented as an (unsigned) int, but since enum class are guaranteed to be typesafe you must explicitly tell the compiler to interpret the integer as another type, ergo the static_cast is needed.

Good catch! Since when was this broken?
-- 
https://code.launchpad.net/~widelands-dev/widelands/bug-1422452/+merge/250232
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/bug-1422452 into lp:widelands.

References

[Merge] lp:~widelands-dev/widelands/bug-1422452 into lp:widelands
From: TiborB, 2015-02-18