On Thursday, 1 August 2013 11:55 PM, Nikolay Sivov wrote:
- p = strchrW(key_name,'\');
- if (!p)
- {
p = 0;- }
- else p++;
I'm not sure what this is supposed to do.
It is equivalent to the following code;
p = strchrW(key_name, '\'); if (p != NULL) p++;
I've now modified the patch to represent this.
case REG_BINARY:case REG_NONE:pValue = value;for (i=0; i<valueSize; i++, pValue++)reg_printfW(formatXW, *pValue);break;There's no need for separate variable here nor for incrementing pointer.
The incrementing pointer is needed because the 'value' is a Byte array. But the separate variable is not needed.