forked from basho/erlang_guidelines
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[Fixes inaka#8] REJECTED: LC instead of lists:map
- Loading branch information
1 parent
1b268ff
commit 578c131
Showing
2 changed files
with
26 additions
and
1 deletion.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,25 @@ | ||
## Rejected rules and suggestions | ||
|
||
*** | ||
|
||
##### rule | ||
> Use tail-recursive functions instead of foldl | ||
##### rejected because | ||
We love high-order functions! | ||
|
||
*** | ||
|
||
##### rule | ||
> Using ``'andalso'``, ``'orelse'``, and the like, might save a 'case' or two | ||
##### rejected because | ||
They're not intended to be used that way and the resulting code messes up with dialyzer | ||
|
||
*** | ||
|
||
##### rule | ||
> Replace ``lists:map`` with either ``lists:foreach`` or list comprehensions | ||
##### rejected because | ||
There is not true that LCs are better than ``list:map`` in all scenarios. |