remove more printf

This commit is contained in:
krakjoe 2013-11-12 23:31:46 +00:00
parent afa216fd7e
commit ce78503e15
4 changed files with 62 additions and 46 deletions

View file

@ -41,12 +41,12 @@ void phpdbg_list_file(const char *filename, long count, long offset TSRMLS_DC) /
unsigned int line = 0, displayed = 0; unsigned int line = 0, displayed = 0;
if (VCWD_STAT(filename, &st) == -1) { if (VCWD_STAT(filename, &st) == -1) {
printf("[Failed to stat file %s]\n", filename); phpdbg_error("Failed to stat file %s", filename);
return; return;
} }
#ifndef _WIN32 #ifndef _WIN32
if ((fd = VCWD_OPEN(filename, O_RDONLY)) == -1) { if ((fd = VCWD_OPEN(filename, O_RDONLY)) == -1) {
printf("[Failed to open file %s to list]\n", filename); phpdbg_error("Failed to open file %s to list", filename);
return; return;
} }
@ -55,20 +55,20 @@ void phpdbg_list_file(const char *filename, long count, long offset TSRMLS_DC) /
#else #else
fd = CreateFile(filename, GENERIC_READ, 0, NULL, OPEN_EXISTING, 0, NULL); fd = CreateFile(filename, GENERIC_READ, 0, NULL, OPEN_EXISTING, 0, NULL);
if (fd == INVALID_HANDLE_VALUE) { if (fd == INVALID_HANDLE_VALUE) {
printf("[Failed to open file!]\n"); phpdbg_error("Failed to open file!");
return; return;
} }
map = CreateFileMapping(fd, NULL, PAGE_EXECUTE_READ, 0, 0, 0); map = CreateFileMapping(fd, NULL, PAGE_EXECUTE_READ, 0, 0, 0);
if (map == NULL) { if (map == NULL) {
printf("[Failed to map file!]\n"); phpdbg_error("Failed to map file!");
CloseHandle(fd); CloseHandle(fd);
return; return;
} }
last_pos = mem = (char*) MapViewOfFile(map, FILE_MAP_READ, 0, 0, 0); last_pos = mem = (char*) MapViewOfFile(map, FILE_MAP_READ, 0, 0, 0);
if (mem == NULL) { if (mem == NULL) {
printf("[Failed to map file in memory]\n"); phpdbg_error("Failed to map file in memory");
CloseHandle(map); CloseHandle(map);
CloseHandle(fd); CloseHandle(fd);
return; return;
@ -87,7 +87,7 @@ void phpdbg_list_file(const char *filename, long count, long offset TSRMLS_DC) /
if (!offset || offset <= line) { if (!offset || offset <= line) {
/* Without offset, or offset reached */ /* Without offset, or offset reached */
printf("%05u: %.*s\n", line, (int)(pos - last_pos), last_pos); phpdbg_writeln("%05u: %.*s", line, (int)(pos - last_pos), last_pos);
++displayed; ++displayed;
} }

View file

@ -162,7 +162,7 @@ static PHPDBG_COMMAND(eval) /* {{{ */
&retval, "eval()'d code" TSRMLS_CC) == SUCCESS) { &retval, "eval()'d code" TSRMLS_CC) == SUCCESS) {
zend_print_zval_r(&retval, 0 TSRMLS_CC); zend_print_zval_r(&retval, 0 TSRMLS_CC);
zval_dtor(&retval); zval_dtor(&retval);
printf("\n"); phpdbg_writeln(EMPTY);
} }
/* switch stepping back on */ /* switch stepping back on */
@ -194,12 +194,12 @@ static PHPDBG_COMMAND(back) /* {{{ */
zend_hash_get_current_data_ex(Z_ARRVAL(zbacktrace), (void**)&tmp, &position) == SUCCESS; zend_hash_get_current_data_ex(Z_ARRVAL(zbacktrace), (void**)&tmp, &position) == SUCCESS;
zend_hash_move_forward_ex(Z_ARRVAL(zbacktrace), &position)) { zend_hash_move_forward_ex(Z_ARRVAL(zbacktrace), &position)) {
if (i++) { if (i++) {
phpdbg_write(","); phpdbg_writeln(",");
} }
zend_print_flat_zval_r(*tmp TSRMLS_CC); zend_print_flat_zval_r(*tmp TSRMLS_CC);
} }
phpdbg_write(EMPTY); phpdbg_writeln(EMPTY);
zval_dtor(&zbacktrace); zval_dtor(&zbacktrace);
return SUCCESS; return SUCCESS;
@ -217,32 +217,32 @@ static PHPDBG_COMMAND(print) /* {{{ */
PHPDBG_SEP_LINE(TSRMLS_C); PHPDBG_SEP_LINE(TSRMLS_C);
phpdbg_notice("Execution Context Information:"); phpdbg_notice("Execution Context Information:");
#ifdef HAVE_LIBREADLINE #ifdef HAVE_LIBREADLINE
phpdbg_write("Readline\tyes"); phpdbg_writeln("Readline\tyes");
#else #else
phpdbg_write("Readline\tno"); phpdbg_writeln("Readline\tno");
#endif #endif
phpdbg_write("Exec\t\t%s", PHPDBG_G(exec) ? PHPDBG_G(exec) : "none"); phpdbg_writeln("Exec\t\t%s", PHPDBG_G(exec) ? PHPDBG_G(exec) : "none");
phpdbg_write("Compiled\t%s", PHPDBG_G(ops) ? "yes" : "no"); phpdbg_writeln("Compiled\t%s", PHPDBG_G(ops) ? "yes" : "no");
phpdbg_write("Stepping\t%s", (PHPDBG_G(flags) & PHPDBG_IS_STEPPING) ? "on" : "off"); phpdbg_writeln("Stepping\t%s", (PHPDBG_G(flags) & PHPDBG_IS_STEPPING) ? "on" : "off");
phpdbg_write("Quietness\t%s", (PHPDBG_G(flags) & PHPDBG_IS_QUIET) ? "on" : "off"); phpdbg_writeln("Quietness\t%s", (PHPDBG_G(flags) & PHPDBG_IS_QUIET) ? "on" : "off");
if (PHPDBG_G(ops)) { if (PHPDBG_G(ops)) {
phpdbg_write("Opcodes\t\t%d", PHPDBG_G(ops)->last); phpdbg_writeln("Opcodes\t\t%d", PHPDBG_G(ops)->last);
if (PHPDBG_G(ops)->last_var) { if (PHPDBG_G(ops)->last_var) {
phpdbg_write("Variables\t%d", PHPDBG_G(ops)->last_var-1); phpdbg_writeln("Variables\t%d", PHPDBG_G(ops)->last_var-1);
} else { } else {
phpdbg_write("Variables\tNone"); phpdbg_writeln("Variables\tNone");
} }
} }
phpdbg_write("Executing\t%s", EG(in_execution) ? "yes" : "no"); phpdbg_writeln("Executing\t%s", EG(in_execution) ? "yes" : "no");
if (EG(in_execution)) { if (EG(in_execution)) {
phpdbg_write("VM Return\t%d", PHPDBG_G(vmret)); phpdbg_writeln("VM Return\t%d", PHPDBG_G(vmret));
} }
phpdbg_write("Classes\t\t%d", zend_hash_num_elements(EG(class_table))); phpdbg_writeln("Classes\t\t%d", zend_hash_num_elements(EG(class_table)));
phpdbg_write("Functions\t%d", zend_hash_num_elements(EG(function_table))); phpdbg_writeln("Functions\t%d", zend_hash_num_elements(EG(function_table)));
phpdbg_write("Constants\t%d", zend_hash_num_elements(EG(zend_constants))); phpdbg_writeln("Constants\t%d", zend_hash_num_elements(EG(zend_constants)));
phpdbg_write("Included\t%d", zend_hash_num_elements(&EG(included_files))); phpdbg_writeln("Included\t%d", zend_hash_num_elements(&EG(included_files)));
phpdbg_print_breakpoints(PHPDBG_BREAK_FILE TSRMLS_CC); phpdbg_print_breakpoints(PHPDBG_BREAK_FILE TSRMLS_CC);
phpdbg_print_breakpoints(PHPDBG_BREAK_SYM TSRMLS_CC); phpdbg_print_breakpoints(PHPDBG_BREAK_SYM TSRMLS_CC);
@ -359,19 +359,19 @@ static PHPDBG_COMMAND(clean) /* {{{ */
if (!EG(in_execution)) { if (!EG(in_execution)) {
phpdbg_notice("Cleaning Execution Environment"); phpdbg_notice("Cleaning Execution Environment");
phpdbg_write("Classes\t%d", zend_hash_num_elements(EG(class_table))); phpdbg_writeln("Classes\t\t\t%d", zend_hash_num_elements(EG(class_table)));
phpdbg_write("Functions\t%d", zend_hash_num_elements(EG(function_table))); phpdbg_writeln("Functions\t\t%d", zend_hash_num_elements(EG(function_table)));
phpdbg_write("Constants\t%d", zend_hash_num_elements(EG(zend_constants))); phpdbg_writeln("Constants\t\t%d", zend_hash_num_elements(EG(zend_constants)));
phpdbg_write("Includes\t%d", zend_hash_num_elements(&EG(included_files))); phpdbg_writeln("Includes\t\t%d", zend_hash_num_elements(&EG(included_files)));
phpdbg_clean(1 TSRMLS_CC); phpdbg_clean(1 TSRMLS_CC);
phpdbg_notice("Clean Execution Environment"); phpdbg_notice("Clean Execution Environment");
phpdbg_write("Classes\t%d", zend_hash_num_elements(EG(class_table))); phpdbg_writeln("Classes\t\t\t%d", zend_hash_num_elements(EG(class_table)));
phpdbg_write("Functions\t%d", zend_hash_num_elements(EG(function_table))); phpdbg_writeln("Functions\t\t%d", zend_hash_num_elements(EG(function_table)));
phpdbg_write("Constants\t%d", zend_hash_num_elements(EG(zend_constants))); phpdbg_writeln("Constants\t\t%d", zend_hash_num_elements(EG(zend_constants)));
phpdbg_write("Includes\t%d", zend_hash_num_elements(&EG(included_files))); phpdbg_writeln("Includes\t\t%d", zend_hash_num_elements(&EG(included_files)));
} else { } else {
phpdbg_error("Cannot clean environment while executing"); phpdbg_error("Cannot clean environment while executing");
return FAILURE; return FAILURE;
@ -384,10 +384,10 @@ static PHPDBG_COMMAND(clear) /* {{{ */
{ {
phpdbg_notice("Clearing Breakpoints"); phpdbg_notice("Clearing Breakpoints");
phpdbg_write("File\t%d", zend_hash_num_elements(&PHPDBG_G(bp)[PHPDBG_BREAK_FILE])); phpdbg_writeln("File\t\t\t%d", zend_hash_num_elements(&PHPDBG_G(bp)[PHPDBG_BREAK_FILE]));
phpdbg_write("Functions\t%d", zend_hash_num_elements(&PHPDBG_G(bp)[PHPDBG_BREAK_SYM])); phpdbg_writeln("Functions\t\t%d", zend_hash_num_elements(&PHPDBG_G(bp)[PHPDBG_BREAK_SYM]));
phpdbg_write("Methods\t%d", zend_hash_num_elements(&PHPDBG_G(bp)[PHPDBG_BREAK_METHOD])); phpdbg_writeln("Methods\t\t\t%d", zend_hash_num_elements(&PHPDBG_G(bp)[PHPDBG_BREAK_METHOD]));
phpdbg_write("Oplines\t%d", zend_hash_num_elements(&PHPDBG_G(bp)[PHPDBG_BREAK_OPLINE])); phpdbg_writeln("Oplines\t\t\t%d", zend_hash_num_elements(&PHPDBG_G(bp)[PHPDBG_BREAK_OPLINE]));
phpdbg_clear_breakpoints(TSRMLS_C); phpdbg_clear_breakpoints(TSRMLS_C);
@ -407,19 +407,19 @@ static PHPDBG_COMMAND(help) /* {{{ */
const phpdbg_command_t *prompt_command = phpdbg_prompt_commands; const phpdbg_command_t *prompt_command = phpdbg_prompt_commands;
const phpdbg_command_t *help_command = phpdbg_help_commands; const phpdbg_command_t *help_command = phpdbg_help_commands;
phpdbg_write("To get help regarding a specific command type \"help command\""); phpdbg_writeln("To get help regarding a specific command type \"help command\"");
phpdbg_notice("Commands"); phpdbg_notice("Commands");
while (prompt_command && prompt_command->name) { while (prompt_command && prompt_command->name) {
phpdbg_write("\t%s\t%s", prompt_command->name, prompt_command->tip); phpdbg_writeln("\t%s\t%s", prompt_command->name, prompt_command->tip);
++prompt_command; ++prompt_command;
} }
phpdbg_notice("Helpers Loaded"); phpdbg_notice("Helpers Loaded");
while (help_command && help_command->name) { while (help_command && help_command->name) {
phpdbg_write("\t%s\t%s", help_command->name, help_command->tip); phpdbg_writeln("\t%s\t%s", help_command->name, help_command->tip);
++help_command; ++help_command;
} }
} }
@ -543,7 +543,7 @@ int phpdbg_interactive(TSRMLS_D) /* {{{ */
char cmd[PHPDBG_MAX_CMD]; char cmd[PHPDBG_MAX_CMD];
phpdbg_interactive_enter: phpdbg_interactive_enter:
printf(PHPDBG_PROMPT_LINE(TSRMLS_C)); phpdbg_write(PROMPT);
while (!(PHPDBG_G(flags) & PHPDBG_IS_QUITTING) && while (!(PHPDBG_G(flags) & PHPDBG_IS_QUITTING) &&
fgets(cmd, PHPDBG_MAX_CMD, stdin) != NULL) { fgets(cmd, PHPDBG_MAX_CMD, stdin) != NULL) {

View file

@ -96,12 +96,23 @@ void phpdbg_print(int type TSRMLS_DC, const char *format, ...) /* {{{ */
((PHPDBG_G(flags) & PHPDBG_IS_COLOURED) ? "]\033[0m" : "]")); ((PHPDBG_G(flags) & PHPDBG_IS_COLOURED) ? "]\033[0m" : "]"));
break; break;
case WRITE: case WRITELN: {
printf("%s%s%s\n", if (buffer) {
((PHPDBG_G(flags) & PHPDBG_IS_COLOURED) ? "\033[37m" : ""), printf("%s%s%s\n",
buffer, ((PHPDBG_G(flags) & PHPDBG_IS_COLOURED) ? "\033[37m" : ""),
((PHPDBG_G(flags) & PHPDBG_IS_COLOURED) ? "\033[0m" : "")); buffer,
break; ((PHPDBG_G(flags) & PHPDBG_IS_COLOURED) ? "\033[0m" : ""));
} else {
printf("\n");
}
} break;
case WRITE: if (buffer) {
printf("%s%s%s",
((PHPDBG_G(flags) & PHPDBG_IS_COLOURED) ? "\033[37m" : ""),
buffer,
((PHPDBG_G(flags) & PHPDBG_IS_COLOURED) ? "\033[0m" : ""));
} break;
} }
if (buffer) { if (buffer) {

View file

@ -36,6 +36,7 @@ int phpdbg_is_class_method(const char*, size_t, char**, char**);
enum { enum {
ERROR = 1, ERROR = 1,
NOTICE, NOTICE,
WRITELN,
WRITE WRITE
}; };
@ -43,9 +44,13 @@ void phpdbg_print(int TSRMLS_DC, const char*, ...);
#define phpdbg_error(fmt, ...) phpdbg_print(ERROR TSRMLS_CC, fmt, ##__VA_ARGS__) #define phpdbg_error(fmt, ...) phpdbg_print(ERROR TSRMLS_CC, fmt, ##__VA_ARGS__)
#define phpdbg_notice(fmt, ...) phpdbg_print(NOTICE TSRMLS_CC, fmt, ##__VA_ARGS__) #define phpdbg_notice(fmt, ...) phpdbg_print(NOTICE TSRMLS_CC, fmt, ##__VA_ARGS__)
#define phpdbg_writeln(fmt, ...) phpdbg_print(WRITELN TSRMLS_CC, fmt, ##__VA_ARGS__)
#define phpdbg_write(fmt, ...) phpdbg_print(WRITE TSRMLS_CC, fmt, ##__VA_ARGS__) #define phpdbg_write(fmt, ...) phpdbg_print(WRITE TSRMLS_CC, fmt, ##__VA_ARGS__)
/* {{{ For writing blank lines */ /* {{{ For writing blank lines */
#define EMPTY "" /* }}} */ #define EMPTY "" /* }}} */
/* {{{ For prompt lines */
#define PROMPT ((PHPDBG_G(flags) & PHPDBG_IS_COLOURED) ? "\033[1;64mphpdbg>\033[0m " : "phpdbg> ") /* }}} */
#endif /* PHPDBG_UTILS_H */ #endif /* PHPDBG_UTILS_H */