blackbeard420/weechat
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

perl: fix value returned in case of error in some functions

Browse Source 
Affected functions:
- config_option_reset
- config_color
- config_color_default
- config_write
- config_read
- config_reload
- buffer_string_replace_local_var
- command
This commit is contained in:
Sébastien Helleu 2015-03-01 10:10:36 +01:00
parent 7624ef417d
commit e6d3a78d2c
2 changed files with 19 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
ChangeLog.asciidoc
View File

@ -36,6 +36,9 @@ https://weechat.org/files/releasenotes/ReleaseNotes-devel.html[release notes]
* relay: fix up/down keys on relay buffer (closes #335) * relay: fix up/down keys on relay buffer (closes #335)
* relay: remove v4-mapped addresses in /help relay.network.allowed_ips * relay: remove v4-mapped addresses in /help relay.network.allowed_ips
(closes #325) (closes #325)
* perl: fix value returned in case of error in functions:
config_option_reset, config_color, config_color_default, config_write,
config_read, config_reload, buffer_string_replace_local_var, command
* python: fix value returned in case of error in functions: * python: fix value returned in case of error in functions:
config_option_reset, config_color, config_color_default, config_write, config_option_reset, config_color, config_color_default, config_write,
config_read, config_reload, config_is_set_plugin, buffer_get_string, config_read, config_reload, config_is_set_plugin, buffer_get_string,

32
src/plugins/perl/weechat-perl-api.c
View File

@ -1256,9 +1256,9 @@ API_FUNC(config_option_reset)
char *option; char *option;
dXSARGS; dXSARGS;
API_INIT_FUNC(1, "config_option_reset", API_RETURN_INT(0)); API_INIT_FUNC(1, "config_option_reset", API_RETURN_INT(WEECHAT_CONFIG_OPTION_SET_ERROR));
if (items < 2) if (items < 2)
API_WRONG_ARGS(API_RETURN_INT(0)); API_WRONG_ARGS(API_RETURN_INT(WEECHAT_CONFIG_OPTION_SET_ERROR));
option = SvPV_nolen (ST (0)); option = SvPV_nolen (ST (0));
@ -1458,9 +1458,9 @@ API_FUNC(config_color)
const char *result; const char *result;
dXSARGS; dXSARGS;
API_INIT_FUNC(1, "config_color", API_RETURN_INT(0)); API_INIT_FUNC(1, "config_color", API_RETURN_EMPTY);
if (items < 1) if (items < 1)
API_WRONG_ARGS(API_RETURN_INT(0)); API_WRONG_ARGS(API_RETURN_EMPTY);
result = weechat_config_color (API_STR2PTR(SvPV_nolen (ST (0)))); /* option */ result = weechat_config_color (API_STR2PTR(SvPV_nolen (ST (0)))); /* option */
@ -1472,9 +1472,9 @@ API_FUNC(config_color_default)
const char *result; const char *result;
dXSARGS; dXSARGS;
API_INIT_FUNC(1, "config_color_default", API_RETURN_INT(0)); API_INIT_FUNC(1, "config_color_default", API_RETURN_EMPTY);
if (items < 1) if (items < 1)
API_WRONG_ARGS(API_RETURN_INT(0)); API_WRONG_ARGS(API_RETURN_EMPTY);
result = weechat_config_color_default (API_STR2PTR(SvPV_nolen (ST (0)))); /* option */ result = weechat_config_color_default (API_STR2PTR(SvPV_nolen (ST (0)))); /* option */
@ -1523,9 +1523,9 @@ API_FUNC(config_write)
int rc; int rc;
dXSARGS; dXSARGS;
API_INIT_FUNC(1, "config_write", API_RETURN_INT(-1)); API_INIT_FUNC(1, "config_write", API_RETURN_INT(WEECHAT_CONFIG_WRITE_ERROR));
if (items < 1) if (items < 1)
API_WRONG_ARGS(API_RETURN_INT(-1)); API_WRONG_ARGS(API_RETURN_INT(WEECHAT_CONFIG_WRITE_ERROR));
rc = weechat_config_write (API_STR2PTR(SvPV_nolen (ST (0)))); /* config_file */ rc = weechat_config_write (API_STR2PTR(SvPV_nolen (ST (0)))); /* config_file */
@ -1537,9 +1537,9 @@ API_FUNC(config_read)
int rc; int rc;
dXSARGS; dXSARGS;
API_INIT_FUNC(1, "config_read", API_RETURN_INT(-1)); API_INIT_FUNC(1, "config_read", API_RETURN_INT(WEECHAT_CONFIG_READ_FILE_NOT_FOUND));
if (items < 1) if (items < 1)
API_WRONG_ARGS(API_RETURN_INT(-1)); API_WRONG_ARGS(API_RETURN_INT(WEECHAT_CONFIG_READ_FILE_NOT_FOUND));
rc = weechat_config_read (API_STR2PTR(SvPV_nolen (ST (0)))); /* config_file */ rc = weechat_config_read (API_STR2PTR(SvPV_nolen (ST (0)))); /* config_file */
@ -1551,9 +1551,9 @@ API_FUNC(config_reload)
int rc; int rc;
dXSARGS; dXSARGS;
API_INIT_FUNC(1, "config_reload", API_RETURN_INT(-1)); API_INIT_FUNC(1, "config_reload", API_RETURN_INT(WEECHAT_CONFIG_READ_FILE_NOT_FOUND));
if (items < 1) if (items < 1)
API_WRONG_ARGS(API_RETURN_INT(-1)); API_WRONG_ARGS(API_RETURN_INT(WEECHAT_CONFIG_READ_FILE_NOT_FOUND));
rc = weechat_config_reload (API_STR2PTR(SvPV_nolen (ST (0)))); /* config_file */ rc = weechat_config_reload (API_STR2PTR(SvPV_nolen (ST (0)))); /* config_file */
@ -3354,9 +3354,9 @@ API_FUNC(buffer_string_replace_local_var)
char *buffer, *string, *result; char *buffer, *string, *result;
dXSARGS; dXSARGS;
API_INIT_FUNC(1, "buffer_string_replace_local_var", API_RETURN_ERROR); API_INIT_FUNC(1, "buffer_string_replace_local_var", API_RETURN_EMPTY);
if (items < 2) if (items < 2)
API_WRONG_ARGS(API_RETURN_ERROR); API_WRONG_ARGS(API_RETURN_EMPTY);
buffer = SvPV_nolen (ST (0)); buffer = SvPV_nolen (ST (0));
string = SvPV_nolen (ST (1)); string = SvPV_nolen (ST (1));
@ -4024,9 +4024,9 @@ API_FUNC(command)
int rc; int rc;
dXSARGS; dXSARGS;
API_INIT_FUNC(1, "command", API_RETURN_ERROR); API_INIT_FUNC(1, "command", API_RETURN_INT(WEECHAT_RC_ERROR));
if (items < 2) if (items < 2)
API_WRONG_ARGS(API_RETURN_ERROR); API_WRONG_ARGS(API_RETURN_INT(WEECHAT_RC_ERROR));
buffer = SvPV_nolen (ST (0)); buffer = SvPV_nolen (ST (0));
command = SvPV_nolen (ST (1)); command = SvPV_nolen (ST (1));