Commits July 2015

commits@lists.geany.org

1 participants
73 discussions

[geany/geany] bb28bd: Fix spawn documentation so it appears in the API docs
by Colomban Wendling 12 Jul '15

12 Jul '15

Branch: refs/heads/master Author: Colomban Wendling <ban(a)herbesfolles.org> Committer: Colomban Wendling <ban(a)herbesfolles.org> Date: Sun, 12 Jul 2015 14:32:10 UTC Commit: bb28bdd1d39a58e8682a54077b1933ad72d3dbb0 https://github.com/geany/geany/commit/bb28bdd1d39a58e8682a54077b1933ad72d3d… Log Message: ----------- Fix spawn documentation so it appears in the API docs Modified Paths: -------------- doc/plugins.dox src/spawn.c Modified: doc/plugins.dox 1 lines changed, 1 insertions(+), 0 deletions(-) =================================================================== @@ -55,6 +55,7 @@ We will try to document as many functions and structs as possible. - @link msgwindow.h @endlink - @link project.h @endlink - @link sciwrappers.h Scintilla Wrapper Functions @endlink +- @link spawn.h Spawning programs @endlink - @link stash.h Stash Pref/Setting Functions @endlink - @link utils.h General Utility Functions @endlink - @link ui_utils.h Widget Utility Functions @endlink Modified: src/spawn.c 4 lines changed, 4 insertions(+), 0 deletions(-) =================================================================== @@ -39,6 +39,10 @@ * This module does not depend on Geany when compiled for testing (-DSPAWN_TEST). */ +/** @file spawn.h + * Portable and convenient process spawning and communication. + */ + #ifdef HAVE_CONFIG_H # include "config.h" #endif -------------- This E-Mail was brought to you by github_commit_mail.py (Source: https://github.com/geany/infrastructure).

1 0

[geany/geany] feca48: spawn: Move macros documentation so Doxygen can find them
by Colomban Wendling 12 Jul '15

12 Jul '15

Branch: refs/heads/master Author: Colomban Wendling <ban(a)herbesfolles.org> Committer: Colomban Wendling <ban(a)herbesfolles.org> Date: Sun, 12 Jul 2015 14:32:07 UTC Commit: feca48b59e838dcd6ca4a8d03697495163d73692 https://github.com/geany/geany/commit/feca48b59e838dcd6ca4a8d03697495163d73… Log Message: ----------- spawn: Move macros documentation so Doxygen can find them Modified Paths: -------------- src/spawn.h Modified: src/spawn.h 12 lines changed, 6 insertions(+), 6 deletions(-) =================================================================== @@ -25,15 +25,15 @@ #include <glib.h> #ifdef G_OS_WIN32 -# define SPAWN_WIFEXITED(status) TRUE /**< non-zero if the child exited normally */ -# define SPAWN_WEXITSTATUS(status) (status) /**< exit status of a child if exited normally */ -# define SPAWN_WIFSIGNALED(status) FALSE /**< non-zero if the child exited due to signal */ +# define SPAWN_WIFEXITED(status) TRUE +# define SPAWN_WEXITSTATUS(status) (status) +# define SPAWN_WIFSIGNALED(status) FALSE #else # include <sys/types.h> # include <sys/wait.h> -# define SPAWN_WIFEXITED(status) WIFEXITED(status) -# define SPAWN_WEXITSTATUS(status) WEXITSTATUS(status) -# define SPAWN_WIFSIGNALED(status) WIFSIGNALED(status) +# define SPAWN_WIFEXITED(status) WIFEXITED(status) /**< non-zero if the child exited normally */ +# define SPAWN_WEXITSTATUS(status) WEXITSTATUS(status) /**< exit status of a child if exited normally */ +# define SPAWN_WIFSIGNALED(status) WIFSIGNALED(status) /**< non-zero if the child exited due to signal */ #endif G_BEGIN_DECLS -------------- This E-Mail was brought to you by github_commit_mail.py (Source: https://github.com/geany/infrastructure).

1 0

[geany/geany] 7819cd: Make the search.c exit status checks platform independent
by Dimitar Zhekov 12 Jul '15

12 Jul '15

