# HG changeset patch # User Matti Hamalainen # Date 1457518241 -7200 # Node ID 371567cf0a3815196535f4af7195ff2dd334065a # Parent 04cb03baf114dfaa11b1b8eb5a9d8f7fa18abeae Add some XXX TODO FIXME in here :P diff -r 04cb03baf114 -r 371567cf0a38 th_string.c --- a/th_string.c Wed Mar 09 12:07:10 2016 +0200 +++ b/th_string.c Wed Mar 09 12:10:41 2016 +0200 @@ -246,6 +246,11 @@ (f_neg ? '-' : ((f_flags & TH_PF_SPACE) ? ' ' : 0))); // Calculate necessary padding, etc + // + // << XXX TODO FIXME: The logic here is not very elegant, and it's incorrect + // at least for some alternate format modifier cases. + // + int nlen = (f_sign ? 1 : 0) + (f_altstr ? strlen(f_altstr) : 0); int qlen = (f_prec > f_len ? f_prec : f_len) + nlen; @@ -276,6 +281,8 @@ PP_PRINTF(": vret=%3d, f_flags=[%s], f_unsig=%d, f_sign='%c', f_len=%3d, f_width=%3d, f_prec=%3d, nwidth=%3d, nprec=%3d, qlen=%3d\n", vret, get_flags(f_flags), f_unsig, f_sign ? f_sign : '?', f_len, f_width, f_prec, nwidth, nprec, qlen); + // << XXX TODO FIXME + // Prefix padding if ((ret = th_printf_pad_pre(ctx, vputch, nwidth, f_flags)) == EOF) return ret;