E704: Funcref variable name must start with a capital (during incpy#SetupOptions).

[arizvisa]

When running the plugin with an earlier version of Vim, the incpy#SetupOptions function will throw error E704 during initialization as a result of trying to assign a Funcref for a script-local function to the incpy#InputStrip variable. This manifests itself on Ubuntu 22.04 when trying to use the vim.basic which comes installed by default.

The version of Vim that complains about this is version 8.2. For the record, the most recent version of Vim as of today is 9.1.

VIM - Vi IMproved 8.2 (2019 Dec 12, compiled Oct 16 2024 17:27:14)
Included patches: 1-16, 647, 17-579, 1969, 580-647, 678, 648-1848, 4975, 5016, 5023, 5072, 2068, 1849-1854, 1857, 1855-1857, 1331, 1858, 1858-1859, 1873, 1860-1969, 1992, 1970-1992, 2010, 1993-2068, 2106, 2069-2106, 2108, 2107-2109, 2109-3995, 4563, 4646, 4774, 4895, 4899, 4901, 4919, 213, 1840, 1846-1847, 2110-2112, 2121

[arizvisa]

For the record, only the global variable namespace needs to be capitalized for these older versions. This'll require renaming the namespace from g:incpy#whatev to g:IncPy#whatev or perhaps g:Incpy#whatev.

[arizvisa]

It seems that you can also assign a Funcref to a dictionary without complaints from older versions of Vim.

[arizvisa]

I think that it's probably better to add support for multiple terminal windows that are managed based on the language that they're associated with. This way multiple ways of stripping the input and formatting the output can be configured for each interpreter type. Due to the lack of complaints when storing a Funcref to a dictionary, this'd result in fixing this issue and also allowing configuration of multiple programming languages.