Am 29.09.2013 20:08, schrieb André Hentschel:
> Am 28.09.2013 21:51, schrieb Bernard Ladenthin:
>> ---
>>  dlls/kernel32/time.c |  149
>> ++++++++++++++++++++++++++++++++++++++++++++++++++
>>  1 file changed, 149 insertions(+)
> 
> 
> Hi,
> please don't use C++ comments, also the rest of the coding style isn't often 
> seen in Wine...
> (e.g. missing space between if/for/... and the bracket, your way of variable 
> definig/initializing, most likely more)
> 
> maybe you also want to change the FIXME into something like "not implemented 
> for this platform" and add a TRACE for the linux way.
> 
> 
> 

Hi, thanks for the feedback, I will write a new patch. Regards, Bernard


Reply via email to