if(arr) now a warning

[via Digitalmars-d]

On Wednesday, 29 April 2015 at 11:28:02 UTC, Vladimir Panteleev 
wrote:
> On Wednesday, 29 April 2015 at 11:07:52 UTC, Marc Schütz wrote:
> ://github.com/rejectedsoftware/vibe.d/commit/e9e66f4e726db64d15e078dc472606b57783728a#diff-a0c0675933703d01a5d6ad8ebfc097abL79
>
> Are you talking about the last diff in particular? Because I 
> would argue that that one is a poor design decision in Vibe 
> (overloaded functions' return values should have the same 
> semantics). What if one overload returned "bool" for 
> success/failure and another a "size_t" for number of characters 
> written to the error message buffer?

You're probably right, it's not the best design. But still, the 
original version compiled fine because that string was implicitly 
convertible to bool, which the DMD change caught.