[Gluster-devel] [PATCH v3 UPDATED 1/2] Qemu/Gluster: Add Gluster protocol as supported network disk formats.

Harsh Bora harsh at linux.vnet.ibm.com
Thu Nov 22 14:16:37 UTC 2012


On 11/22/2012 07:05 PM, Eric Blake wrote:
> On 11/22/2012 01:49 AM, Harsh Bora wrote:
>
>>>> +        transp = strstr(uri->scheme, "+");
>
> strstr() for a single-byte needle is overkill.
>
>>>> +        transp++;
>>>
>>> You could even squash the increment into the previous line :-)
>>
>> That would give me:
>>
>> qemu/qemu_command.c: In function 'qemuParseGlusterString':
>> qemu/qemu_command.c:2048:18: error: lvalue required as left operand of
>> assignment.
>
> Not if you do:
>
> transp = strchr(uri->scheme, '+') + 1;
>

Oh yes, thanks.





More information about the Gluster-devel mailing list