On 2019.12.30 21:41, Pratyush Yadav wrote:
Hi Zoli,
On 30/12/19 03:56PM, Zoli Szabó via GitGitGadget wrote:
From: =?UTF-8?q?Zoli=20Szab=C3=B3?= <zoli.szabo@xxxxxxxxx>
Many times there's the need to quickly open a source file (the one you're
looking at in Git GUI) in the predefined text editor / IDE. Of course,
the file can be searched for in your preferred file manager or directly
in the text editor, but having the option to directly open the current
file from Git GUI would be just faster. This change enables just that by:
- clicking the diff header path (which is now highlighted as a hyperlink)
- or diff header path context menu -> Open;
Semi-colon left in by mistake?
Yes, that should have been a simple period (.), signaling the end of the
sentence.
+ eval exec $explorer [list [file nativename $full_file_path]] &
IIUC, this line can be simplified to:
exec $explorer [file nativename $full_file_path] &
It works fine for me including on files with spaces in their names, but
a test on Windows would be appreciated just to rule out any hidden
surprises.
You're right. It can be simplified like you have proposed. Tested it and
works correctly also on Windows. (Since I have copied that line from the
existing `do_explore` proc, the same simplification can be done there
too. (I did not realize this myself, since I am just learning some TCL
in order to have this feature in Git GUI...))
No need to send a re-roll just for these two small things. I have
updated the commit locally before pushing the new version out [0]. The
rest of the patch looks good. Will merge. Thanks.
[0] https://github.com/prati0100/git-gui/tree/zs/open-current-file
Thank you very much for your support!
All the best,
Zoli