On 2017-02-16 18:21, Jean-Yves Avenard wrote:
>
>
> On 17/02/17 00:17, Jeff Gilbert wrote:
>> If there are so many parens that we require an editor to disambiguate
>> them, it's too dense. Add whitespace or decompose the expression until
>> readable.
>>
> I'd love to know how adding any whitespaces could make any logical
> expressions clearer that what I posted above.
>
> That's the point, there's no more whitespace to add, nor a need to
> decompose further, it's already been fully expanded.
>
There is lots of whitespace to add: If the expression is over a line
long, then use indentation so the block structure follows the logical
structure:
____return
________(
____________(aCodecMask & VPXDecoder::VP8)
____________&&
____________aMimeType.EqualsLiteral("video/webm; codecs=vp8")
________)
________||
________(
____________(aCodecMask & VPXDecoder::VP9)
____________&&
____________aMimeType.EqualsLiteral("video/webm; codecs=vp9")
________)
________||
________(
____________(aCodecMask & VPXDecoder::VP9)
____________&&
____________aMimeType.EqualsLiteral("video/vp9")
________)
____;
_______________________________________________
dev-platform mailing list
dev-platform@lists.mozilla.org
https://lists.mozilla.org/listinfo/dev-platform
