[Vm-dev] renaming dprintf

[Eliot Miranda]

On Fri, Aug 14, 2009 at 12:11 AM, Andrew Gaylard <ag at computer.org> wrote:

>
> On Thu, Aug 13, 2009 at 11:16 PM, Bert Freudenberg<bert at freudenbergs.de>
> wrote:
> >
> > We could rename the Balloon3D DPRINTF to DPRINTF3D and the regular
> dprintf
> > to DPRINTF. This seems to get the job done:
> >
> > svn update ...
> > cd platforms
> > grep -rl DPRINTF .  \
> > | egrep '(B3D|X11)' \
> > | grep -v '/\.svn/' \
> > | tr '\n' '\000'    \
> > | xargs -0 sed -i '' s/DPRINTF/DPRINTF3D/g
> > grep -rl dprintf .  \
> > | grep -v '/\.svn/' \
> > | tr '\n' '\000'    \
> > | xargs -0 sed -i '' s/dprintf/DPRINTF/g
> > svn diff | less
> >
> > This changes all platforms, but should not be a problem right? Anyone
> > opposed?
>
> Looks good to me.  It's a good idea to have macros
> in upper-case anyway.  Thanks for sorting this out.
>
> - Andrew
>
> PS: is there any particular reason why we have filenames with
> spaces in them?  It makes find/xargs painful to use (as in the
> 'tr' above).
>

I think it is because on Mac OS
Smalltalk getSystemAttribute: 1001
answers
'Mac OS'
and fixing this would cause too much pain (I think).
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.squeakfoundation.org/pipermail/vm-dev/attachments/20090814/757f14ac/attachment.htm