<p dir="ltr"><br>
On 23 May 2014 09:04, "Tobias Grosser" <<a href="mailto:tobias@grosser.es">tobias@grosser.es</a>> wrote:<br>
><br>
> On 23/05/2014 09:02, Alexander Kornienko wrote:<br>
>><br>
>> Slightly shorter and more expensive, I would say.<br>
><br>
><br>
> It is just a minor change, but more expensive? That does not sound beneficial?</p>
<p dir="ltr">s/expensive/expressive/ (Android keyboard...)</p>
<p dir="ltr">><br>
> Also, you missed this question:<br>
><br>
><br>
>>>         }<br>
>>>><br>
>>>>      }<br>
>>>>      return llvm::make_error_code(llvm::errc::not_supported);<br>
>>>><br>
>>><br>
>>> Should this line be changed too?</p>
<p dir="ltr">AFAIK, there's no shortcut for this error code. Success code is special, it has a dedicated static member for simplicity and expressiveness.</p>
<p dir="ltr">><br>
><br>
> Cheers,<br>
> Tobias<br>
</p>