| Yes you are correct. I misread the code. | | However - I'd prefer another constant to match last in enum at the | same point as the declaration of the enum. This way if values are | added other code doesn't break. I've seen this done before quite a few | times. It is dangerous on relying on the last value when it's not | explicitly marked as the last value. Sorry I was replying to emails in-order and this one was later in the pile, so please disregard most of the previous email. Your suggestion is a useful improvement, which means that I will rework and the resubmit. - To unsubscribe from this list: send the line "unsubscribe dccp" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html