On Fri, Aug 14, 2009 at 12:11 AM, Andrew Gaylard ag@computer.org wrote:
On Thu, Aug 13, 2009 at 11:16 PM, Bert Freudenbergbert@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).