Revision 3891a84ccdad249c426c3e036d4fb086c388b4ce authored by Pratyush Yadav on 14 March 2020, 21:38:36 UTC, committed by Pratyush Yadav on 17 March 2020, 13:18:54 UTC
Evaluating the script in the same namespace as the chord itself creates potential for variable name collision. And in that case the script would unknowingly use the chord's variables. For example, say the script has a variable called 'is_completed', which also exists in the chord's namespace. The script then calls 'eval' and sets 'is_completed' to 1 thinking it is setting its own variable, completely unaware of how the chord works behind the scenes. This leads to the chord never actually executing because it sees 'is_completed' as true and thinks it has already completed. Avoid the potential collision by creating a separate namespace for the script that is a child of the chord's namespace. Signed-off-by: Pratyush Yadav <me@yadavpratyush.com>
1 parent 8a8efbe
File | Mode | Size |
---|---|---|
AppMain.tcl | -rw-r--r-- | 706 bytes |
Info.plist | -rw-r--r-- | 985 bytes |
git-gui.icns | -rw-r--r-- | 28.2 KB |
![swh spinner](/static/img/swh-spinner.gif)
Computing file changes ...