|
Зависания на printf, нужен форматированный вывод в DBGU |
|
|
|
Jul 15 2010, 15:09
|

Частый гость
 
Группа: Свой
Сообщений: 156
Регистрация: 10-03-10
Из: Уфа
Пользователь №: 55 882

|
Цитата(MALLOY2 @ Jul 15 2010, 18:43)  format_buff нестоит делать локальной!!!!! стека не напасетесь!!!! Сделал Цитата(MALLOY2 @ Jul 15 2010, 18:43)  Стоит также заменить vsprintf на vsnprintf, но есть НО, почитайте о результатах которые возвращет эта функция, она может вернуть -1, для вашего кода это будет баг. Надо перед отправкой в порт проверить значение len и тип len сменить на знаковый. Думаю, теперь верно? Код void DBGU_fprint(const char *args, ...) { char len,i_len; va_list ap; va_start(ap, args); len = vsnprintf(format_buff, sizeof(format_buff), args, ap); va_end(ap); if (len > 0) for (i_len=0; i_len < len; i_len++) DBGU_PutChar(format_buff[i_len]); }
--------------------
Руслан
|
|
|
|
|
Jul 16 2010, 03:34
|

Частый гость
 
Группа: Свой
Сообщений: 156
Регистрация: 10-03-10
Из: Уфа
Пользователь №: 55 882

|
Цитата(aaarrr @ Jul 15 2010, 21:18)  Нет. Еще char на int замените.
Вообще, старайтесь не использовать переменные не полной размерности там, где в этом нет необходимости. У меня буфер коротенький - 100 байт  мне этого достаточно
--------------------
Руслан
|
|
|
|
|
Jul 16 2010, 10:35
|
Профессионал
    
Группа: Свой
Сообщений: 1 481
Регистрация: 10-04-05
Пользователь №: 4 007

|
Цитата(athlon64 @ Jul 15 2010, 18:09)  if (len > 0) Код if тут совсем лишний
|
|
|
|
|
Jul 18 2010, 15:46
|

Частый гость
 
Группа: Свой
Сообщений: 156
Регистрация: 10-03-10
Из: Уфа
Пользователь №: 55 882

|
Цитата(sergeeff @ Jul 16 2010, 16:35)  в смысле ">0" лишний? Код if (len) Цитата(aaarrr @ Jul 16 2010, 11:38)  С int'ом код получится короче и быстрее, размер буфера ни при чем. Хм, почему то всегда думал что с char экономится ОЗУ и код выполняется быстрее
--------------------
Руслан
|
|
|
|
|
Jul 18 2010, 16:11
|
Профессионал
    
Группа: Свой
Сообщений: 1 481
Регистрация: 10-04-05
Пользователь №: 4 007

|
Цитата(athlon64 @ Jul 18 2010, 18:46)  в смысле ">0" лишний? Код if (len) Хм, почему то всегда думал что с char экономится ОЗУ и код выполняется быстрее  Именно проверка >0 лишняя. Надо не думать, а знать. ARM - 32-битный процессор.
|
|
|
|
|
Jul 18 2010, 16:14
|
Гуру
     
Группа: Свой
Сообщений: 10 713
Регистрация: 11-12-04
Пользователь №: 1 448

|
Цитата(athlon64 @ Jul 18 2010, 19:46)  в смысле ">0" лишний? Код if (len) Нет, совсем лишний: при len <= 0 цикл не выполнится ни разу. Цитата(athlon64 @ Jul 18 2010, 19:46)  Хм, почему то всегда думал что с char экономится ОЗУ и код выполняется быстрее  ОЗУ в данном случае задействовано не будет, под локальные переменные будут использованы регистры процессора. А так как они 32 битные, то на эмуляции 8 битности получите оверхед по размеру кода и скорости.
|
|
|
|
|
Jul 19 2010, 15:20
|

Частый гость
 
Группа: Свой
Сообщений: 156
Регистрация: 10-03-10
Из: Уфа
Пользователь №: 55 882