Branch: refs/heads/master Author: Dimitar Zhekov <dimitar.zhekov(a)gmail.com> Committer: Dimitar Zhekov <dimitar.zhekov(a)gmail.com> Date: Sun, 12 Jul 2015 14:12:20 UTC Commit: 7819cd11c7ac237eb08fbabc210a55eb95c9a0fb https://github.com/geany/geany/commit/7819cd11c7ac237eb08fbabc210a55eb95c9a… Log Message: ----------- Make the search.c exit status checks platform independent Modified Paths: -------------- src/search.c Modified: src/search.c 10 lines changed, 5 insertions(+), 5 deletions(-) =================================================================== @@ -1846,8 +1846,7 @@ static void search_read_io_stderr(GString *string, GIOCondition condition, gpoin static void search_finished(GPid child_pid, gint status, gpointer user_data) { const gchar *msg = _("Search failed."); -#ifdef G_OS_UNIX - gint exit_status = 1; + gint exit_status; if (SPAWN_WIFEXITED(status)) { @@ -1858,9 +1857,10 @@ static void search_finished(GPid child_pid, gint status, gpointer user_data) exit_status = -1; g_warning("Find in Files: The command failed unexpectedly (signal received)."); } -#else - gint exit_status = status; -#endif + else + { + exit_status = 1; + } switch (exit_status) { -------------- This E-Mail was brought to you by github_commit_mail.py (Source: https://github.com/geany/infrastructure).

1 0

[geany/geany] 2f237c: Prefix the WIF* macros with SPAWN_ and add short doc comments
by Dimitar Zhekov 12 Jul '15

12 Jul '15

Branch: refs/heads/master Author: Dimitar Zhekov <dimitar.zhekov(a)gmail.com> Committer: Dimitar Zhekov <dimitar.zhekov(a)gmail.com> Date: Sun, 12 Jul 2015 14:06:02 UTC Commit: 2f237c91a104a8b8260047f5a7de9a265278a150 https://github.com/geany/geany/commit/2f237c91a104a8b8260047f5a7de9a265278a… Log Message: ----------- Prefix the WIF* macros with SPAWN_ and add short doc comments Modified Paths: -------------- src/build.c src/search.c src/spawn.c src/spawn.h src/tools.c Modified: src/build.c 2 lines changed, 1 insertions(+), 1 deletions(-) =================================================================== @@ -1042,7 +1042,7 @@ static void show_build_result_message(gboolean failure) static void build_exit_cb(GPid child_pid, gint status, gpointer user_data) { - show_build_result_message(!WIFEXITED(status) || WEXITSTATUS(status) != EXIT_SUCCESS); + show_build_result_message(!SPAWN_WIFEXITED(status) || SPAWN_WEXITSTATUS(status) != EXIT_SUCCESS); utils_beep(); build_info.pid = 0; Modified: src/search.c 6 lines changed, 3 insertions(+), 3 deletions(-) =================================================================== @@ -1849,11 +1849,11 @@ static void search_finished(GPid child_pid, gint status, gpointer user_data) #ifdef G_OS_UNIX gint exit_status = 1; - if (WIFEXITED(status)) + if (SPAWN_WIFEXITED(status)) { - exit_status = WEXITSTATUS(status); + exit_status = SPAWN_WEXITSTATUS(status); } - else if (WIFSIGNALED(status)) + else if (SPAWN_WIFSIGNALED(status)) { exit_status = -1; g_warning("Find in Files: The command failed unexpectedly (signal received)."); Modified: src/spawn.c 4 lines changed, 2 insertions(+), 2 deletions(-) =================================================================== @@ -1164,8 +1164,8 @@ static void print_status(gint status) { fputs("finished, ", stderr); - if (WIFEXITED(status)) - fprintf(stderr, "exit code %d\n", WEXITSTATUS(status)); + if (SPAWN_WIFEXITED(status)) + fprintf(stderr, "exit code %d\n", SPAWN_WEXITSTATUS(status)); else fputs("abnormal termination\n", stderr); } Modified: src/spawn.h 9 lines changed, 6 insertions(+), 3 deletions(-) =================================================================== @@ -25,12 +25,15 @@ #include <glib.h> #ifdef G_OS_WIN32 -# define WIFEXITED(status) TRUE -# define WEXITSTATUS(status) (status) -# define WIFSIGNALED(status) FALSE +# define SPAWN_WIFEXITED(status) TRUE /**< non-zero if the child exited normally */ +# define SPAWN_WEXITSTATUS(status) (status) /**< exit status of a child if exited normally */ +# define SPAWN_WIFSIGNALED(status) FALSE /**< non-zero if the child exited due to signal */ #else # include <sys/types.h> # include <sys/wait.h> +# define SPAWN_WIFEXITED(status) WIFEXITED(status) +# define SPAWN_WEXITSTATUS(status) WEXITSTATUS(status) +# define SPAWN_WIFSIGNALED(status) WIFSIGNALED(status) #endif G_BEGIN_DECLS Modified: src/tools.c 2 lines changed, 1 insertions(+), 1 deletions(-) =================================================================== @@ -226,7 +226,7 @@ void tools_execute_custom_command(GeanyDocument *doc, const gchar *command) "Your selection was not changed. Error message: %s"), errors->str); } - else if (!WIFEXITED(status) || WEXITSTATUS(status) != EXIT_SUCCESS) + else if (!SPAWN_WIFEXITED(status) || SPAWN_WEXITSTATUS(status) != EXIT_SUCCESS) { /* TODO maybe include the exit code in the error message */ ui_set_statusbar(TRUE, -------------- This E-Mail was brought to you by github_commit_mail.py (Source: https://github.com/geany/infrastructure).

