On Fri, Oct 04, 2013 at 03:18:17PM +0530, Sachin Kamat wrote: > default: > return (selector * 100000) + 950000; > } > - break; > default:
I'm not sure this is actually a clarity improvement. While the break is redundant the return statements are all inside the nested switch so it's not entirely obvious at a glance that all the paths were covered.
signature.asc
Description: Digital signature