]> rtime.felk.cvut.cz Git - eurobot/public.git/commitdiff
cand: Fix warnings
authorMichal Sojka <sojkam1@fel.cvut.cz>
Fri, 23 Apr 2010 06:15:43 +0000 (08:15 +0200)
committerMichal Sojka <sojkam1@fel.cvut.cz>
Fri, 23 Apr 2010 06:15:43 +0000 (08:15 +0200)
- comparsion of signed and unsigned
- return something or nothing

src/cand/cand.cc

index 7449b57105ed31154f77ea1e142aef92718e59df..238776cd102bf6d4357b1832c676f591bdd90744 100644 (file)
@@ -97,7 +97,7 @@ int set_pwr_alert(struct robottype_orte_data *orte_data)
 
 int send_can_msg(struct robottype_orte_data *orte_data)
 {
-       can_send(orte_data->can_msg.id, orte_data->can_msg.len, orte_data->can_msg.data);
+       return can_send(orte_data->can_msg.id, orte_data->can_msg.len, orte_data->can_msg.data);
 }
 
 int set_motor_speed(struct robottype_orte_data *orte_data)
@@ -130,7 +130,7 @@ int set_hokuyo_pitch(struct robottype_orte_data *orte_data)
 int can_send(canid_t id, unsigned char length, unsigned char *data)
 {
        struct can_frame frame;
-       int size;
+       size_t size;
 
        frame.can_id = id;
        frame.can_dlc = length;
@@ -152,7 +152,7 @@ int can_send(canid_t id, unsigned char length, unsigned char *data)
 /**
  * Parse frame ID and react as required
  */
-int cand_parse_frame(struct robottype_orte_data *orte, struct can_frame frame)
+void cand_parse_frame(struct robottype_orte_data *orte, struct can_frame frame)
 {
        int status_cnt = 0;
 
@@ -402,10 +402,9 @@ int main(int argc, char *argv[])
                /* parse data */
                cand_parse_frame(&orte, frame);
        }
+       return EXIT_SUCCESS;
 
 err:
        close(sock);
        return 1;
-out:
-       return EXIT_SUCCESS;
 }