view fix-casts.sh @ 31167:060739bdc84d

Suppress msgfmt errors by removing c-format markers This is unfortunate, but it's the best that I could come up with. The translator, Khaled Hosny, informed me that C does not allow you to skip positional format string arguments, and printf(3) on my system agrees. It does work on glibc, but I'm not sure if gettext guarantees this. It does make other guarantees about the printf implementation above those from C, but this involves replacing the printf function if the native one doesn't provide it. So, even assuming that gettext's printf allows this like glibc's does, gettext may not replace the printf function if the native C printf provides positional arguments but does not allow skipping them. Finally, intltool-update does not seem to put the c-format marker back, which actually surprised me. So we'll see how this goes long-term.

#!/bin/sh

if [ $# -eq 0 ]; then
	echo "Usage: `basename "$0"` PurpleFoo..."
	echo
	echo "This script searches the *current working directory* and replaces casts"
	echo "with GObject-style type checking and casting macros."
	echo 'For example, "(PurpleBuddy *)b" becomes "PURPLE_BUDDY(b)".'
	exit 0
fi

for struct in $* ; do
	cast=`echo $struct | sed "s|[A-Z]|_\0|g" | tr "a-z" "A-Z" | sed "s|^_||"`
	for file in `grep -rl "([[:space:]]*$struct[[:space:]]*\*[[:space:]]*)" . --include=*.c --exclude=purple-client-bindings.c` ; do
		sed -i "s|([[:space:]]*$struct[[:space:]]*\*[[:space:]]*)[[:space:]]*(|$cast(|g" $file
		sed -i "s|([[:space:]]*$struct[[:space:]]*\*[[:space:]]*)[[:space:]]*\([^(][^,);]*\)|$cast(\1)|g" $file
	done
done