On 7 Aug 2009 at 12:27, Hannes Reinecke wrote:

> +static int iscsi_check_tmf_restrictions(struct iscsi_task *task, int opcode)
> 

A line of comment for each routine, explaining what it is expected to do would 
not 
harm IMHO ;-)

e.g. "This function returns true IFF ..."

Regards,
Ulrich


--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"open-iscsi" group.
To post to this group, send email to open-iscsi@googlegroups.com
To unsubscribe from this group, send email to 
open-iscsi+unsubscr...@googlegroups.com
For more options, visit this group at http://groups.google.com/group/open-iscsi
-~----------~----~----~----~------~----~------~--~---

Reply via email to