This PR modifies `replace_all` to avoid the infinite loop. In each iteration, `replace_all` searches for `needle` starting from the beginning of `haystack`. If `replacement` contains `needle`, the result is an infinite loop. To prevent this from happening, `replace_all` should continue searching for `needle` from the end of the previous `replacement`.
Squashed and rebased from #1128. Resolves #936.
You can view, comment on, or merge this pull request online at:
https://github.com/geany/geany-plugins/pull/1232
-- Commit Summary --
* Markdown: Modify replace_all to avoid infinite loop
-- File Changes --
M markdown/src/viewer.c (7)
-- Patch Links --
https://github.com/geany/geany-plugins/pull/1232.patchhttps://github.com/geany/geany-plugins/pull/1232.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany-plugins/pull/1232
You are receiving this because you are subscribed to this thread.
Message ID: <geany/geany-plugins/pull/1232(a)github.com>
Hi, this is happening in Ubuntu 20.04, compiled version of geany 1.37.1. I'm a long time user of Geany, and I've always compiled from source. Compilation was fine, as usual.
╭─nalonso@satellite ~
╰─$ geany
[1] 1884036 segmentation fault (core dumped) geany
╭─nalonso@satellite ~
╰─$
If I open documents everything works fine.
Maybe unrelated, but Geany is taking a whole lot more time to load with a lot of documents. I use to have 30 or more files opened, mostly YAML and Dockerfiles, but up to 1.36 Geany was lightning fast even with all those documents open.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/2736
Everything is easy on Linux, but I'm using Windows. I tried `gtk-3.0/settings.ini` but it doesn't work.
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/discussions/3856
You are receiving this because you are subscribed to this thread.
Message ID: <geany/geany/repo-discussions/3856(a)github.com>
Please could **Geany** highlight the **var** keyword in Java?
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/3854
You are receiving this because you are subscribed to this thread.
Message ID: <geany/geany/issues/3854(a)github.com>
I'm using geany 2.0-1 on Arch Linux with XFCE.
Since the newest update, I can not use paste via middle-click repeatedly anymore. This was a frequent use case for me: select some text and then paste it at a few places. It does not matter if the selection was made inside or outside geany.
What makes me think geany clears the X selection:
* I can select and paste into a different window multiple times, but after pasting into geany, I can't anymore.
* Also, the clipboard manager plugin for the XFCE panel shows no mouse selection anymore afterwards.
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/3658
You are receiving this because you are subscribed to this thread.
Message ID: <geany/geany/issues/3658(a)github.com>
The real name of the struct in Lua 5.1 is luaL_Reg. Lua 5.1 contains a define for compatibility, but it is not present in LuaJIT or later 5.x releases.
You can view, comment on, or merge this pull request online at:
https://github.com/geany/geany-plugins/pull/1231
-- Commit Summary --
* GeanyLua: Use luaL_Reg instead of luaL_reg
-- File Changes --
M geanylua/glspi_app.c (2)
M geanylua/glspi_dlg.c (2)
M geanylua/glspi_doc.c (2)
M geanylua/glspi_init.c (2)
M geanylua/glspi_kfile.c (2)
M geanylua/glspi_run.c (2)
M geanylua/glspi_sci.c (2)
M geanylua/gsdlg_lua.c (2)
-- Patch Links --
https://github.com/geany/geany-plugins/pull/1231.patchhttps://github.com/geany/geany-plugins/pull/1231.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany-plugins/pull/1231
You are receiving this because you are subscribed to this thread.
Message ID: <geany/geany-plugins/pull/1231(a)github.com>
While working on [reproducible builds](https://reproducible-builds.org/) for [openSUSE](https://en.opensuse.org/openSUSE:Reproducible_Builds), I found that our `glfw` package varies in every build since the upgrade to geany-2.0
I extracted this minimal reproducer:
```bash
cat > test2.c <<EOF
typedef struct {
struct
{
} __value32;
} __atomic_wide_counter;
EOF
for i in 1 2 ; do geany -g glfw.c.tags test2.c ; md5sum glfw.c.tags ; done | sort -u | wc -l
2
```
The output differs thusly:
```diff
-__value32�64�__atomic_wide_counter�0�__anonceed23430108::anon_struct_1
+__value32�64�__atomic_wide_counter�0�__anone35807a60108::anon_struct_1
```
I think, there is the same problem with anon unions.
Please make output deterministic.
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/3717
You are receiving this because you are subscribed to this thread.
Message ID: <geany/geany/issues/3717(a)github.com>
In addition to scope, anonymous type can appear also inside var_type which should be renamed as well.
Fixes #3717.
You can view, comment on, or merge this pull request online at:
https://github.com/geany/geany/pull/3785
-- Commit Summary --
* Rename both scope and var_type of anonymous types
-- File Changes --
M src/tagmanager/tm_ctags.c (40)
M tests/ctags/Makefile.am (1)
A tests/ctags/nested_anon.c (5)
A tests/ctags/nested_anon.c.tags (6)
M tests/meson.build (1)
-- Patch Links --
https://github.com/geany/geany/pull/3785.patchhttps://github.com/geany/geany/pull/3785.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3785
You are receiving this because you are subscribed to this thread.
Message ID: <geany/geany/pull/3785(a)github.com>