On 11/28/2011 03:10 PM, Wolfgang Grandegger wrote:

>>>   enum {
>>>     CC770_OBJ_RX0 = 0,      /* for receiving normal messages */
>>>     CC770_OBJ_RX1,          /* for receiving normal messages */
>>>     CC770_OBJ_RX_RTR0,      /* for receiving remote transmission requests */
>>>     CC770_OBJ_RX_RTR1,      /* for receiving remote transmission requests */
>>>     CC770_OBJ_TX,           /* for sending messages */
>>>     CC770_OBJ_MAX  <================
>>
>> ...then add a "," here :P
> 
> Why?

Okay - here the following argument will be not valid :), because OBJ_MAX
will and has always to be the last element in the enum.

But if you don't have a _MAX argument and a new element will be added to
the enum, you'll have to change two lines: 1. add the missing "," 2. add
the new element. Diff will look better if the last element always has a ",".

Marc

-- 
Pengutronix e.K.                  | Marc Kleine-Budde           |
Industrial Linux Solutions        | Phone: +49-231-2826-924     |
Vertretung West/Dortmund          | Fax:   +49-5121-206917-5555 |
Amtsgericht Hildesheim, HRA 2686  | http://www.pengutronix.de   |

Attachment: signature.asc
Description: OpenPGP digital signature

_______________________________________________
Socketcan-users mailing list
[email protected]
https://lists.berlios.de/mailman/listinfo/socketcan-users

Reply via email to