Getting rid of compiler warnings, especially

- using "abs" function for long or double arguments
 - passing data which is not initialized

dorji/dra818.c

@@ -511,7 +511,6 @@ const struct rig_caps dra818u_caps = {
 	.set_mode =             dra818_set_mode,
 	.get_mode =             dra818_get_mode,
 	.get_dcd = 		dra818_get_dcd,
-	.get_freq =		dra818_get_freq,
 	.set_level =		dra818_set_level,
 	.get_level =		dra818_get_level,
 	.set_dcs_code =         dra818_set_dcs_code,
@@ -612,7 +611,6 @@ const struct rig_caps dra818v_caps = {
 	.set_mode =             dra818_set_mode,
 	.get_mode =             dra818_get_mode,
 	.get_dcd = 		dra818_get_dcd,
-	.get_freq =		dra818_get_freq,
 	.set_level =		dra818_set_level,
 	.get_level =		dra818_get_level,
 	.set_dcs_code =         dra818_set_dcs_code,

icom/ic7300.c

@@ -310,7 +310,7 @@ int ic7300_set_rit(RIG *rig, vfo_t vfo, shortfreq_t ts)
 
 	// set sign bit
 	if (ts < 0) tsbuf[2] = 1;
-	snprintf(tmpbuf,sizeof(tmpbuf),"%04d",abs(ts));
+	snprintf(tmpbuf,sizeof(tmpbuf),"%04d",abs((int)ts));
 	unsigned int b1,b2;
 	sscanf(tmpbuf,"%02x%02x",&b1,&b2);
 	tsbuf[1] = b1;

icom/icom.c

@@ -2732,6 +2732,7 @@ int icom_get_parm(RIG *rig, setting_t parm, value_t *val)
 	case RIG_PARM_BACKLIGHT:
 		prm_cn = C_CTL_MEM;
 		if (priv->civ_version == 1) {
+			prm_cn = 0;  // avoid passing uninitialized data
 			prm_sc = 0x05;
 			prm_len = 2;
 			prmbuf[0] = 0x00;
@@ -2745,6 +2746,7 @@ int icom_get_parm(RIG *rig, setting_t parm, value_t *val)
 		break;
 	case RIG_PARM_BEEP:
 		if (priv->civ_version == 1) {
+			prm_cn = 0;  // avoid passing uninitialized data
 			prm_sc = 0x05;
 			prm_len = 2;
 			prmbuf[0] = 0x00;
@@ -2759,6 +2761,7 @@ int icom_get_parm(RIG *rig, setting_t parm, value_t *val)
 		break;
 	case RIG_PARM_TIME:
 		if (priv->civ_version == 1) {
+			prm_cn = 0;  // avoid passing uninitialized data
 			prm_sc = 0x05;
 			prm_len = 2;
 			prmbuf[0] = 0x00;

kenwood/kenwood.c

@@ -1278,7 +1278,7 @@ int kenwood_set_rit(RIG * rig, vfo_t vfo, shortfreq_t rit)
   if (retval != RIG_OK)
     return retval;
 
-  for (i = 0; i < abs(rint(rit/10)); i++)
+  for (i = 0; i < labs(lrint(rit/10)); i++)
     retval = kenwood_transaction(rig, buf, NULL, 0);
 
   return retval;

kenwood/th.c

@@ -184,7 +184,7 @@ th_set_freq(RIG *rig, vfo_t vfo, freq_t freq)
 
 	freq5=round(freq/5000)*5000;
 	freq625=round(freq/6250)*6250;
-	if (abs(freq5-freq)<abs(freq625-freq)) {
+	if (fabs(freq5-freq)<fabs(freq625-freq)) {
 	  step=0;
 	  freq_sent=freq5;
 	}

kenwood/thg71.c

@@ -446,9 +446,9 @@ int thg71_open(RIG *rig)
 		frng.low_power=-1;
 		rig->state.rx_range_list[i]=frng;
 		if(frng.start> MHz(200))
-			frng.high_power=5.5;
+			frng.high_power=mW(5500);
 		else
-			frng.high_power=6;
+			frng.high_power=mW(6000);
 		frng.low_power=mW(50);
 		rig->state.tx_range_list[i]=frng;
 	}

kenwood/tmd710.c

@@ -483,7 +483,7 @@ tmd710_set_freq(RIG *rig, vfo_t vfo, freq_t freq)
 
   freq5=round(freq/5000)*5000;
   freq625=round(freq/6250)*6250;
-  if (abs(freq5-freq)<abs(freq625-freq)) {
+  if (fabs(freq5-freq)<fabs(freq625-freq)) {
     step=0;
     freq_sent=freq5;
   }
@@ -855,7 +855,7 @@ tmd710_set_rptr_offs(RIG *rig, vfo_t vfo, shortfreq_t freq)
 
   freq5=round(freq/5000)*5000;
   freq625=round(freq/6250)*6250;
-  if (abs(freq5-freq)<abs(freq625-freq)) {
+  if (abs((int)(freq5-freq))<abs((int)(freq625-freq))) {
     freq_sent=freq5;
   }
   else {

kenwood/ts570.c

@@ -614,7 +614,7 @@ int ts570_set_rit(RIG * rig, vfo_t vfo, shortfreq_t rit)
 	if (retval != RIG_OK)
 		return retval;
 
-        for (i = 0; i < abs(rint(rit/10)); i++)
+        for (i = 0; i < labs(lrint(rit/10)); i++)
         {
                 retval = kenwood_transaction(rig, buf, NULL, 0);
 		if (retval != RIG_OK)
@@ -654,7 +654,7 @@ int ts570_set_xit(RIG * rig, vfo_t vfo, shortfreq_t rit)
 	if (retval != RIG_OK)
 		return retval;
 
-        for (i = 0; i < abs(rint(rit/10)); i++)
+        for (i = 0; i < labs(lrint(rit/10)); i++)
         {
                 retval = kenwood_transaction(rig, buf, NULL, 0);
 		if (retval != RIG_OK)

kenwood/ts850.c

@@ -267,7 +267,7 @@ int ts850_set_rit(RIG * rig, vfo_t vfo, shortfreq_t rit)
 	if (retval != RIG_OK)
 		return retval;
 
-	for (i = 0; i < abs(rint(rit/20)); i++)
+	for (i = 0; i < labs(lrint(rit/20)); i++)
 	{
 		retval = kenwood_transaction(rig, buf, NULL, 0);
 		if (retval != RIG_OK)
@@ -303,7 +303,7 @@ int ts850_set_xit(RIG * rig, vfo_t vfo, shortfreq_t xit)
 		c = 'D';
 	sprintf(buf, "R%c", c);
 
-	for (i = 0; i < abs(rint(xit/20)); i++)
+	for (i = 0; i < labs(lrint(xit/20)); i++)
 	{
 		retval = kenwood_transaction(rig, buf, NULL, 0);
 		if (retval != RIG_OK)