comparison src/prefs.c @ 13710:848c9685ecc7

[gaim-migrate @ 16114] CID 153 Same idea as CID 154: Do the sanity checks directly, in case g_return_if_fail()s are being compiled out. committer: Tailor Script <[e-mail]>

13709:ff25f6d2b484

gaim_prefs_rename(const char *oldname, const char *newname)
{
	struct gaim_pref *oldpref, *newpref;

	oldpref = find_pref(oldname);
	newpref = find_pref(newname);

	/* it's already been renamed, call off the dogs */
	if(!oldpref)
		return;

	gaim_debug_info("prefs", "Renaming %s to %s\n", oldname, newname);

	g_return_if_fail(newpref != NULL); /* the new one needs to be created first */
	g_return_if_fail(oldpref->type == newpref->type);
	g_return_if_fail(oldpref->first_child == NULL); /* can't rename parents */

	switch(oldpref->type) {
		case GAIM_PREF_NONE:
			break;
		case GAIM_PREF_BOOLEAN:

13710:848c9685ecc7

gaim_prefs_rename(const char *oldname, const char *newname)
{
	struct gaim_pref *oldpref, *newpref;

	oldpref = find_pref(oldname);

	/* it's already been renamed, call off the dogs */
	if(!oldpref)
		return;

	if (oldpref->first_child != NULL) /* can't rename parents */
	{
		gaim_debug_error("prefs", "Unable to rename %s to %s: can't rename parents\n", oldname, newname);
		return;
	}


	newpref = find_pref(newname);

	if (newpref == NULL)
	{
		gaim_debug_error("prefs", "Unable to rename %s to %s: new pref not created\n", oldname, newname);
		return;
	}

	if (oldpref->type != newpref->type)
	{
		gaim_debug_error("prefs", "Unable to rename %s to %s: differing types\n", oldname, newname);
		return;
	}

	gaim_debug_info("prefs", "Renaming %s to %s\n", oldname, newname);

	switch(oldpref->type) {
		case GAIM_PREF_NONE:
			break;
		case GAIM_PREF_BOOLEAN: