Hello!
On latest master `clang` throws these warnings:
```
In file included from src/ViewStyle.cxx:26:
./src/Style.h:76:2: warning: explicitly defaulted move constructor is implicitly deleted [-Wdefaulted-function-deleted]
Style(Style &&) = default;
^
./src/Style.h:71:12: note: move constructor of 'Style' is implicitly deleted because field 'font' has a deleted move constructor
FontAlias font;
^
./src/Style.h:38:2: note: 'FontAlias' has been explicitly marked deleted here
FontAlias(FontAlias &&) = delete;
^
```
Thanks!
--
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/2891
Hello!
While chasing down these `clang` warnings I found that some code is redundant and can be safely removed:
```
main/parse.c:5007:26: warning: unused function 'FallbackParser' [-Wunused-function]
static parserDefinition *FallbackParser (void)
^
main/parse.c:5040:26: warning: unused function 'CTagsParser' [-Wunused-function]
static parserDefinition *CTagsParser (void)
^
main/parse.c:5355:26: warning: unused function 'CTagsSelfTestParser' [-Wunused-function]
static parserDefinition *CTagsSelfTestParser (void)
^
```
This is because we explicitly define `EXTERNAL_PARSER_LIST ` in tm_parsers.h at https://github.com/geany/geany/blob/master/src/tagmanager/tm_parsers.h#L16 so guess no need to check for not defined case?
Thanks!
You can view, comment on, or merge this pull request online at:
https://github.com/geany/geany/pull/2890
-- Commit Summary --
* Remove redundant code
-- File Changes --
M ctags/main/parse.c (338)
M ctags/main/parse_p.h (13)
-- Patch Links --
https://github.com/geany/geany/pull/2890.patchhttps://github.com/geany/geany/pull/2890.diff
--
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/pull/2890
The ide info window has a translation mismatch.
Where is it:
"Uma IDE rápido e leve"
Right:
"Uma IDE rápida e leve" or
"Um editor rápido e leve".
--
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/2871
It would be better to add to the search and replace dialog an option whether to wrap around the document (continue from the beginning when the end is reached and vice versa). Asking it each time is inconvenient and makes Geany unusable.
--
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/2887
I am currently running Geany 1.36 on Linux Mint.
It seems that the column size doesn't scale properly, or possibly at all, when you change your font size or font face.
I changed my editor font to Ubuntu Regular 14pt and found that the drawn line from Long line market, set at 72 columns, drew the line at 32 characters, give or take.
I am new to geany and bug reporting in general, so I apologize in advance if this is a duplicate, is expected behavior, or is otherwise an invalid report.
--
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/2886
Greetings,
AFAICS there’s currently no way to insert a marker through any
of the menu items; as a result the `Ctrl-M` shortcut is hardly
discoverable (though there _are_ menu items to skip to the
next/previous marker, which are very convenient).
I’d suggest adding that command to either in Edit>Insert submenu
or to the “Search” menu where (for some reason) other marker-related
commands already are to be found.
Additionally, it would nice to have it in the editor’s context menu.
--
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/2882