mirror of
https://github.com/facebook/sapling.git
synced 2024-10-10 08:47:12 +03:00
834be0af5c
This patch changes "==" (equals operator) to "===" (strict equals operator). The difference between them is that the latter doesn't do any type coercions. It's handy to compare string '1' to number 1 sometimes, but most of the time using "==" is inadvertent and can be replaced by an explicit type conversion. (This corresponds to "eqeqeq" option of jshint). Some of the changes in this patch are straightforward, e.g. when comparing results of typeof (they could only be strings). The same goes for 'none' and similar strings that can't be sensibly coerced to some other type. Two changes that compare values to "1" and "0" can be clarified: getAttribute() returns either a string or null, but comparing null to a string is always false, so no logic is lost. |
||
|---|---|---|
| .. | ||
| atom | ||
| coal | ||
| gitweb | ||
| json | ||
| monoblue | ||
| paper | ||
| raw | ||
| rss | ||
| spartan | ||
| static | ||
| map-cmdline.bisect | ||
| map-cmdline.changelog | ||
| map-cmdline.compact | ||
| map-cmdline.default | ||
| map-cmdline.phases | ||
| map-cmdline.show | ||
| map-cmdline.status | ||
| map-cmdline.xml | ||