The FX_BSTRC() macro automatically applies a sizeof()-1 to its argument, so as
to avoid having to do a strlen in the constructor. But compilers will optimize
strlen("foo") => 3, so long as the strlen is visible to it in the header.
(I believe blink relies on this exact behaviour).
Even if not, there are better ways to trick a C++ compiler into deducing a
string length.
Having this littered throughout the code base reduces readability; also it's
not used consistently where it would be needed.
Original issue reported on code.google.com by tsepez@chromium.org on 5 May 2015 at 8:31
Original issue reported on code.google.com by
tsepez@chromium.orgon 5 May 2015 at 8:31