Closed oarinv closed 1 year ago
这个有两个问题:
1。 保存的文件名需要走一下过滤,不能有操作系统不允许的字符存在。这个好解决,替换一下就行。比如 https://github.com/ludoux/cloudlrc/blob/master/utils/utils.go
以及那个遗留 issue 的想法,应该是文件夹不用帖子id命名而用帖子名命名。我当时也是考虑到第二点(即需要有一个文件存储所有的 tid<>帖子名 映射),嫌麻烦才没有做。
你看看你放不方便做后续修改,假如方便的话,不用关这个 PR,应该继续往你 Fork 的仓库里面推提交,这里 pr 也会更新进来的。假如不方便的话,我先合并你的PR,再后续做修改
谢谢 :)
这个有两个问题:
1。 保存的文件名需要走一下过滤,不能有操作系统不允许的字符存在。这个好解决,替换一下就行。比如 https://github.com/ludoux/cloudlrc/blob/master/utils/utils.go
以及那个遗留 issue 的想法,应该是文件夹不用帖子id命名而用帖子名命名。我当时也是考虑到第二点(即需要有一个文件存储所有的 tid<>帖子名 映射),嫌麻烦才没有做。
你看看你放不方便做后续修改,假如方便的话,不用关这个 PR,应该继续往你 Fork 的仓库里面推提交,这里 pr 也会更新进来的。假如不方便的话,我先合并你的PR,再后续做修改