|
Цитата(aaarrr @ Jul 18 2010, 22:14)  Нет, совсем лишний: при len <= 0 цикл не выполнится ни разу. Хм, логично  Цитата(sergeeff @ Jul 18 2010, 22:11)  Надо не думать, а знать. ARM - 32-битный процессор. Ну что ж, учту
--------------------
Руслан
|
|
|
|
|
Jul 19 2010, 22:06
|

кекс
     
Группа: Свой
Сообщений: 3 825
Регистрация: 17-12-05
Из: Киев
Пользователь №: 12 326

|
Цитата(MALLOY2 @ Jul 15 2010, 08:44)  Попробуйте так Код char format_buff[512]; //Сюда будет копироваться отформатированная строка
void out_fdbg(const char *args, ...) { .... } А что, очень даже талантливо. © Сделать совершенно мертвую память "ни под что".. под один только dbg_printf... По хорошему лучше вообще без доп буфера, чтобы putc выводил сразу туда куда надо.
|
|
|
|
|
Jul 20 2010, 06:19
|
Профессионал
    
Группа: Свой
Сообщений: 1 481
Регистрация: 10-04-05
Пользователь №: 4 007

|
Цитата(defunct @ Jul 20 2010, 02:06)  А что, очень даже талантливо. © Сделать совершенно мертвую память "ни под что".. под один только dbg_printf... По хорошему лучше вообще без доп буфера, чтобы putc выводил сразу туда куда надо. Ловите: Код /* Copyright 2001, 2002 Georges Menie (www.menie.org)
This program is free software; you can redistribute it and/or modify it under the terms of the GNU Lesser General Public License as published by the Free Software Foundation; either version 2 of the License, or (at your option) any later version.
This program is distributed in the hope that it will be useful, but WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU Lesser General Public License for more details.
You should have received a copy of the GNU Lesser General Public License along with this program; if not, write to the Free Software Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA */
/* putchar is the only external dependency for this file, if you have a working putchar, just remove the following define. If the function should be called something else, replace outbyte(c) by your own function call. */ #define putchar(c) outbyte(c)
static void printchar(char **str, int c) { extern int putchar(int c); if (str) { **str = c; ++(*str); } else (void)putchar(c); }
#define PAD_RIGHT 1 #define PAD_ZERO 2
static int prints(char **out, const char *string, int width, int pad) { register int pc = 0, padchar = ' ';
if (width > 0) { register int len = 0; register const char *ptr; for (ptr = string; *ptr; ++ptr) ++len; if (len >= width) width = 0; else width -= len; if (pad & PAD_ZERO) padchar = '0'; } if (!(pad & PAD_RIGHT)) { for (; width > 0; --width) { printchar (out, padchar); ++pc; } } for (; *string; ++string) { printchar (out, *string); ++pc; } for (; width > 0; --width) { printchar (out, padchar); ++pc; }
return pc; }
/* the following should be enough for 32 bit int */ #define PRINT_BUF_LEN 12
static int printi(char **out, int i, int b, int sg, int width, int pad, int letbase) { char print_buf[PRINT_BUF_LEN]; register char *s; register int t, neg = 0, pc = 0; register unsigned int u = i;
if (i == 0) { print_buf[0] = '0'; print_buf[1] = '\0'; return prints (out, print_buf, width, pad); }
if (sg && b == 10 && i < 0) { neg = 1; u = -i; }
s = print_buf + PRINT_BUF_LEN-1; *s = '\0';
while (u) { t = u % b; if( t >= 10 ) t += letbase - '0' - 10; *--s = t + '0'; u /= b; }
if (neg) { if( width && (pad & PAD_ZERO) ) { printchar (out, '-'); ++pc; --width; } else { *--s = '-'; } }
return pc + prints (out, s, width, pad); }
static int print(char **out, int *varg) { register int width, pad; register int pc = 0; register char *format = (char *)(*varg++); char scr[2];
for (; *format != 0; ++format) { if (*format == '%') { ++format; width = pad = 0; if (*format == '\0') break; if (*format == '%') goto out; if (*format == '-') { ++format; pad = PAD_RIGHT; } while (*format == '0') { ++format; pad |= PAD_ZERO; } for (; *format >= '0' && *format <= '9'; ++format) { width *= 10; width += *format - '0'; } if( *format == 's' ) { register char *s = *((char **)varg++); pc += prints (out, s?s:"(null)", width, pad); continue; } if( *format == 'd' ) { pc += printi (out, *varg++, 10, 1, width, pad, 'a'); continue; } if( *format == 'x' ) { pc += printi (out, *varg++, 16, 0, width, pad, 'a'); continue; } if( *format == 'X' ) { pc += printi (out, *varg++, 16, 0, width, pad, 'A'); continue; } if( *format == 'u' ) { pc += printi (out, *varg++, 10, 0, width, pad, 'a'); continue; } if( *format == 'c' ) { /* char are converted to int then pushed on the stack */ scr[0] = *varg++; scr[1] = '\0'; pc += prints (out, scr, width, pad); continue; } } else { out: printchar (out, *format); ++pc; } } if (out) **out = '\0'; return pc; }
/* assuming sizeof(void *) == sizeof(int) */
int qprintf(const char *format, ...) { register int *varg = (int *)(&format); return print(0, varg); }
int qsprintf(char *out, const char *format, ...) { register int *varg = (int *)(&format); return print(&out, varg); }
#ifdef TEST_PRINTF int main(void) { char *ptr = "Hello world!"; char *np = 0; int i = 5; unsigned int bs = sizeof(int)*8; int mi; char buf[80];
mi = (1 << (bs-1)) + 1; printf("%s\n", ptr); printf("printf test\n"); printf("%s is null pointer\n", np); printf("%d = 5\n", i); printf("%d = - max int\n", mi); printf("char %c = 'a'\n", 'a'); printf("hex %x = ff\n", 0xff); printf("hex %02x = 00\n", 0); printf("signed %d = unsigned %u = hex %x\n", -3, -3, -3); printf("%d %s(s)%", 0, "message"); printf("\n"); printf("%d %s(s) with %%\n", 0, "message"); sprintf(buf, "justif: \"%-10s\"\n", "left"); printf("%s", buf); sprintf(buf, "justif: \"%10s\"\n", "right"); printf("%s", buf); sprintf(buf, " 3: %04d zero padded\n", 3); printf("%s", buf); sprintf(buf, " 3: %-4d left justif.\n", 3); printf("%s", buf); sprintf(buf, " 3: %4d right justif.\n", 3); printf("%s", buf); sprintf(buf, "-3: %04d zero padded\n", -3); printf("%s", buf); sprintf(buf, "-3: %-4d left justif.\n", -3); printf("%s", buf); sprintf(buf, "-3: %4d right justif.\n", -3); printf("%s", buf);
return 0; }
/* * if you compile this file with * gcc -Wall $(YOUR_C_OPTIONS) -DTEST_PRINTF -c printf.c * you will get a normal warning: * printf.c:214: warning: spurious trailing `%' in format * this line is testing an invalid % at the end of the format string. * * this should display (on 32bit int machine) : * * Hello world! * printf test * (null) is null pointer * 5 = 5 * -2147483647 = - max int * char a = 'a' * hex ff = ff * hex 00 = 00 * signed -3 = unsigned 4294967293 = hex fffffffd * 0 message(s) * 0 message(s) with % * justif: "left " * justif: " right" * 3: 0003 zero padded * 3: 3 left justif. * 3: 3 right justif. * -3: -003 zero padded * -3: -3 left justif. * -3: -3 right justif. */
#endif
|
|
|
|
1 чел. читают эту тему (гостей: 1, скрытых пользователей: 0)
Пользователей: 0
|
|
|