Adam Nielsen [2009-09-13 16:17]: Hi,
> +static void
> +xmms_modplug_config_changed (xmms_object_t *obj, xmmsv_t *_value, gpointer
> udata)
> +{
> + xmms_modplug_data_t *data = udata;
> + const gchar *name;
> + const gchar *value;
> + gint intvalue;
> +
> + name = xmms_config_property_get_name ((xmms_config_property_t *) obj);
> + value = xmms_config_property_get_string ((xmms_config_property_t *)
> obj);
> +
> + if (!g_ascii_strcasecmp (name, "modplug.freq")) {
> + data->settings.mFrequency = atoi(value);
Please use xmms_config_property_get_int() instead of _get_string
and converting it yourself.
> + } else {
> + intvalue = atoi(value);
> + if (!g_ascii_strcasecmp (name, "modplug.reverb_depth")) {
> + data->settings.mReverbDepth = intvalue;
Same here.
Regards,
Tilman
--
A: Because it messes up the order in which people normally read text.
Q: Why is top-posting such a bad thing?
A: Top-posting.
Q: What is the most annoying thing on usenet and in e-mail?
pgpfSOEj4dw7F.pgp
Description: PGP signature
-- _______________________________________________ Xmms2-devel mailing list Xmms2-devel@lists.xmms.se http://lists.xmms.se/cgi-bin/mailman/listinfo/xmms2-devel
