[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Octave-patch-tracker] [patch #7934] Adding tests for operator overloadi
[Octave-patch-tracker] [patch #7934] Adding tests for operator overloading in the legacy class system (bug #38128)
Mon, 28 Jan 2013 17:28:23 +0000
Mozilla/5.0 (X11; Ubuntu; Linux i686; rv:18.0) Gecko/20100101 Firefox/18.0
Follow-up Comment #2, patch #7934 (project octave):
Concerning the first point: yes, I did that because Matlab doesn't have a
two-argument version of assert. It was explicitely stated just above the two
lines that you quote:
%% Note: This script and all classes are also intended to run
%% in MATLAB to test compatibility. Don't break that!
Also, I think that assert(isequal()) is clearer than the two-argument syntax,
but that's obviously a matter of taste (since you call it "ugly").
For the second point: I have no idea which branch it belongs to, please put it
wherever you think it fits.
Reply to this item at:
Message posté via/par Savannah