Misc (re #1559): fixed compile errors/warnings on MSVC

git-svn-id: http://svn.pjsip.org/repos/pjproject/trunk@4412 74dad513-b988-da41-8d7b-12977e46ad98

3 files changed, 19 insertions, 4 deletions

diff --git a/pjnath/src/pjnath-test/concur_test.c b/pjnath/src/pjnath-test/concur_test.c
index 92c2d394..fbb0bc49 100644
--- a/pjnath/src/pjnath-test/concur_test.c
+++ b/pjnath/src/pjnath-test/concur_test.c
@@ -158,6 +158,10 @@ static pj_bool_t stun_sock_on_status(pj_stun_sock *stun_sock,
                                      pj_status_t status)
 {
     struct stun_test_session *test_sess = (struct stun_test_session*)pj_stun_sock_get_user_data(stun_sock);
+
+    PJ_UNUSED_ARG(op);
+    PJ_UNUSED_ARG(status);
+
     test_sess->param.client_got_response++;
     return PJ_TRUE;
 }
@@ -192,7 +196,7 @@ static int stun_destroy_test_session(struct stun_test_session *test_sess)
     for (i=0; i<MAX_SOCK_CLIENTS; ++i) {
 	pj_str_t server_ip = pj_str("127.0.0.1");
 	status = pj_stun_sock_start(stun_sock[i], &server_ip,
-	                            test_sess->server_port, NULL);
+	                            (pj_uint16_t)test_sess->server_port, NULL);
 	if (status != PJ_SUCCESS) {
 	    PJ_PERROR(1,(THIS_FILE, status, "Error starting stun socket"));
 	    return -20;
@@ -323,6 +327,9 @@ static int stun_destroy_test(void)
 	pj_thread_sleep(10);
     }
 
+    /* Avoid compiler warning */
+    goto on_return;
+
 
 on_return:
     test_sess.thread_quit_flag = PJ_TRUE;
diff --git a/pjnath/src/pjnath-test/ice_test.c b/pjnath/src/pjnath-test/ice_test.c
index 0a4c8a3e..66d48314 100644
--- a/pjnath/src/pjnath-test/ice_test.c
+++ b/pjnath/src/pjnath-test/ice_test.c
@@ -467,7 +467,8 @@ static int check_pair(const struct ice_ept *ept1, const struct ice_ept *ept2,
 					break; \
 				    } \
 				    PJ_TIME_VAL_SUB(t, t0); \
-				    if (PJ_TIME_VAL_MSEC(t) >= (timeout)) break; \
+				    if ((unsigned)PJ_TIME_VAL_MSEC(t) >= (timeout)) \
+					break; \
 				} \
 			    }
 
@@ -1015,7 +1016,7 @@ int ice_one_conc_test(pj_stun_config *stun_cfg, int err_quit)
 
 int ice_conc_test(void)
 {
-    const int LOOP = 100;
+    const unsigned LOOP = 100;
     pj_pool_t *pool;
     pj_stun_config stun_cfg;
     unsigned i;
@@ -1034,6 +1035,9 @@ int ice_conc_test(void)
 	if (rc)
 	    break;
     }
+
+    /* Avoid compiler warning */
+    goto on_return;
     
 on_return:
     destroy_stun_config(&stun_cfg);
diff --git a/pjnath/src/pjnath/ice_strans.c b/pjnath/src/pjnath/ice_strans.c
index e9b1dfba..95c6cf78 100644
--- a/pjnath/src/pjnath/ice_strans.c
+++ b/pjnath/src/pjnath/ice_strans.c
@@ -31,8 +31,9 @@
 #include <pj/string.h>
 #include <pj/compat/socket.h>
 
+#define ENABLE_TRACE 0
 
-#if 0
+#if defined(ENABLE_TRACE) && (ENABLE_TRACE != 0)
 #  define TRACE_PKT(expr)	    PJ_LOG(5,expr)
 #else
 #  define TRACE_PKT(expr)
@@ -1320,6 +1321,9 @@ static pj_status_t ice_tx_pkt(pj_ice_sess *ice,
     pj_ice_strans *ice_st = (pj_ice_strans*)ice->user_data;
     pj_ice_strans_comp *comp;
     pj_status_t status;
+#if defined(ENABLE_TRACE) && (ENABLE_TRACE != 0)
+    char daddr[PJ_INET6_ADDRSTRLEN];
+#endif
 
     PJ_ASSERT_RETURN(comp_id && comp_id <= ice_st->comp_cnt, PJ_EINVAL);