opts: extension filters now always use comma separators

The root separator is no-longer used.

Author: CogentRedTester

docs/file_browser.conf

@@ -59,7 +59,7 @@ filter_files=yes
 
 # file-browser only shows files that are compatible with mpv by default
 # adding a file extension to this list will add it to the extension whitelist
-# extensions are separated with the root separators, do not use any spaces
+# extensions are separated with commas, do not use any spaces
 extension_whitelist=
 
 # add file extensions to this list to disable default filetypes

modules/options.lua

@@ -35,7 +35,7 @@ local o = {
     custom_keybinds_file = "~~/script-opts/file-browser-keybinds.json",
 
     --blacklist compatible files, it's recommended to use this rather than to edit the
-    --compatible list directly. A semicolon separated list of extensions without spaces
+    --compatible list directly. A comma separated list of extensions without spaces
     extension_blacklist = "",
 
     --add extra file extensions

modules/setup.lua

@@ -8,13 +8,13 @@ local fb = require 'modules.apis.fb'
 --sets up the compatible extensions list
 local function setup_extensions_list()
     --setting up subtitle extensions
-    for ext in fb_utils.iterate_opt(o.subtitle_extensions:lower()) do
+    for ext in fb_utils.iterate_opt(o.subtitle_extensions:lower(), ',') do
         g.sub_extensions[ext] = true
         g.extensions[ext] = true
     end
 
     --setting up audio extensions
-    for ext in fb_utils.iterate_opt(o.audio_extensions:lower()) do
+    for ext in fb_utils.iterate_opt(o.audio_extensions:lower(), ',') do
         g.audio_extensions[ext] = true
         g.extensions[ext] = true
     end
@@ -25,12 +25,12 @@ local function setup_extensions_list()
     end
 
     --adding extra extensions on the whitelist
-    for str in fb_utils.iterate_opt(o.extension_whitelist:lower()) do
+    for str in fb_utils.iterate_opt(o.extension_whitelist:lower(), ',') do
         g.extensions[str] = true
     end
 
     --removing extensions that are in the blacklist
-    for str in fb_utils.iterate_opt(o.extension_blacklist:lower()) do
+    for str in fb_utils.iterate_opt(o.extension_blacklist:lower(), ',') do
         g.extensions[str] = nil
     end
 end

modules/utils.lua

@@ -402,9 +402,10 @@ end
 
 ---Returns a string iterator that uses the root separators.
 ---@param str any
+---@param separators? string Override the root separators.
 ---@return fun():(string, ...)
-function fb_utils.iterate_opt(str)
-    return string.gmatch(str, "([^"..fb_utils.pattern_escape(o.root_separators).."]+)")
+function fb_utils.iterate_opt(str, separators)
+    return string.gmatch(str, "([^"..fb_utils.pattern_escape(separators or o.root_separators).."]+)")
 end
 
 ---Sorts a table into an array of selected items in the correct order.