The attached diff fixes the errors reported by Dana. It appears the N/S
E/W options in the preferences dialog were not saved correctly. Also,
changing the locator field in the same dialog would enter incorrect
values for latitude /longitude because the minus sign needs to be
ignored.

This diff needs a clean xlog-2.0.1 source tree. After recompiling and
installing xlog check your location, especially the N/S and E/W options.

I will release an updated xlog version (2.0.2) for this bug in about 2
weeks time.

Regards,
Joop PG4I
diff -u -r xlog-2.0.1/src/dxcc.c xlog-2.0.1.fixed_qrb/src/dxcc.c
--- xlog-2.0.1/src/dxcc.c	2009-01-28 20:47:15.000000000 +0100
+++ xlog-2.0.1.fixed_qrb/src/dxcc.c	2009-05-06 12:15:22.000000000 +0200
@@ -830,11 +830,11 @@
 	
 	if (lookup.country > 0)
 	{
-		if (preferences.NS == 0)
+		if (preferences.NS == 1)
 			lat = -1.0 * preferences.latitude;
 		else
 			lat = preferences.latitude;
-		if (preferences.EW == 0)
+		if (preferences.EW == 1)
 			lon = -1.0 * preferences.longitude;
 		else
 			lon = preferences.longitude;
Only in xlog-2.0.1.fixed_qrb/src: dxcc.c.orig
diff -u -r xlog-2.0.1/src/gui_preferencesdialog.c xlog-2.0.1.fixed_qrb/src/gui_preferencesdialog.c
--- xlog-2.0.1/src/gui_preferencesdialog.c	2009-01-04 21:32:19.000000000 +0100
+++ xlog-2.0.1.fixed_qrb/src/gui_preferencesdialog.c	2009-05-06 12:15:22.000000000 +0200
@@ -81,12 +81,18 @@
 
 		/* update widgets */
 		if (lat < 0.00)
+		{
+			lat = -1.0 * lat;
 			gtk_combo_box_set_active (GTK_COMBO_BOX (NScombo), 1);
+		}
 		else
 			gtk_combo_box_set_active (GTK_COMBO_BOX (NScombo), 0);
 
 		if (lon < 0.00)
+		{
+			lon = -1.0 * lon;
 			gtk_combo_box_set_active (GTK_COMBO_BOX (EWcombo), 1);
+		}
 		else
 			gtk_combo_box_set_active (GTK_COMBO_BOX (EWcombo), 0);
 
@@ -841,13 +847,13 @@
 	/* location */
 	temp = g_strdup_printf ("%3.2f", preferences.latitude);
 	gtk_entry_set_text (GTK_ENTRY (latentry), temp);
-	if (preferences.NS == 1)
+	if (preferences.NS == 0)
 		gtk_combo_box_set_active (GTK_COMBO_BOX (NScombo), 0);
 	else
 		gtk_combo_box_set_active (GTK_COMBO_BOX (NScombo), 1);
 	temp = g_strdup_printf ("%3.2f", preferences.longitude);
 	gtk_entry_set_text (GTK_ENTRY (longentry), temp);
-	if (preferences.EW == 1)
+	if (preferences.EW == 0)
 		gtk_combo_box_set_active (GTK_COMBO_BOX (EWcombo), 0);
 	else
 		gtk_combo_box_set_active (GTK_COMBO_BOX (EWcombo), 1);
@@ -1049,8 +1055,8 @@
 		else
 			preferences.longitude = atof (temp);
 
-		preferences.NS = gtk_combo_box_get_active (GTK_COMBO_BOX (NScombo)) + 1;
-		preferences.EW = gtk_combo_box_get_active (GTK_COMBO_BOX (EWcombo)) + 1;
+		preferences.NS = gtk_combo_box_get_active (GTK_COMBO_BOX (NScombo));
+		preferences.EW = gtk_combo_box_get_active (GTK_COMBO_BOX (EWcombo));
 		preferences.units = gtk_combo_box_get_active 
 			(GTK_COMBO_BOX (unitscombo))  == 1 ? 0 : 1;
 
Only in xlog-2.0.1.fixed_qrb/src: gui_preferencesdialog.c.orig
_______________________________________________
Xlog-discussion mailing list
Xlog-discussion@nongnu.org
http://lists.nongnu.org/mailman/listinfo/xlog-discussion

Reply via email to