search

gnu-octave / statistics

The Statistics package for GNU Octave
GNU General Public License v3.0
24 stars 22 forks source link

files with CRLF line terminators #133

Closed dasergatskov closed 5 months ago

dasergatskov commented 5 months ago

As of 1.6.4 the following files are with CRLF line terminators:

binotest.m:                  ASCII text, with CRLF line terminators
fitcknn.m:                   ASCII text, with CRLF, LF line terminators
fitrgam.m:                   ASCII text, with CRLF, LF line terminators
friedman.m:                  ASCII text, with CRLF, LF line terminators
grpstats.m:                  ASCII text, with CRLF, LF line terminators
PKG_ADD:                     ASCII text, with CRLF line terminators

dist_fun/bvncdf.m:                              UTF-8 Unicode text, with CRLF, LF line terminators
Regression/RegressionGAM.m:                     ASCII text, with CRLF, LF line terminators
x86_64-pc-linux-gnu-api-v58/libsvmwrite.cc-tst: ASCII text, with CRLF, LF line terminators

dist_obj/private/__disp__.m:    ASCII text, with CRLF, LF line terminators
dist_obj/private/__paramci__.m: ASCII text, with CRLF, LF line terminators
dist_obj/private/__plot__.m:    ASCII text, with CRLF line terminators
dist_obj/private/__proflik__.m: ASCII text, with CRLF, LF line terminators
pr0m1th3as commented 5 months ago

Can you make a PR for this please?

dasergatskov commented 5 months ago

I am not sure what does it mean. Do you want a diff?

Dmitri.

On Mon, Mar 4, 2024 at 11:48 AM Andreas Bertsatos @.***> wrote:

Can you make a PR for this please?

— Reply to this email directly, view it on GitHub https://github.com/gnu-octave/statistics/issues/133#issuecomment-1977028808, or unsubscribe https://github.com/notifications/unsubscribe-auth/AT4NK7LGKU3WQCEAPSZATZ3YWSQW5AVCNFSM6AAAAABEFQTKY6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZXGAZDQOBQHA . You are receiving this because you authored the thread.Message ID: @.***>

pr0m1th3as commented 5 months ago

A diff would do also. BTW, I cannot see any CRLF lines in those files.

pr0m1th3as commented 5 months ago

Sorry for the comment above. @NRJank already made a PR, which I merged yesterday. Closing this issue.