For a verilog file, the variables list parses the old (1995) style verilog module declarations. Verilog 2001 enhanced the syntax and seems to confuse Geany....
1995 syntax:
module (foo, bar, buz);
input foo; // foo port
output bar; // bar port
output buz; // buz port
reg buz; // buz variable
2001 syntax:
module (
input wire foo, // foo port is a wire
output wire bar, // bar port is a wire
output reg buz // buz port is a variable
);
// note: the "wire" is optional and the ports could have been declared with or without
Geany variables list seems to get confused by the 2001 style. It seems to parse the wire keyword as the variable name if I keep the optional "wire" keyword:
If I omit the "wire" keyword, Geany still gets confused, but in a different way. It seems to parse every other variable and then parse the "input" or "output" keywords as variables in some cases:
If someone who knows the code base can even point me to the module(s) doing the parsing for the variables, I may be able to help create the fix/enhancement for this. I'd be happy to try anyway. I am not familiar with the Geany code base so just getting started seems like an insurmountable task.
---
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/670
I am looking for some kind of "Console View" like it is available for example in Eclipse.
Currently a terminal / shell opens when executing a script in Geany. In this shell one can see the process streams stdout and stderr and can write to stdin of the new process.
However, I would rather eliminate this new terminal window popping up and changing focus each time I launch a script in Geany.
I think it is better to have the process communication (stdout, stdin, stderr) embedded in a view within Geany like it is done in many IDEs. The different streams could be colored e.g. stderr in red and stdin (input from the user) in green.
---
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/675
In the Build Commands dialog there are additional commands under `Conf commands`, `Independent commands` and `Execute commands` that are not available to setup in Preferences|Keybindings. It would be very useful to be able to set keybindings for these. Further to this the command names in Build commands can be modified but this is not reflected in Keybindings preferences.
Just a quick thought too; a button linking to the keybindings prefecences would be a helpful addition to the Build commands dialog.
---
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/705
I was using [`git-new-workdir`](http://nuclearsquid.com/writings/git-new-workdir/) and recently switched to using the newer form [`git worktree`](https://github.com/blog/2042-git-2-5-including-multiple-worktree… however found this stops the plugin working.
The reason appears to be that instead of a `.git` directory in the 'worktee directory' there is a `.git` file that contains a reference to the original directory. e.g. `gitdir: /home/ubuntu/origdir.git/.git/worktrees/newdir.git`.
---
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany-plugins/issues/285
Bash allows more characters besides `[[:alnum:]_]` when declaring
function names using the `function` keyword. It also does not require
having a pair of parentheses after the name. Some shells may actually
implement it differently but we don't have to be that strict since the
user explicitly specifies the `function` keyword anyway.
This update implements the ones described above, and also invalidates
function names that are completely made up of digits.
You can view, comment on, or merge this pull request online at:
https://github.com/geany/geany/pull/662
-- Commit Summary --
* Enhance detection of sh functions
-- File Changes --
M tagmanager/ctags/sh.c (125)
-- Patch Links --
https://github.com/geany/geany/pull/662.patchhttps://github.com/geany/geany/pull/662.diff
---
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/662
For example `10.5`. Some other editors accept that value.
Thanks :-)
---
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/703
Hi :smiley:
I've added a keybinding for switching focus to the split window editor (in case it's enabled). This solves #631
I've set F3 as keybinding by default because:
1. Follows the idea of the other "Focus Keybindings" family: F2 -> Switch to Editor, F4 -> Terminal, F7 -> Search bar
2. Doesn't collides with any other default keybindings and has very few possibilities of collision with user-defined keybindings.
3. Even in the case it collides with other user-defined keybinding, it's not a big deal since it just grab the focus on split window only in the case it's being displayed. If user doesn't like it, he/she can always change this.
4. It's a plugin shipped along with geany.
You can view, comment on, or merge this pull request online at:
https://github.com/geany/geany/pull/721
-- Commit Summary --
* Keybinding for switching focus to splitwindow. Fix #631
-- File Changes --
M plugins/splitwindow.c (16)
-- Patch Links --
https://github.com/geany/geany/pull/721.patchhttps://github.com/geany/geany/pull/721.diff
---
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/721
This is an improved version of PR #637.
No more "--" - not needed.
No guard for is_osx_bundle() under Windows - does not belong here.
Split into proper sequential commits.
Fixed the encoding of options and doc_name.
Fixed the OSX bundle executable name.
Created a function to return the reproduced arguments instead of exposing global variables.
Re-tested under Windows and Linux.
There is only one problem left. As described in PR #637, under Windows, mscvrt often breaks unquoted locale strings on 2+ pieces, considering some characters "spaces". So spawn should quote any argv elements containing locale. But that must obviously be a separate PR.
You can view, comment on, or merge this pull request online at:
https://github.com/geany/geany/pull/653
-- Commit Summary --
* Add reverse option parser to main
* Add support for geany executable to utils_resource_dir()
* Improve New Window: pass the proper arguments and save configuration
-- File Changes --
M src/libmain.c (121)
M src/main.h (2)
M src/plugins.c (2)
M src/utils.c (111)
M src/utils.h (17)
-- Patch Links --
https://github.com/geany/geany/pull/653.patchhttps://github.com/geany/geany/pull/653.diff
---
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/653
'Geany' no longer detects if there is an instance of itself already running, thus multiple instances are starting up, instead of files being opened in the current instance.
I doubt it is a "#ifdef HAVE_SOCKET" not being defined since the mutexs (called mutants in win32) are being created with the name 'Geany'.
I suspect it had to do with recent cleanups and changes in https://github.com/geany/geany/commits/c5b0fc459a1251c824a44edb5dfab4a8feb0…
---
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/641
){kind=link}