Changeset 138941 for trunk/base


Ignore:
Timestamp:
Jul 24, 2015, 3:24:40 PM (5 years ago)
Author:
raimue@…
Message:

pextlib: Use new ui_* functions with printf style

Location:
trunk/base/src/pextlib1.0
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/base/src/pextlib1.0/system.c

    r125617 r138941  
    289289                }
    290290
    291                 if (ui_info(interp, sbuf) != TCL_OK) {
    292                     read_failed = 1;
    293                     break;
    294                 }
     291                ui_info(interp, "%s", sbuf);
    295292            }
    296293            fclose(pdes);
     
    308305            status = TCL_OK;
    309306        } else {
    310             char *errorstr;
    311             size_t errorstrlen;
    312307            Tcl_Obj* errorCode;
    313308
    314309            /* print error */
    315             /* get buffer large enough for additional message or the error code */
    316             errorstrlen = strlen(cmdstring) + strlen("Command failed: ") + 12;
    317             errorstr = malloc(errorstrlen);
    318             if (errorstr) {
    319                 snprintf(errorstr, errorstrlen, "Command failed: %s", cmdstring);
    320                 ui_info(interp, errorstr);
    321                 snprintf(errorstr, errorstrlen, "Exit code: %d", WEXITSTATUS(ret));
    322                 ui_info(interp, errorstr);
    323                 free(errorstr);
    324             }
     310            ui_info(interp, "Command failed: %s", cmdstring);
     311            ui_info(interp, "Exit code: %d", WEXITSTATUS(ret));
    325312
    326313            /* set errorCode [list CHILDSTATUS <pid> <code>] */
  • trunk/base/src/pextlib1.0/xinstall.c

    r120059 r138941  
    311311                }
    312312                else {
    313                         char msg[255];
    314 
    315                         snprintf(msg, sizeof msg, "%s: chdir(%s)\n", funcname, curdir);
    316                         ui_info(interp, msg);
     313                        ui_info(interp, "%s: chdir(%s)\n", funcname, curdir);
    317314                }
    318315        }
     
    444441        char backup[MAXPATHLEN], *p, pathbuf[MAXPATHLEN], tempfile[MAXPATHLEN];
    445442
    446         /* message contains function name, two paths and a little bit extra formatting */
    447         char msg[MAXPATHLEN * 2 + 32];
    448 
    449443        files_match = 0;
    450444
     
    548542                                return TCL_ERROR;
    549543                        }
    550                         snprintf(msg, sizeof msg, "%s: %s -> %s\n", funcname, from_name, to_name);
    551                         ui_info(interp, msg);
     544                        ui_info(interp, "%s: %s -> %s\n", funcname, from_name, to_name);
    552545                }
    553546                if (!devnull) {
     
    651644                                return TCL_ERROR;
    652645                        }
    653                         snprintf(msg, sizeof msg, "%s: %s -> %s\n", funcname, to_name, backup);
    654                         ui_info(interp, msg);
     646                        ui_info(interp, "%s: %s -> %s\n", funcname, to_name, backup);
    655647                        if (rename(to_name, backup) < 0) {
    656648                                char errmsg[255];
     
    665657                        }
    666658                }
    667                 snprintf(msg, sizeof msg, "%s: %s -> %s\n", funcname, from_name, to_name);
    668                 ui_info(interp, msg);
     659                ui_info(interp, "%s: %s -> %s\n", funcname, from_name, to_name);
    669660                if (rename(tempfile, to_name) < 0) {
    670661                        char errmsg[255];
     
    886877        int saved_errno = 0;
    887878        int newfd;
    888         char msg[256];
    889879
    890880        if (target) {
     
    910900                        }
    911901                        (void)snprintf(backup, MAXPATHLEN, "%s%s", path, suffix);
    912                         snprintf(msg, sizeof msg, "%s: %s -> %s\n", funcname, path, backup);
    913                         ui_info(interp, msg);
     902                        ui_info(interp, "%s: %s -> %s\n", funcname, path, backup);
    914903                        if (rename(path, backup) < 0) {
    915904                                char errmsg[255];
     
    10691058                                }
    10701059                                else {
    1071                                         char msg[255];
    1072 
    1073                                         snprintf(msg, sizeof msg, "%s: mkdir %s\n", funcname, path);
    1074                                         ui_info(interp, msg);
     1060                                        ui_info(interp, "%s: mkdir %s\n", funcname, path);
    10751061                                }
    10761062                        } else if (!S_ISDIR(sb.st_mode)) {
Note: See TracChangeset for help on using the changeset viewer.