Changeset a000878c in mainline for uspace/app/trace


Ignore:
Timestamp:
2010-02-25T19:11:25Z (15 years ago)
Author:
Martin Decky <martin@…>
Branches:
lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export
Children:
958de16
Parents:
a634485
Message:

make sure that all statically allocated strings are declared as "const char *"
and are treated as read-only

Location:
uspace/app/trace
Files:
6 edited

Legend:

Unmodified
Added
Removed

  • uspace/app/trace/errors.h

    ra634485 ra000878c  
    3737
    3838typedef struct {
    39         char *name;     /**< Error value name (Exx) */
    40         char *desc;     /**< Error description */
     39        const char *name;  /**< Error value name (Exx) */
     40        const char *desc;  /**< Error description */
    4141} err_desc_t;
    4242

  • uspace/app/trace/ipc_desc.h

    ra634485 ra000878c  
    3838typedef struct {
    3939        int number;
    40         char *name;
     40        const char *name;
    4141} ipc_m_desc_t;
    4242

  • uspace/app/trace/proto.c

    ra634485 ra000878c  
    157157}
    158158
    159 static void proto_struct_init(proto_t *proto, char *name)
     159static void proto_struct_init(proto_t *proto, const char *name)
    160160{
    161161        proto->name = name;
    … …  
    164164}
    165165
    166 proto_t *proto_new(char *name)
     166proto_t *proto_new(const char *name)
    167167{
    168168        proto_t *p;
    … …  
    206206}
    207207
    208 static void oper_struct_init(oper_t *oper, char *name)
     208static void oper_struct_init(oper_t *oper, const char *name)
    209209{
    210210        oper->name = name;
    211211}
    212212
    213 oper_t *oper_new(char *name, int argc, val_type_t *arg_types,
     213oper_t *oper_new(const char *name, int argc, val_type_t *arg_types,
    214214    val_type_t rv_type, int respc, val_type_t *resp_types)
    215215{

  • uspace/app/trace/proto.h

    ra634485 ra000878c  
    4343
    4444typedef struct {
    45         char *name;
     45        const char *name;
    4646
    4747        int argc;
    … …  
    5656typedef struct {
    5757        /** Protocol name */
    58         char *name;
     58        const char *name;
    5959
    6060        /** Maps method number to operation */
    … …  
    7070void proto_register(int srv, proto_t *proto);
    7171proto_t *proto_get_by_srv(int srv);
    72 proto_t *proto_new(char *name);
     72proto_t *proto_new(const char *name);
    7373void proto_delete(proto_t *proto);
    7474void proto_add_oper(proto_t *proto, int method, oper_t *oper);
    7575oper_t *proto_get_oper(proto_t *proto, int method);
    7676
    77 oper_t *oper_new(char *name, int argc, val_type_t *arg_types,
     77oper_t *oper_new(const char *name, int argc, val_type_t *arg_types,
    7878    val_type_t rv_type, int respc, val_type_t *resp_types);
    7979

  • uspace/app/trace/syscalls.h

    ra634485 ra000878c  
    3939
    4040typedef struct {
    41         char *name;
     41        const char *name;
    4242        int n_args;
    4343        val_type_t rv_type;

  • uspace/app/trace/trace.c

    ra634485 ra000878c  
    585585}
    586586
    587 static loader_t *preload_task(const char *path, char *const argv[],
     587static loader_t *preload_task(const char *path, char **argv,
    588588    task_id_t *task_id)
    589589{
    … …  
    591591        int rc;
    592592
    593         /* Spawn a program loader */   
     593        /* Spawn a program loader */
    594594        ldr = loader_connect();
    595595        if (ldr == NULL)
    … …  
    607607
    608608        /* Send arguments */
    609         rc = loader_set_args(ldr, argv);
     609        rc = loader_set_args(ldr, (const char **) argv);
    610610        if (rc != EOK)
    611611                goto error;
    … …  
    870870}
    871871
    872 static display_mask_t parse_display_mask(char *text)
     872static display_mask_t parse_display_mask(const char *text)
    873873{
    874874        display_mask_t dm;
    875         char *c;
    876 
    877         c = text;
    878 
     875        const char *c = text;
     876       
    879877        while (*c) {
    880878                switch (*c) {
    881                 case 't': dm = dm | DM_THREAD; break;
    882                 case 's': dm = dm | DM_SYSCALL; break;
    883                 case 'i': dm = dm | DM_IPC; break;
    884                 case 'p': dm = dm | DM_SYSTEM | DM_USER; break;
     879                case 't':
     880                        dm = dm | DM_THREAD;
     881                        break;
     882                case 's':
     883                        dm = dm | DM_SYSCALL;
     884                        break;
     885                case 'i':
     886                        dm = dm | DM_IPC;
     887                        break;
     888                case 'p':
     889                        dm = dm | DM_SYSTEM | DM_USER;
     890                        break;
    885891                default:
    886892                        printf("Unexpected event type '%c'.\n", *c);
    887893                        exit(1);
    888894                }
    889 
     895               
    890896                ++c;
    891897        }
    892 
     898       
    893899        return dm;
    894900}
    … …  
    896902static int parse_args(int argc, char *argv[])
    897903{
    898         char *arg;
    899904        char *err_p;
    900905
    901906        task_id = 0;
    902907
    903         --argc; ++argv;
     908        --argc;
     909        ++argv;
    904910
    905911        while (argc > 0) {
    906                 arg = *argv;
     912                char *arg = *argv;
    907913                if (arg[0] == '+') {
    908914                        display_mask = parse_display_mask(&arg[1]);
    … …  
    910916                        if (arg[1] == 't') {
    911917                                /* Trace an already running task */
    912                                 --argc; ++argv;
     918                                --argc;
     919                                ++argv;
    913920                                task_id = strtol(*argv, &err_p, 10);
    914921                                task_ldr = NULL;
    … …  
    927934                        break;
    928935                }
    929 
    930                 --argc; ++argv;
     936               
     937                --argc;
     938                ++argv;
    931939        }
    932940
    933941        if (task_id != 0) {
    934                 if (argc == 0) return 0;
     942                if (argc == 0)
     943                        return 0;
    935944                printf("Extra arguments\n");
    936945                print_syntax();
    … …  
    946955        /* Preload the specified program file. */
    947956        printf("Spawning '%s' with arguments:\n", *argv);
    948         {
    949                 char **cp = argv;
    950                 while (*cp) printf("'%s'\n", *cp++);
    951         }
     957       
     958        char **cp = argv;
     959        while (*cp)
     960                printf("'%s'\n", *cp++);
     961       
    952962        task_ldr = preload_task(*argv, argv, &task_id);
    953963        task_wait_for = true;
Note: See TracChangeset for help on using the changeset viewer.