diff options
| author | Philipp Krones <hello@philkrones.com> | 2022-11-21 19:59:07 +0100 |
|---|---|---|
| committer | Philipp Krones <hello@philkrones.com> | 2022-11-21 20:01:17 +0100 |
| commit | fd5b85c957d92a3bc5fc754407d1e56aa9178d27 (patch) | |
| tree | cba25a7cb16b8f5981eb75b1138ece61bb412fb6 /src/docs/comparison_to_empty.txt | |
| parent | 80a4699aaeb17903cd55c3159764febf3ff4e7e9 (diff) | |
| parent | 51ec465cc3dfb62a2dad7e808b399fa76a1c9170 (diff) | |
Merge remote-tracking branch 'upstream/master' into rustup
Diffstat (limited to 'src/docs/comparison_to_empty.txt')
| -rw-r--r-- | src/docs/comparison_to_empty.txt | 31 |
1 files changed, 0 insertions, 31 deletions
diff --git a/src/docs/comparison_to_empty.txt b/src/docs/comparison_to_empty.txt deleted file mode 100644 index db6f74fe270..00000000000 --- a/src/docs/comparison_to_empty.txt +++ /dev/null @@ -1,31 +0,0 @@ -### What it does -Checks for comparing to an empty slice such as `""` or `[]`, -and suggests using `.is_empty()` where applicable. - -### Why is this bad? -Some structures can answer `.is_empty()` much faster -than checking for equality. So it is good to get into the habit of using -`.is_empty()`, and having it is cheap. -Besides, it makes the intent clearer than a manual comparison in some contexts. - -### Example - -``` -if s == "" { - .. -} - -if arr == [] { - .. -} -``` -Use instead: -``` -if s.is_empty() { - .. -} - -if arr.is_empty() { - .. -} -``` \ No newline at end of file |