Tom,
On a related note, I get a failure on test 66 even after the revert back to %d,
so you seem to have exposed another consistency issue as well. Testing printf
directly, I get:
printf("%010s", "1");
warning: '0' flag used with ‘%s’ printf format
0000000001
Apparently 0-padding strings isn't standard anyways.
Cheers!
Sean
On Feb 11, 2014, at 2:49 PM, tbrowd...@users.sourceforge.net wrote:
> Revision: 59842
> http://sourceforge.net/p/brlcad/code/59842
> Author: tbrowder2
> Date: 2014-02-11 19:49:28 +0000 (Tue, 11 Feb 2014)
> Log Message:
> -----------
> add two new tests to prove revision 58221 and disprove its reversion at
> revision 58231
>
> Revision Links:
> --------------
> http://sourceforge.net/p/brlcad/code/58221
> http://sourceforge.net/p/brlcad/code/58231
>
> Modified Paths:
> --------------
> brlcad/trunk/src/libbu/tests/CMakeLists.txt
> brlcad/trunk/src/libbu/tests/bu_vls_vprintf.c
>
> Modified: brlcad/trunk/src/libbu/tests/CMakeLists.txt
> ===================================================================
> --- brlcad/trunk/src/libbu/tests/CMakeLists.txt 2014-02-11 17:45:31 UTC
> (rev 59841)
> +++ brlcad/trunk/src/libbu/tests/CMakeLists.txt 2014-02-11 19:49:28 UTC
> (rev 59842)
> @@ -332,6 +332,8 @@
> add_test(bu_vls_vprintf_test_63 tester_bu_vls_vprintf 63)
> add_test(bu_vls_vprintf_test_64 tester_bu_vls_vprintf 64)
> add_test(bu_vls_vprintf_test_65 tester_bu_vls_vprintf 65)
> +add_test(bu_vls_vprintf_test_66 tester_bu_vls_vprintf 66)
> +add_test(bu_vls_vprintf_test_67 tester_bu_vls_vprintf 67)
>
> #
> # ************ vls.c tests *************
>
> Modified: brlcad/trunk/src/libbu/tests/bu_vls_vprintf.c
> ===================================================================
> --- brlcad/trunk/src/libbu/tests/bu_vls_vprintf.c 2014-02-11 17:45:31 UTC
> (rev 59841)
> +++ brlcad/trunk/src/libbu/tests/bu_vls_vprintf.c 2014-02-11 19:49:28 UTC
> (rev 59842)
> @@ -297,6 +297,20 @@
> return test_vls("%20.10s", "123456789012345");
> case 65:
> return test_vls("%-20.10s", "123456789012345");
> + case 66:
> + /* testing the '%o' inside vls_printf.c, at line 554, the
> + * '010' is currently taken as octal which is 8 in decimal
> + * (base 10), thus the printed string width is 8; however,
> + * sprintf correctly interprets the number as decimal so
> + * the printed field width is 10 characters */
> + return test_vls("%010s", "1");
> + case 67:
> + /* testing the '%o' inside vls_printf.c, at line 561, the
> + * '030' is currently taken as octal which is 24 in
> + * decimal (base 10), thus the max printed string width is
> + * 24; however, sprintf correctly interprets the number as
> + * decimal so 30 characters are printed */
> + return test_vls("%.030s",
> "1234567890123456789012345678901234567890");
>
> /* this test needs a relook
> return test_vls("%H", 123);
>
> This was sent by the SourceForge.net collaborative development platform, the
> world's largest Open Source development site.
>
>
> ------------------------------------------------------------------------------
> Android apps run on BlackBerry 10
> Introducing the new BlackBerry 10.2.1 Runtime for Android apps.
> Now with support for Jelly Bean, Bluetooth, Mapview and more.
> Get your Android app in front of a whole new audience. Start now.
> http://pubads.g.doubleclick.net/gampad/clk?id=124407151&iu=/4140/ostg.clktrk
> _______________________________________________
> BRL-CAD Source Commits mailing list
> brlcad-comm...@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/brlcad-commits
------------------------------------------------------------------------------
Android apps run on BlackBerry 10
Introducing the new BlackBerry 10.2.1 Runtime for Android apps.
Now with support for Jelly Bean, Bluetooth, Mapview and more.
Get your Android app in front of a whole new audience. Start now.
http://pubads.g.doubleclick.net/gampad/clk?id=124407151&iu=/4140/ostg.clktrk
_______________________________________________
BRL-CAD Developer mailing list
brlcad-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/brlcad-devel