1 0

[geany/geany] 64fb9f: Don't return with an argument in a function returning void
by Colomban Wendling 12 Jul '15

12 Jul '15

Branch: refs/heads/master Author: Colomban Wendling <ban(a)herbesfolles.org> Committer: Colomban Wendling <ban(a)herbesfolles.org> Date: Sun, 12 Jul 2015 13:27:27 UTC Commit: 64fb9f9a51bc1a6d406bd902161e4dd4bde8cdd4 https://github.com/geany/geany/commit/64fb9f9a51bc1a6d406bd902161e4dd4bde8c… Log Message: ----------- Don't return with an argument in a function returning void Modified Paths: -------------- src/search.c Modified: src/search.c 4 lines changed, 2 insertions(+), 2 deletions(-) =================================================================== @@ -1833,13 +1833,13 @@ static void read_fif_io(gchar *msg, GIOCondition condition, gchar *enc, gint msg static void search_read_io(GString *string, GIOCondition condition, gpointer data) { - return read_fif_io(string->str, condition, data, COLOR_BLACK); + read_fif_io(string->str, condition, data, COLOR_BLACK); } static void search_read_io_stderr(GString *string, GIOCondition condition, gpointer data) { - return read_fif_io(string->str, condition, data, COLOR_DARK_RED); + read_fif_io(string->str, condition, data, COLOR_DARK_RED); } -------------- This E-Mail was brought to you by github_commit_mail.py (Source: https://github.com/geany/infrastructure).

1 0

[geany/geany] f11a2e: Merge pull request #563 from geany/elextr-news-addition
by Enrico Tröger 12 Jul '15

12 Jul '15

Branch: refs/heads/master Author: Enrico Tröger <enrico.troeger(a)uvena.de> Committer: Enrico Tröger <enrico.troeger(a)uvena.de> Date: Sun, 12 Jul 2015 12:51:12 UTC Commit: f11a2eb0139b39e4513394af62bcda3743d54cb3 https://github.com/geany/geany/commit/f11a2eb0139b39e4513394af62bcda3743d54… Log Message: ----------- Merge pull request #563 from geany/elextr-news-addition Update NEWS to mention windows command quoting Modified Paths: -------------- NEWS Modified: NEWS 2 lines changed, 2 insertions(+), 0 deletions(-) =================================================================== @@ -144,6 +144,8 @@ Geany 1.25 (unreleased) * Add scintilla_get_type(). Windows + * Use native Windows quoting rules for commands (on Windows, part of + subprocess spawning improvements). * Prompt before overwriting existing files when using native Save As dialog (PR#113, Adam Coyne). * View -> Change Font now respects the native dialog setting. -------------- This E-Mail was brought to you by github_commit_mail.py (Source: https://github.com/geany/infrastructure).

1 0

[geany/geany] b891c8: Update NEWS to mention windows command quoting
by elextr 12 Jul '15

12 Jul '15

