i#1734: Create Dr. Fuzz: fix print_target_buffer bug

Commit log for first patchset:
---------------
i#1734: Create Dr. Fuzz: fix print_target_buffer bug

- terminate printing if (buffer_size - *sofar) < 2*MAX_PRINT_PER_ITERATION
---------------

[bruening, 2015-10-29 19:51:40]

https://codereview.appspot.com/270560043/diff/1/drmemory/fuzzer.c
File drmemory/fuzzer.c (right):

https://codereview.appspot.com/270560043/diff/1/drmemory/fuzzer.c#newcode352
drmemory/fuzzer.c:352: #define MAX_PRINT_PER_ITERATION 0x10 /* xx\nNOTE: ...\0
*/
Why in hex?  Why 16?  What is "xx"?  Should "NOTE" be hardcoded here when it's
parametrized in the actual print -- wouldn't strlen(prefix) be better?  How long
might "..." be?

https://codereview.appspot.com/270560043/diff/1/drmemory/fuzzer.c#newcode357
drmemory/fuzzer.c:357: if ((buffer_size - *sofar) < 2*MAX_PRINT_PER_ITERATION) {
I don't understand where this comes from: why 2x?  Why check this in the middle
of the loop between the 2 prints if it's talking about per-iter amounts? 
Doesn't it make more sense at the start or end of an iter?

Could alternatively use BUFPRINT_NO_ASSERT and when the buffer actually fills
up, clobber the end with "...", but that's not any simpler.

[zhaoqin, 2015-10-29 20:49:50]

Commit log for latest patchset:
---------------
i#1734: Create Dr. Fuzz: fix print_target_buffer bug

- stop printing if cannot print a new line

Review-URL: https://codereview.appspot.com/270560043
---------------

[bruening, 2015-10-29 21:03:17]

LGTM

Ideally there would be a test that hits the limit

[zhaoqin, 2015-10-30 19:23:22]

Committed as
https://github.com/DynamoRIO/drmemory/commit/8a1bb53356bdf61dad624530bf20c544...

Final commit log: 
---------------
i#1734: Create Dr. Fuzz: fix print_target_buffer bug

- stop printing if cannot print a full new line

Review-URL: https://codereview.appspot.com/270560043
---------------