[Bf-committers] presets.py as_filename bug?

[Domino Marama]

Just spotted this browsing the .py files..

In ./release/scripts/op/presets.py starting at line 35

    def _as_filename(self, name): # could reuse for other presets
        for char in " !@#$%^&*(){}:\";'[]<>,./?":
            name = name.replace('.', '_')
        return name.lower()

shouldn't it be

    def _as_filename(self, name): # could reuse for other presets
        for char in " !@#$%^&*(){}:\";'[]<>,./?":
            name = name.replace(char, '_')
        return name.lower()

so all the funky characters are replaced and not just '.'?