Support Custom Header-Printing During Transform
AbandonedPublic
Actions

Authored by Julian Eisel (Severin) on Jun 30 2015, 3:13 AM.

Details

Reviewers

Campbell Barton (campbellbarton)

Summary

Planned to be used in D1373.

Also added two BLI_str functions to remove trailing/leading whitespace in a string.

The custom string (custom as in: Defined in code but not on an ugly low level) is currently appended to the existing string, although it might be better to do it the other way around since the existing string changes its width continuously while transforming. The functions to remove whitespace wouldn't be needed anymore then, but think it would still be useful for some other cases?!

Diff Detail

Repository: rB Blender
Branch: master

Event Timeline

Julian Eisel (Severin) updated this revision to Diff 4544.Jun 30 2015, 3:13 AM

Julian Eisel (Severin) retitled this revision from to Support Custom Header-Printing During Transform.

Julian Eisel (Severin) added a reviewer: Campbell Barton (campbellbarton).

Julian Eisel (Severin) updated this object.

Julian Eisel (Severin) updated this object.Jun 30 2015, 3:20 AM

Fixing whitespace issues in whitespace remove functions ;)

Generally having whitespace stripping functions is good.

Would use Python like naming BLI_str_lstrip / BLI_str_rstrip (we already have BLI_str_rstrip_float_zero for example).
Its assumed src, dst dont overlap, can use __restrict in header (also BLI_assert(src != dst) in function for good measure) that only covers basic mistakes and wont catch overlap.
When theres a source and destination string, its good to pass the length of the destination string to avoid buffer overrun.
For utility functions like this its good to add gtest, Try pass in all whitespace, no whitespace, single character surrounded with whitespace, empty string... etc.