corelib.parseNameValueArguments doesn't work on new versions of matlab

[alec-hoyland]

On new versions of matlab, passing varargin as an argument to a function instantiates a "new instance" of varargin...for example

myfunc(varargin)

would create an n x 1 cell array variable named varargin in the local scope of the function, but calling corelib.parseNameValueArguments would create a 1x1 cell array that itself contains the n x 1 varargin cell array. This breaks the function because corelib.parseNameValueArguments throws an error if you provide it one input and it's not a struct. The simple fix is to just rename the variable in the function signature so that matlab doesn't interpret it as another instance of varargin specific to corelib.parseNameValueArguments.

[sg-s]

this should be removed, since namedargs2cell does the job. remove from:

• +axlib/label.m:
• +axlib/makeEphys.m:
• +axlib/separate.m:
• +clusterlib/densityPeaks.m:
• +corelib/parseNameValueArguments.m:
• +figlib/label.m:
• +figlib/pretty.m:
• +figlib/saveall.m:
• +figlib/shrink.m:
• +imglib/triangle.m:
• +neurolib/raster.m:
• +neurolib/spiketrain.m:
• +neurolib/staggerSpikes.m:
• +plotlib/cplot.m:
• +plotlib/pairwise.m:
• +plotlib/pieceWiseLinear.m:
• +plotlib/raincloud.m:
• +plotlib/scatterhist.m:
• +plotlib/trajectory.m:
• +veclib/closeReturn.m:
• deprecated/abf2kontroller.m:
• deprecated/animatePhasePlot.m: