Hi Ivana,
<snip>
>> + if ((name[strlen(name)-1] == '/') ||
>> + ((strlen(name) == 0) &&
>> + (tmp[strlen(tmp)-1] == '/')))
>> + snprintf(path, FILENAME_MAX,
>> + "%s%s", tmp,
>> + name+((name[0] == '/') ?
>> + 1 : 0));
>> + else
>> + snprintf(path, FILENAME_MAX,
>> + "%s%s/", tmp,
>> + name+((name[0] == '/') ?
>> + 1 : 0));
>
> while not needed, can we please have {} surrounding the if conditions.
> It makes the code readable. Also, maybe we could just move this if
> condition to another static function. And then see if it can be
> simplified.
Could you please incorporate this as well? moving it to a different function.
Thanks,
Dhaval
------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and
threat landscape has changed and how IT managers can respond. Discussions
will include endpoint security, mobile security and the latest in malware
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
Libcg-devel mailing list
Libcg-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/libcg-devel
