Assert Messages with printf style formatting #2

burner · 2013-05-09T03:26:07Z

For easier more outspoken error messages asserts now have printf style formatting.
This pull request also removes some symmetric code paths.

Old api has not changed

…sage I cleaned up some semetric impls like assertSame and assertNotSame to use the same underlaying impl as well. I also added unittest to for all changes. The preexisting api did not change.

linkrope · 2013-05-12T17:05:06Z

What about "template bloat"?
A D.learn post explains that file and line should better not be template parameters:
http://forum.dlang.org/post/[email protected]

What's so bad about "print%c style formatting".format('c')?

burner · 2013-05-13T08:30:51Z

IMHO not really a problem as these functions are quite small.
because typing wastes times and format is a template function anyway so you have template bloat one way or the other, sure a bit more this way but with less typing and less space wasted

burner added 5 commits May 3, 2013 19:40

small performance tweak

5c62c60

assertArrayEqual can have some more const

182aa88

file and line are now template arguments, this is much nicer

b7d9f8f

one needs printf style formatting for assert messsages

56b1669

now every assert but the assertEventually has printf style assert mes…

c135c2c

…sage I cleaned up some semetric impls like assertSame and assertNotSame to use the same underlaying impl as well. I also added unittest to for all changes. The preexisting api did not change.

burner closed this Nov 21, 2013

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Assert Messages with printf style formatting #2

Assert Messages with printf style formatting #2

burner commented May 9, 2013

linkrope commented May 12, 2013

burner commented May 13, 2013

Assert Messages with printf style formatting #2

Assert Messages with printf style formatting #2

Conversation

burner commented May 9, 2013

linkrope commented May 12, 2013

burner commented May 13, 2013