jyapayne/swig
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Reenable and fix octave horzcat operator test

Browse source 
This test was disabled with 0a0743f25c since
it fails with octave 7.2.0

The test function horzcat now uses a variable length argument list to fix this.
Before octave 7 it seems to be possible to call a N-argument octave function
with > N arguments without any error. With octave 7 this seems no longer to be
possible which caused the test failure.
This commit is contained in:
Markus Friedrich 2022-09-18 19:40:37 +02:00
commit 720c28f1ed
2 changed files with 8 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

9
Examples/octave/operator/@swig_ref/horzcat.m
View file

@ -1,6 +1,9 @@
% test octaves concatenation operator
function ret=horzcat(a, b)
% return the concatenation of two ComplexVal values as a cell array.
function ret=horzcat(varargin)
% return the concatenation of several ComplexVal values as a cell array.
% (not really useful but it tests the concatenation of swig_ref objects)
ret={a, b};
ret={};
for i=1:length(varargin)
ret{i}=varargin{i};
end
end

5
Examples/octave/operator/runme.m
View file

@ -47,6 +47,5 @@ if swig_octave_prereq(3,8,0)
endif
# concatenation operator, note: calls @swig_ref/horzcat.m
# g = [a, b, c];
# printf("g = %s\n",disp(g));
# Above temporarily removed as broken in octave-7.2.0, see https://github.com/swig/swig/issues/2353
g = [a, b, c];
printf("g = %s\n",disp(g));