diff options
author | Brian Salomon <bsalomon@google.com> | 2018-03-13 09:40:32 -0400 |
---|---|---|
committer | Mike Klein <mtklein@chromium.org> | 2018-03-13 13:52:10 +0000 |
commit | 4000a5e6226e2a9bc9107eee05cfceefa67ee384 (patch) | |
tree | 239d6bfd2928221d8bfdbf4f04bccd7d5cf69d67 /site/dev/contrib | |
parent | 2f06a53b257571b566cbce792032bd0be208ae5d (diff) |
Remove yoda rule from style guide
No-Try: true
Docs-Preview: https://skia.org/?cl=114023
Change-Id: Iefd2205b1aa68a19a835b033464a3407edb7cf18
Reviewed-on: https://skia-review.googlesource.com/114023
Reviewed-by: Mike Klein <mtklein@chromium.org>
Diffstat (limited to 'site/dev/contrib')
-rw-r--r-- | site/dev/contrib/style.md | 22 |
1 files changed, 0 insertions, 22 deletions
diff --git a/site/dev/contrib/style.md b/site/dev/contrib/style.md index 6a8c604af5..bded3a3b70 100644 --- a/site/dev/contrib/style.md +++ b/site/dev/contrib/style.md @@ -387,28 +387,6 @@ Method calls within method calls should be prefixed with dereference of the this->method(); ~~~~ -Comparisons ------------ - -We prefer that equality operators between lvalues and rvalues place the lvalue -on the right: - -<!--?prettify?--> -~~~~ -if (7 == luckyNumber) { - ... -} -~~~~ - -However, inequality operators need not follow this rule: - -<!--?prettify?--> -~~~~ -if (count > 0) { - ... -} -~~~~ - Comments -------- |