Skip to content

GeanyLua: Update to match latest Geany's Scintilla API - #1533

Open
Skif-off wants to merge 1 commit into
geany:masterfrom
Skif-off:glspi_sci
Open

GeanyLua: Update to match latest Geany's Scintilla API#1533
Skif-off wants to merge 1 commit into
geany:masterfrom
Skif-off:glspi_sci

Conversation

@Skif-off

Copy link
Copy Markdown
Contributor

No description provided.

@Skif-off

Copy link
Copy Markdown
Contributor Author

@xiota, could you look at these changes? I started to suspect that new types should be added to glspi_scintilla_param...
(I'm guessing they're numbers, but I'm not sure.)

@Skif-off
Skif-off force-pushed the glspi_sci branch 2 times, most recently from 9930f9d to 0e09d1a Compare January 24, 2026 13:50
@Skif-off

Copy link
Copy Markdown
Contributor Author

Updated.

@xiota

xiota commented Jan 24, 2026

Copy link
Copy Markdown
Contributor

I don't know what the new values are, but it's fine with me, since the header is autogenerated.

I wonder whether running the scripts to generate headers should be part of the build process.

@Skif-off

Copy link
Copy Markdown
Contributor Author

Some of them (SLT_*FULL) are just like the old ones, only bigger: 64-bit instead of 32-bit (well, I am not familiar with people who open files larger than 2 GB in Geany, but why not?).

I wonder whether running the scripts to generate headers should be part of the build process.

I'm not sure because there's no need to do this every time and such a case cannot be caught automatically, the user can only see with his eyes (?).
(But I could be wrong, I usually use ready-made files for build systems.)

@xiota

xiota commented Apr 8, 2026

Copy link
Copy Markdown
Contributor

@b4n I think this should be fine to merge, but I can't really test because my local builds are against stable geany. (I regenerate the symbols before build).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants