in reply to Backdating strict
If you revisit the code for maintenance there is some argument for using that as an opportunity to roll in strictures. But it's a bang for buck thing. With modern editors that perform background syntax checking it's pretty quick to identify where variables need to be declared and fix that. Because declaration gives a defined scope for variables it can also be a good way to think about how the code works. So the process of declaring variables can actually make the maintenance task easier.
If you have no other need to visit the code then generally it's best to leave it as is. Another reason to visit the code may be occasional flakiness. In that case adding strictures, especially warnings, is very compelling.
|
|---|
| Replies are listed 'Best First'. | |
|---|---|
|
Re^2: Backdating strict
by Bod (Parson) on Nov 17, 2020 at 21:03 UTC | |
by GrandFather (Saint) on Nov 17, 2020 at 21:22 UTC | |
by marto (Cardinal) on Nov 17, 2020 at 21:06 UTC | |
by Bod (Parson) on Nov 17, 2020 at 21:15 UTC | |
by jcb (Parson) on Nov 18, 2020 at 04:04 UTC | |
by LanX (Saint) on Nov 18, 2020 at 21:37 UTC | |
by GrandFather (Saint) on Nov 18, 2020 at 22:22 UTC | |
by Fletch (Bishop) on Nov 18, 2020 at 22:01 UTC | |
by LanX (Saint) on Nov 18, 2020 at 22:42 UTC | |
by jcb (Parson) on Nov 19, 2020 at 00:21 UTC | |
by LanX (Saint) on Nov 19, 2020 at 00:28 UTC | |
| |
by Bod (Parson) on Dec 06, 2020 at 21:48 UTC | |
by stevieb (Canon) on Nov 21, 2020 at 02:20 UTC | |
by Bod (Parson) on Nov 21, 2020 at 11:14 UTC |