Accepting request 1077716 from devel:languages:tcl

Automatic submission by obs-autosubmit

OBS-URL: https://build.opensuse.org/request/show/1077716
OBS-URL: https://build.opensuse.org/package/show/openSUSE:Factory/tcl?expand=0&rev=69
This commit is contained in:
Dominique Leuenberger 2023-04-08 15:38:18 +00:00 committed by Git OBS Bridge
commit d38a99d65f
2 changed files with 34 additions and 43 deletions

View File

@ -13,16 +13,7 @@
supported by the \fIcmdPrefix\fR. supported by the \fIcmdPrefix\fR.
--- generic/tclIORChan.c.orig --- generic/tclIORChan.c.orig
+++ generic/tclIORChan.c +++ generic/tclIORChan.c
@@ -425,7 +425,7 @@ static void UnmarshallErrorResult(Tcl_I @@ -532,7 +532,7 @@ TclChanCreateObjCmd(
*/
static int EncodeEventMask(Tcl_Interp *interp,
- const char *objName, Tcl_Obj *obj, int *mask);
+ const char *objName, Tcl_Obj *obj, int *mask, int needed);
static Tcl_Obj * DecodeEventMask(int mask);
static ReflectedChannel * NewReflectedChannel(Tcl_Interp *interp,
Tcl_Obj *cmdpfxObj, int mode, Tcl_Obj *handleObj);
@@ -532,11 +532,11 @@ TclChanCreateObjCmd(
/* /*
* First argument is a list of modes. Allowed entries are "read", "write". * First argument is a list of modes. Allowed entries are "read", "write".
@ -31,51 +22,46 @@
*/ */
modeObj = objv[MODE]; modeObj = objv[MODE];
- if (EncodeEventMask(interp, "mode", objv[MODE], &mode) != TCL_OK) { @@ -905,6 +905,11 @@ TclChanPostEventObjCmd(
+ if (EncodeEventMask(interp, "mode", objv[MODE], &mode, 0) != TCL_OK) { if (EncodeEventMask(interp, "event", objv[EVENT], &events) != TCL_OK) {
return TCL_ERROR; return TCL_ERROR;
} }
+ if (events == 0) {
+ Tcl_SetObjResult(interp,
+ Tcl_NewStringObj("bad event list: is empty", -1));
+ return TCL_ERROR;
+ }
@@ -902,7 +902,7 @@ TclChanPostEventObjCmd( /*
* "write". Expect at least one list element. Abbreviations are ok. * Check that the channel is actually interested in the provided events.
*/ @@ -2007,10 +2012,10 @@ ReflectGetOption(
- if (EncodeEventMask(interp, "event", objv[EVENT], &events) != TCL_OK) {
+ if (EncodeEventMask(interp, "event", objv[EVENT], &events, 1) != TCL_OK) {
return TCL_ERROR;
}
@@ -2007,8 +2007,9 @@ ReflectGetOption(
* EncodeEventMask -- * EncodeEventMask --
* *
* This function takes a list of event items and constructs the * This function takes a list of event items and constructs the
- * equivalent internal bitmask. The list must contain at least one - * equivalent internal bitmask. The list must contain at least one
- * element. Elements are "read", "write", or any unique abbreviation of - * element. Elements are "read", "write", or any unique abbreviation of
+ * equivalent internal bitmask. The list may be empty if the last - * them. Note that the bitmask is not changed if problems are
+ * argument is 0, otherwise it must contain at least one element. - * encountered.
+ * Elements are "read", "write", or any unique abbreviation of + * equivalent internal bitmask. The list may be empty but will usually
* them. Note that the bitmask is not changed if problems are + * contain at least one element. Valid elements are "read", "write", or
* encountered. + * any unique abbreviation of them. Note that the bitmask is not changed
+ * if problems are encountered.
* *
@@ -2028,7 +2029,8 @@ EncodeEventMask( * Results:
Tcl_Interp *interp, * A standard Tcl error code. A bitmask where TCL_READABLE and/or
const char *objName, @@ -2040,12 +2045,6 @@ EncodeEventMask(
Tcl_Obj *obj,
- int *mask)
+ int *mask,
+ int needed)
{
int events; /* Mask of events to post */
int listc; /* #elements in eventspec list */
@@ -2040,7 +2042,7 @@ EncodeEventMask(
return TCL_ERROR; return TCL_ERROR;
} }
- if (listc < 1) { - if (listc < 1) {
+ if (needed && listc < 1) { - Tcl_SetObjResult(interp, Tcl_ObjPrintf(
Tcl_SetObjResult(interp, Tcl_ObjPrintf( - "bad %s list: is empty", objName));
"bad %s list: is empty", objName)); - return TCL_ERROR;
return TCL_ERROR; - }
-
events = 0;
while (listc > 0) {
if (Tcl_GetIndexFromObj(interp, listv[listc-1], eventOptions,
--- tests/ioCmd.test.orig --- tests/ioCmd.test.orig
+++ tests/ioCmd.test +++ tests/ioCmd.test
@@ -670,12 +670,12 @@ test iocmd-21.1 {chan create, wrong#args @@ -670,12 +670,12 @@ test iocmd-21.1 {chan create, wrong#args
@ -86,7 +72,7 @@
- proc foo {} {} - proc foo {} {}
- catch {chan create {} foo} msg - catch {chan create {} foo} msg
+test iocmd-21.2 {chan create, r/w mode empty} { +test iocmd-21.2 {chan create, r/w mode empty} {
+ proc foo {cmd args} { return "initialize finalize watch" } + proc foo {cmd args} { return {initialize finalize watch} }
+ set chan [chan create {} foo] + set chan [chan create {} foo]
+ close $chan + close $chan
rename foo {} rename foo {}

View File

@ -1,3 +1,8 @@
-------------------------------------------------------------------
Thu Mar 30 09:30:57 UTC 2023 - Reinhard Max <max@suse.com>
- Update tcl-refchan-mode-needed.patch to the upstream version.
------------------------------------------------------------------- -------------------------------------------------------------------
Mon Feb 6 18:29:45 UTC 2023 - Reinhard Max <max@suse.com> Mon Feb 6 18:29:45 UTC 2023 - Reinhard Max <max@suse.com>