view fix-casts.sh @ 25977:0e93bbb7f5ca

Woo boy this function is a little crazy. I shuffled some things around: 1. We now call aim_srv_setextrainfo() after aim_locate_setprofile() This fixes the bug where, if you set an available message, then set an away message, then change your state to available but leave the same away message up, the AIM servers would use your old available message. I'm not really sure why it happens... maybe some sort of race condition in how the server parses our requests. In any case, this fixes it, and it's what the official clients seem to do (or AIM Lite at least) 2. Combine the code for stripping HTML and truncating the available message. It was happening in two places before (one for available messages and another for ICQ away messages) 3. Put our away message in the field we've been calling an "available" message. This seems to be what the official clients do. Or AIM Lite at least. I'm not sure if there will be other side effects.

#!/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