Branch: refs/heads/elextr-news-addition Author: elextr <elextr(a)gmail.com> Committer: Enrico Tröger <enrico.troeger(a)uvena.de> Date: Sun, 12 Jul 2015 12:42:11 UTC Commit: b891c84582ea1932206fea074f96150947c4db40 https://github.com/geany/geany/commit/b891c84582ea1932206fea074f96150947c4d… Log Message: ----------- Update NEWS to mention windows command quoting Modified Paths: -------------- NEWS Modified: NEWS 2 lines changed, 2 insertions(+), 0 deletions(-) =================================================================== @@ -144,6 +144,8 @@ Geany 1.25 (unreleased) * Add scintilla_get_type(). Windows + * Use native Windows quoting rules for commands (on Windows, part of + subprocess spawning improvements). * Prompt before overwriting existing files when using native Save As dialog (PR#113, Adam Coyne). * View -> Change Font now respects the native dialog setting. -------------- This E-Mail was brought to you by github_commit_mail.py (Source: https://github.com/geany/infrastructure).

1 0

[geany/geany] b891c8: Update NEWS to mention windows command quoting
by elextr 12 Jul '15

12 Jul '15

Branch: refs/heads/master Author: elextr <elextr(a)gmail.com> Committer: Enrico Tröger <enrico.troeger(a)uvena.de> Date: Sun, 12 Jul 2015 12:42:11 UTC Commit: b891c84582ea1932206fea074f96150947c4db40 https://github.com/geany/geany/commit/b891c84582ea1932206fea074f96150947c4d… Log Message: ----------- Update NEWS to mention windows command quoting Modified Paths: -------------- NEWS Modified: NEWS 2 lines changed, 2 insertions(+), 0 deletions(-) =================================================================== @@ -144,6 +144,8 @@ Geany 1.25 (unreleased) * Add scintilla_get_type(). Windows + * Use native Windows quoting rules for commands (on Windows, part of + subprocess spawning improvements). * Prompt before overwriting existing files when using native Save As dialog (PR#113, Adam Coyne). * View -> Change Font now respects the native dialog setting. -------------- This E-Mail was brought to you by github_commit_mail.py (Source: https://github.com/geany/infrastructure).

1 0

[geany/geany] 5b341b: spawn: Fix a typo in Windows compatibility code
by Colomban Wendling 12 Jul '15

12 Jul '15

Branch: refs/heads/master Author: Colomban Wendling <ban(a)herbesfolles.org> Committer: Colomban Wendling <ban(a)herbesfolles.org> Date: Sun, 12 Jul 2015 12:23:08 UTC Commit: 5b341b12316ed0a23d8809a86df5c3f2459fb6eb https://github.com/geany/geany/commit/5b341b12316ed0a23d8809a86df5c3f2459fb… Log Message: ----------- spawn: Fix a typo in Windows compatibility code Thanks to Dimitar for spotting this. Modified Paths: -------------- src/spawn.h Modified: src/spawn.h 2 lines changed, 1 insertions(+), 1 deletions(-) =================================================================== @@ -27,7 +27,7 @@ #ifdef G_OS_WIN32 # define WIFEXITED(status) TRUE # define WEXITSTATUS(status) (status) -# define WIFSIGNALLED(status) FALSE +# define WIFSIGNALED(status) FALSE #else # include <sys/types.h> # include <sys/wait.h> -------------- This E-Mail was brought to you by github_commit_mail.py (Source: https://github.com/geany/infrastructure).

1 0

[geany/geany] 01b01c: Merge pull request #561 from b4n/filebrowser-use-spawn
by Colomban Wendling 12 Jul '15

12 Jul '15

Branch: refs/heads/master Author: Colomban Wendling <ban(a)herbesfolles.org> Committer: Colomban Wendling <ban(a)herbesfolles.org> Date: Sun, 12 Jul 2015 12:15:30 UTC Commit: 01b01cc83a9c15f41b509072aff476184e2a6a4e https://github.com/geany/geany/commit/01b01cc83a9c15f41b509072aff476184e2a6… Log Message: ----------- Merge pull request #561 from b4n/filebrowser-use-spawn filebrowser: Use Spawn module to launch the external command Modified Paths: -------------- plugins/filebrowser.c Modified: plugins/filebrowser.c 2 lines changed, 1 insertions(+), 1 deletions(-) =================================================================== @@ -477,7 +477,7 @@ static void open_external(const gchar *fname, gboolean dir_found) cmd = g_string_free(cmd_str, FALSE); locale_cmd = utils_get_locale_from_utf8(cmd); - if (! g_spawn_command_line_async(locale_cmd, &error)) + if (! spawn_async(NULL, locale_cmd, NULL, NULL, NULL, &error)) { gchar *c = strchr(cmd, ' '); -------------- This E-Mail was brought to you by github_commit_mail.py (Source: https://github.com/geany/infrastructure).

1 0

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

Commits July 2015