Skip to content

Commit 1883b3f

Browse files
Zhenghang Humeta-codesync[bot]
authored andcommitted
Correct @or macro documentation
Summary: - Looked into `or` macro impl to confirm if it supports short circuit evaluation (it does). Found that the documentation is wrong. It was [brought up in diff review](https://www.internalfb.com/diff/D2659697?dst_version_fbid=512163485661447&transaction_fbid=1230557897025059) 10+ years ago but wasn't corrected at the time Differential Revision: D99478498 fbshipit-source-id: 76d58748ecfc20f25e5cb7c20424a7a55cfe7b35
1 parent 93b1d99 commit 1883b3f

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

mcrouter/lib/config/ConfigPreprocessor.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1131,7 +1131,7 @@ class ConfigPreprocessor::BuiltIns {
11311131
}
11321132

11331133
/**
1134-
* Returns true if A || B. B is evaluated only if A is true.
1134+
* Returns true if A || B. B is evaluated only if A is false.
11351135
* A and B should be booleans.
11361136
* Usage: @or(A,B)
11371137
*/

0 commit comments

Comments
 (0)