uzh / edf-converter

Converts EyeLink 1000 Edf files into a MATLAB structure - eye edf
MIT License
49 stars 25 forks source link

Max length for input filename on Windows #5

Closed iamamutt closed 1 year ago

iamamutt commented 5 years ago

If the input file name to Edf2Mat is 112 characters or more it crashes Matlab on Windows.

-----------------------------------------------------------------------------------------------------
MATLAB Version: 9.4.0.813654 (R2018a)
MATLAB License Number: 972711
Operating System: Microsoft Windows 10 Pro Version 10.0 (Build 17134)
Java Version: Java 1.8.0_144-b01 with Oracle Corporation Java HotSpot(TM) 64-Bit Server VM mixed mode
-----------------------------------------------------------------------------------------------------
MATLAB                                                Version 9.4         (R2018a)
Simulink                                              Version 9.1         (R2018a)
Control System Toolbox                                Version 10.4        (R2018a)
Curve Fitting Toolbox                                 Version 3.5.7       (R2018a)
DSP System Toolbox                                    Version 9.6         (R2018a)
Data Acquisition Toolbox                              Version 3.13        (R2018a)
Image Processing Toolbox                              Version 10.2        (R2018a)
Instrument Control Toolbox                            Version 3.13        (R2018a)
Optimization Toolbox                                  Version 8.1         (R2018a)
Signal Processing Toolbox                             Version 8.0         (R2018a)
Simulink Control Design                               Version 5.1         (R2018a)
Statistics and Machine Learning Toolbox               Version 11.3        (R2018a)
Symbolic Math Toolbox                                 Version 8.1         (R2018a)
Wavelet Toolbox                                       Version 5.0         (R2018a)
shiralupkin commented 4 years ago

This is also an issue on macs

mabiede commented 1 year ago

Thanks for reporting it back then. I added an input argument check for the filename (#9).