[prev in list] [next in list] [prev in thread] [next in thread] 

List:       wine-devel
Subject:    Re: [PATCH] ucrtbase: Implement vsscanf
From:       Martin_Storsjö <martin () martin ! st>
Date:       2015-10-29 10:38:41
Message-ID: alpine.DEB.2.02.1510291238010.6942 () cone ! martin ! st
[Download RAW message or body]

On Thu, 29 Oct 2015, Piotr Caban wrote:

> Hi,
>
> On 10/28/15 22:59, Martin Storsjö wrote:
>>>> +int CDECL MSVCRT__stdio_common_vsscanf(unsigned __int64 options,
>>>> const char *input, MSVCRT_size_t length, const char *format,
>>>> MSVCRT__locale_t locale, __ms_va_list valist)
> Sorry for not asking about it earlier but could you please avoid adding long 
> lines. It's not a hard limit but in general we're trying to use up to 
> 80-characters in line. It's OK to have longer lines if it makes code more 
> readable but I think you should try to not exceed e.g. 120 characters in line 
> (if there's a good reason I'm OK with longer lines).

Ok, will repost with that fixed. Apparently I did at least some 
rudimentary wrapping of the lines for the printf functions, but not 
here...

// Martin



[prev in list] [next in list] [prev in thread] [next in thread]