@b4n commented on this pull request.
> static gint glspi_activate(lua_State* L)
{
- gint idx=-1;
- if (lua_gettop(L)>0) {
- if (lua_isnumber(L,1)) {
- idx=(lua_tonumber(L,1));
- if (idx<0) { /* Negative number refers to (absolute)
GtkNotebook index */
- idx=(0-idx)-1;
- if (idx>=gtk_notebook_get_n_pages(NOTEBOOK)) {
idx=-1;}
- } else { /* A positive number refers to the
geany->documents_array index */
- idx=doc_idx_to_tab_idx(idx-1);
+ gint lua_idx = -1;
+ gint tab_idx = -1;
+
+ if (lua_gettop(L) > 0) {
+ if (lua_isnumber(L, 1)) {
BTW, unrelated to this, but it seems an awful idea to allow a string
representation of a number, and then a string for the file name: this means
that if the filename is e.g. `"1.2"` it's not gonna work.
Swapping the integer/string checks would be slightly incompatible but probably
a future change worth considering.
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany-plugins/pull/1234#discussion_r2009936202
You are receiving this because you are subscribed to this thread.
Message ID: <geany/geany-plugins/pull/1234/review/[email protected]>
