Changeset 5d0f1bc in mainline


Ignore:
Timestamp:
2010-11-07T12:41:29Z (14 years ago)
Author:
Jiri Svoboda <jiri@…>
Branches:
lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export
Children:
987930f
Parents:
774e9ecd
Message:

Remove use of ERROR_ macros from netecho.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • uspace/app/netecho/netecho.c

    r774e9ecd r5d0f1bc  
    4141#include <task.h>
    4242#include <arg_parse.h>
    43 #include <err.h>
    4443
    4544#include <net/in.h>
    … …  
    9190int main(int argc, char *argv[])
    9291{
    93         ERROR_DECLARE;
    94 
    9592        size_t size = 1024;
    9693        int verbose = 0;
    … …  
    117114        size_t reply_length;
    118115        int value;
     116        int rc;
    119117
    120118        // parse the command line arguments
    … …  
    123121                        switch (argv[index][1]) {
    124122                        case 'b':
    125                                 ERROR_PROPAGATE(arg_parse_int(argc, argv, &index, &backlog, 0));
     123                                rc = arg_parse_int(argc, argv, &index, &backlog, 0);
     124                                if (rc != EOK)
     125                                        return rc;
    126126                                break;
    127127                        case 'c':
    128                                 ERROR_PROPAGATE(arg_parse_int(argc, argv, &index, &count, 0));
     128                                rc = arg_parse_int(argc, argv, &index, &count, 0);
     129                                if (rc != EOK)
     130                                        return rc;
    129131                                break;
    130132                        case 'f':
    131                                 ERROR_PROPAGATE(arg_parse_name_int(argc, argv, &index, &family, 0, socket_parse_protocol_family));
     133                                rc = arg_parse_name_int(argc, argv, &index, &family, 0, socket_parse_protocol_family);
     134                                if (rc != EOK)
     135                                        return rc;
    132136                                break;
    133137                        case 'h':
    … …  
    136140                                break;
    137141                        case 'p':
    138                                 ERROR_PROPAGATE(arg_parse_int(argc, argv, &index, &value, 0));
     142                                rc = arg_parse_int(argc, argv, &index, &value, 0);
     143                                if (rc != EOK)
     144                                        return rc;
    139145                                port = (uint16_t) value;
    140146                                break;
    141147                        case 'r':
    142                                 ERROR_PROPAGATE(arg_parse_string(argc, argv, &index, &reply, 0));
     148                                rc = arg_parse_string(argc, argv, &index, &reply, 0);
     149                                if (rc != EOK)
     150                                        return rc;
    143151                                break;
    144152                        case 's':
    145                                 ERROR_PROPAGATE(arg_parse_int(argc, argv, &index, &value, 0));
     153                                rc = arg_parse_int(argc, argv, &index, &value, 0);
     154                                if (rc != EOK)
     155                                        return rc;
    146156                                size = (value >= 0) ? (size_t) value : 0;
    147157                                break;
    148158                        case 't':
    149                                 ERROR_PROPAGATE(arg_parse_name_int(argc, argv, &index, &value, 0, socket_parse_socket_type));
     159                                rc = arg_parse_name_int(argc, argv, &index, &value, 0, socket_parse_socket_type);
     160                                if (rc != EOK)
     161                                        return rc;
    150162                                type = (sock_type_t) value;
    151163                                break;
    … …  
    156168                        case '-':
    157169                                if (str_lcmp(argv[index] + 2, "backlog=", 6) == 0) {
    158                                         ERROR_PROPAGATE(arg_parse_int(argc, argv, &index, &backlog, 8));
     170                                        rc = arg_parse_int(argc, argv, &index, &backlog, 8);
     171                                        if (rc != EOK)
     172                                                return rc;
    159173                                } else if (str_lcmp(argv[index] + 2, "count=", 6) == 0) {
    160                                         ERROR_PROPAGATE(arg_parse_int(argc, argv, &index, &count, 8));
     174                                        rc = arg_parse_int(argc, argv, &index, &count, 8);
    161175                                } else if (str_lcmp(argv[index] + 2, "family=", 7) == 0) {
    162                                                 ERROR_PROPAGATE(arg_parse_name_int(argc, argv, &index, &family, 9, socket_parse_protocol_family));
     176                                        rc = arg_parse_name_int(argc, argv, &index, &family, 9, socket_parse_protocol_family);
     177                                        if (rc != EOK)
     178                                                return rc;
    163179                                } else if (str_lcmp(argv[index] + 2, "help", 5) == 0) {
    164180                                        echo_print_help();
    165181                                        return EOK;
    166182                                } else if (str_lcmp(argv[index] + 2, "port=", 5) == 0) {
    167                                         ERROR_PROPAGATE(arg_parse_int(argc, argv, &index, &value, 7));
     183                                        rc = arg_parse_int(argc, argv, &index, &value, 7);
     184                                        if (rc != EOK)
     185                                                return rc;
    168186                                        port = (uint16_t) value;
    169187                                } else if (str_lcmp(argv[index] + 2, "reply=", 6) == 0) {
    170                                         ERROR_PROPAGATE(arg_parse_string(argc, argv, &index, &reply, 8));
     188                                        rc = arg_parse_string(argc, argv, &index, &reply, 8);
     189                                        if (rc != EOK)
     190                                                return rc;
    171191                                } else if (str_lcmp(argv[index] + 2, "size=", 5) == 0) {
    172                                         ERROR_PROPAGATE(arg_parse_int(argc, argv, &index, &value, 7));
     192                                        rc = arg_parse_int(argc, argv, &index, &value, 7);
     193                                        if (rc != EOK)
     194                                                return rc;
    173195                                        size = (value >= 0) ? (size_t) value : 0;
    174196                                } else if (str_lcmp(argv[index] + 2, "type=", 5) == 0) {
    175                                         ERROR_PROPAGATE(arg_parse_name_int(argc, argv, &index, &value, 7, socket_parse_socket_type));
     197                                        rc = arg_parse_name_int(argc, argv, &index, &value, 7, socket_parse_socket_type);
     198                                        if (rc != EOK)
     199                                                return rc;
    176200                                        type = (sock_type_t) value;
    177201                                } else if (str_lcmp(argv[index] + 2, "verbose", 8) == 0) {
    … …  
    240264                }
    241265                // set the backlog
    242                 if (ERROR_OCCURRED(listen(listening_id, backlog))) {
    243                         socket_print_error(stderr, ERROR_CODE, "Socket listen: ", "\n");
    244                         return ERROR_CODE;
     266                rc = listen(listening_id, backlog);
     267                if (rc != EOK) {
     268                        socket_print_error(stderr, rc, "Socket listen: ", "\n");
     269                        return rc;
    245270                }
    246271        }
    247272
    248273        // bind the listenning socket
    249         if (ERROR_OCCURRED(bind(listening_id, address, addrlen))) {
    250                 socket_print_error(stderr, ERROR_CODE, "Socket bind: ", "\n");
    251                 return ERROR_CODE;
     274        rc = bind(listening_id, address, addrlen);
     275        if (rc != EOK) {
     276                socket_print_error(stderr, rc, "Socket bind: ", "\n");
     277                return rc;
    252278        }
    253279
    … …  
    301327                                        // parse the source address
    302328                                        if (address_start) {
    303                                                 if (ERROR_OCCURRED(inet_ntop(address->sa_family, address_start, address_string, sizeof(address_string)))) {
    304                                                         fprintf(stderr, "Received address error %d\n", ERROR_CODE);
     329                                                rc = inet_ntop(address->sa_family, address_start, address_string, sizeof(address_string));
     330                                                if (rc != EOK) {
     331                                                        fprintf(stderr, "Received address error %d\n", rc);
    305332                                                } else {
    306333                                                        data[length] = '\0';
    … …  
    311338
    312339                                // answer the request either with the static reply or the original data
    313                                 if (ERROR_OCCURRED(sendto(socket_id, reply ? reply : data, reply ? reply_length : length, 0, address, addrlen)))
    314                                         socket_print_error(stderr, ERROR_CODE, "Socket send: ", "\n");
     340                                rc = sendto(socket_id, reply ? reply : data, reply ? reply_length : length, 0, address, addrlen);
     341                                if (rc != EOK)
     342                                        socket_print_error(stderr, rc, "Socket send: ", "\n");
    315343                        }
    316344
    317345                        // close the accepted stream socket
    318346                        if (type == SOCK_STREAM) {
    319                                 if (ERROR_OCCURRED(closesocket(socket_id)))
    320                                         socket_print_error(stderr, ERROR_CODE, "Close socket: ", "\n");
     347                                rc = closesocket(socket_id);
     348                                if (rc != EOK)
     349                                        socket_print_error(stderr, rc, "Close socket: ", "\n");
    321350                        }
    322351
    … …  
    335364
    336365        // close the listenning socket
    337         if (ERROR_OCCURRED(closesocket(listening_id))) {
    338                 socket_print_error(stderr, ERROR_CODE, "Close socket: ", "\n");
    339                 return ERROR_CODE;
     366        rc = closesocket(listening_id);
     367        if (rc != EOK) {
     368                socket_print_error(stderr, rc, "Close socket: ", "\n");
     369                return rc;
    340370        }
    341371
Note: See TracChangeset for help on using the changeset viewer.