Skip to content

Commit 670e273

Browse files
author
Carlos Une
committed
Use join_path() in list_files()
1 parent 1c6673c commit 670e273

File tree

1 file changed

+3
-16
lines changed

1 file changed

+3
-16
lines changed

src/fpm_filesystem.F90

Lines changed: 3 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -351,11 +351,10 @@ end subroutine mkdir
351351
!! - File/directory names return are relative to cwd, ie. preprended with `dir`
352352
!! - Includes files starting with `.` except current directory and parent directory
353353
!!
354-
recursive subroutine list_files(dir, files, recurse, separator)
354+
recursive subroutine list_files(dir, files, recurse)
355355
character(len=*), intent(in) :: dir
356356
type(string_t), allocatable, intent(out) :: files(:)
357357
logical, intent(in), optional :: recurse
358-
character(len=1), optional :: separator
359358

360359
integer :: i
361360
type(string_t), allocatable :: dir_files(:)
@@ -368,18 +367,6 @@ recursive subroutine list_files(dir, files, recurse, separator)
368367
integer, parameter :: N_MAX = 256
369368
type(string_t) :: files_tmp(N_MAX)
370369
integer(kind=c_int) :: r
371-
character(len=1) :: filesep
372-
373-
if (present(separator)) then
374-
filesep = separator
375-
else
376-
select case (get_os_type())
377-
case default
378-
filesep = '/'
379-
case (OS_WINDOWS)
380-
filesep = '\'
381-
end select
382-
end if
383370

384371
if (c_is_dir(dir(1:len_trim(dir))//c_null_char) .eq. 0) then
385372
allocate (files(0))
@@ -413,7 +400,7 @@ recursive subroutine list_files(dir, files, recurse, separator)
413400
i = 1
414401
end if
415402

416-
files_tmp(i)%s = dir // filesep // string_fortran
403+
files_tmp(i)%s = join_path(dir, string_fortran)
417404
end if
418405
end do
419406

@@ -435,7 +422,7 @@ recursive subroutine list_files(dir, files, recurse, separator)
435422

436423
do i=1,size(files)
437424
if (c_is_dir(files(i)%s//c_null_char) .ne. 0) then
438-
call list_files(files(i)%s, dir_files, recurse=.true., separator=filesep)
425+
call list_files(files(i)%s, dir_files, recurse=.true.)
439426
sub_dir_files = [sub_dir_files, dir_files]
440427
end if
441428
end do

0 commit comments

Comments
 (0)