{"action":"created","issue":{"url":"https://api.github.com/repos/sampsyo/beets/issues/496","labels_url":"https://api.github.com/repos/sampsyo/beets/issues/496/labels{/name}","comments_url":"https://api.github.com/repos/sampsyo/beets/issues/496/comments","events_url":"https://api.github.com/repos/sampsyo/beets/issues/496/events","html_url":"https://github.com/sampsyo/beets/issues/496","id":25776100,"number":496,"title":"Trailing whitespace not removed after max_filename_length truncation","user":{"login":"LordSputnik","id":1898174,"avatar_url":"https://avatars.githubusercontent.com/u/1898174?v=3","gravatar_id":"","url":"https://api.github.com/users/LordSputnik","html_url":"https://github.com/LordSputnik","followers_url":"https://api.github.com/users/LordSputnik/followers","following_url":"https://api.github.com/users/LordSputnik/following{/other_user}","gists_url":"https://api.github.com/users/LordSputnik/gists{/gist_id}","starred_url":"https://api.github.com/users/LordSputnik/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/LordSputnik/subscriptions","organizations_url":"https://api.github.com/users/LordSputnik/orgs","repos_url":"https://api.github.com/users/LordSputnik/repos","events_url":"https://api.github.com/users/LordSputnik/events{/privacy}","received_events_url":"https://api.github.com/users/LordSputnik/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/sampsyo/beets/labels/bug","name":"bug","color":"e102d8"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":12,"created_at":"2014-01-16T23:35:26Z","updated_at":"2015-02-24T03:44:54Z","closed_at":null,"body":"Just been updating all my path names to try to fit them onto an NTFS drive legally. To do this, I set \"max_filename_length: 80\". However, on a few folders, this seems to have set the folder name so that it ends with a space.\r\n\r\nSince I haven't set a custom \"replace: ...\" parameter in my configuration, I believe that beets should be using the default character replacement set, which includes a regex for removing trailing spaces (\"\\s+$\").\r\n\r\nTherefore, I think that the character replacement isn't being applied to the path name again after it's been truncated."},"comment":{"url":"https://api.github.com/repos/sampsyo/beets/issues/comments/75694651","html_url":"https://github.com/sampsyo/beets/issues/496#issuecomment-75694651","issue_url":"https://api.github.com/repos/sampsyo/beets/issues/496","id":75694651,"user":{"login":"wootencl","id":8561687,"avatar_url":"https://avatars.githubusercontent.com/u/8561687?v=3","gravatar_id":"","url":"https://api.github.com/users/wootencl","html_url":"https://github.com/wootencl","followers_url":"https://api.github.com/users/wootencl/followers","following_url":"https://api.github.com/users/wootencl/following{/other_user}","gists_url":"https://api.github.com/users/wootencl/gists{/gist_id}","starred_url":"https://api.github.com/users/wootencl/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/wootencl/subscriptions","organizations_url":"https://api.github.com/users/wootencl/orgs","repos_url":"https://api.github.com/users/wootencl/repos","events_url":"https://api.github.com/users/wootencl/events{/privacy}","received_events_url":"https://api.github.com/users/wootencl/received_events","type":"User","site_admin":false},"created_at":"2015-02-24T03:44:54Z","updated_at":"2015-02-24T03:44:54Z","body":"Quick question to whomever. Why allow the user to entirely disable the default regex replacements in place of their own? I understand for customization purposes, but isn't it highly recommended in the documentation to keep the defaults because of the multi OS wide illegal characters. I was just wondering how a user could could benefit from not using them. I only ask because the way i was heading in fixing this bug i was thinking of making those particular regex default replacements immutable (still keeping the user defined replacements of course.) Thanks for the help!  "}}