update: Action build type to MinSizeRel

.github/ISSUE_TEMPLATE/bug_report.md

@@ -1,25 +1,21 @@
 ---
 name: "[请按照此模板填写] 报告 Bug"
-about: "创建一个 Bug 报告，不按照模板的 Issue 会被关闭。"
+about: "创建一个 Bug 报告，不按照模板的 Issue 会被直接关闭，不予回复。"
 title: "[Bug] 总结你的 Bug 报告"
 labels: bug
 assignees: taurusxin
 ---
 
 **Bug 描述**
-清晰地描述一下 Bug 的大致问题。
+
+清晰地描述一下 Bug 的大致问题，例如无法转换，或者其他问题。
 
 **复现方法**
 
 复现此 Bug 的方法
 
-1. 打开 '...'
+1. 使用本项目处理文件 '...'
-2. 点击 '....'
+2. 发生报错
-3. 发生报错
-
-**预期行为**
-
-解释一下原本应该出现的结果。
 
 **屏幕截图**
 
@@ -27,9 +23,10 @@ assignees: taurusxin
 
 **环境**
 
-- OS: [e.g. Windows 11]
+- 操作系统: Windows / macOS / Linux
-- 软件版本: [e.g. 1.3.2]
+- 网易云版本（重要）: [e.g. 3.0.1]
+- 所选择的音质: 极高、无损等
 
 **附加内容**
 
-添加更多其他内容以帮助开发者更好地了解这个 Bug。
+如果遇到无法转换的问题，请将样本附加到这里，便于分析。

.github/workflows/autobuild.yml

@@ -1,6 +1,6 @@
 name: CI
 env:
-  BUILD_TYPE: Release
+  BUILD_TYPE: MinSizeRel
   BUILD_PATH: build
 on:
   push:

.gitignore

@@ -5,3 +5,5 @@ ncmdump
 .idea
 
 build
+
+cmake-build-*

README.md

@@ -48,19 +48,19 @@ ncmdump -v
 ncmdump 1.ncm 2.ncm...
 ```
 
-你可以使用 `-d` 参数来指定一个文件夹，对文件夹下的所有以 ncm 为扩展名的文件进行批量处理
+使用 `-d` 参数来指定一个文件夹，对文件夹下的所有以 ncm 为扩展名的文件进行批量处理
 
 ```shell
 ncmdump -d source_dir
 ```
 
-你可以使用 `-r` 配合 `-d` 参数来递归处理文件夹下的所有以 ncm 为扩展名的文件
+使用 `-r` 配合 `-d` 参数来递归处理文件夹下的所有以 ncm 为扩展名的文件
 
 ```shell
 ncmdump -d source_dir -r
 ```
 
-你可以使用 `-o` 参数来指定输出目录，将转换后的文件输出到指定目录，该参数支持与 `-r` 参数一起使用
+使用 `-o` 参数来指定输出目录，将转换后的文件输出到指定目录，该参数支持与 `-r` 参数一起使用
 
 ```shell
 # 处理单个或多个文件并输出到指定目录

src/include/ncmcrypt.h

@@ -47,7 +47,7 @@ private:
 	NcmFormat mFormat;
 	std::string mImageData;
 	std::ifstream mFile;
-	unsigned char mKeyBox[256];
+	unsigned char mKeyBox[256]{};
 	NeteaseMusicMetadata* mMetaData;
 
 private:

src/ncmcrypt.cpp

@@ -224,7 +224,7 @@ void NeteaseCrypt::FixMetadata()
         audioFile = new TagLib::MPEG::File(mDumpFilepath.c_str());
         tag = dynamic_cast<TagLib::MPEG::File *>(audioFile)->ID3v2Tag(true);
 
-        if (mImageData.length() > 0)
+        if (!mImageData.empty())
         {
             TagLib::ID3v2::AttachedPictureFrame *frame = new TagLib::ID3v2::AttachedPictureFrame;
 
@@ -239,7 +239,7 @@ void NeteaseCrypt::FixMetadata()
         audioFile = new TagLib::FLAC::File(mDumpFilepath.c_str());
         tag = audioFile->tag();
 
-        if (mImageData.length() > 0)
+        if (!mImageData.empty())
         {
             TagLib::FLAC::Picture *cover = new TagLib::FLAC::Picture;
             cover->setMimeType(mimeType(mImageData));
@@ -257,7 +257,7 @@ void NeteaseCrypt::FixMetadata()
         tag->setAlbum(TagLib::String(mMetaData->album(), TagLib::String::UTF8));
     }
 
-    tag->setComment(TagLib::String("Create by taurusxin/ncmdump.", TagLib::String::UTF8));
+    // tag->setComment(TagLib::String("Create by taurusxin/ncmdump.", TagLib::String::UTF8));
 
     audioFile->save();
     audioFile->~File();
@@ -301,7 +301,7 @@ void NeteaseCrypt::Dump(std::string const &outputDir = "")
                 mFormat = NeteaseCrypt::FLAC;
             }
 
-            output.open(mDumpFilepath, output.out | output.binary);
+            output.open(mDumpFilepath, std::ofstream::out | std::ofstream::binary);
         }
 
         output.write((char *)buffer.data(), n);

taglib

@@ -1 +1 @@
-Subproject commit 0896fb90920c125e55248360d271d1a1674e2a4d
+Subproject commit e3de03501ff66221d1f1f971022b248d5b38ba06