Re: Win64 patch 2/2 (winedbg)

[Alexandre Julliard]

Ge van Geldorp  writes:
>  case AddrModeFlat: 
> -
>    dbg_printf("0x%08lx", addr->Offset); 
> -
>    dbg_printf("0x%08llx", addr->Offset); 

%ll formats are not portable, please avoid them.

Win64 patch 2/2 (winedbg)

[Francois Gouget]

On Tue, 25 Jul 2006, Alexandre Julliard wrote:
> Ge van Geldorp  writes:
> >  case AddrModeFlat:
> > -
> >    dbg_printf("0x%08lx", addr->Offset);
> > -
> >    dbg_printf("0x%08llx", addr->Offset);
>
> %ll formats are not portable, please avoid them.

'grep -t "%ll" *' finds 149 instances, mostly traces who there are some 
printfs in tools/winedump.

Should this be turned into a janitorial project?
What is the recommended alternative?

Win64 patch 2/2 (winedbg)

[Alexandre Julliard]

Francois Gouget  writes:
> 'grep -t "%ll" *' finds 149 instances, mostly traces who there are
> some printfs in tools/winedump.
>
> Should this be turned into a janitorial project?
> What is the recommended alternative?

That would be a good idea. In traces we should use
wine_dbgstr_longlong. In tests we don't want to use debug macros
so we should use the %lx%08lx trick.