关于modlaunch.vb中，classpath构造阶段对cleanroom的处理代码问题

[qiMengStars]

如题，在modlaunch.vb中，有这样一段代码：

PCL-CE/Plain Craft Launcher 2/Modules/Minecraft/ModLaunch.vb

Lines 2059 to 2061 in c241fdd

	If Library.Name IsNot Nothing AndAlso Library.Name.Contains("com.cleanroommc:cleanroom:0.2") Then 'Cleanroom 的主 Jar 必须放在 ClassPath 第一位
	CpStrings.Insert(0, Library.LocalPath)
	End If

它被添加于feat(modpack): MMC JSON-Patches 支持
按照注释来看，Library.Name应当不包含版本号

这意味着，这个精确到0.2版本号的if判断应当始终不会为true，因而它也从来不会执行
是有什么我没注意到的地方吗

[qiMengStars]

@Pigeon0v0 大佬打扰了，但是真的挺好奇的，我看这段代码是你加的就直接@了
它在C#化之后依然留着

PCL-CE/Plain Craft Launcher 2/Modules/Minecraft/ModLaunch.cs

Lines 2894 to 2896 in c241fdd

	if (Library.Name is not null &&
	Library.Name.Contains("com.cleanroommc:cleanroom:0.2")) // Cleanroom 的主 Jar 必须放在 ClassPath 第一位
	CpStrings.Insert(0, Library.LocalPath);

按道理来说，它永远都不会命中，自然也不会有用，但是我没敢在pr里删
此外我问了一下某位cleanroom维护者，他说cleanroom并没要求过在classpath中的位置
可以解释一下它是确实在解决一些早期问题，还是说已经没用了吗，谢谢

[Pigeon0v0]

Uhhh 你删了试试看？我也不知道当时为啥这么写了.jpg

另外，早期我测试下来确实是放在 cp 第一位就可以解决一些崩溃问题，你可以删了测试看看有没有问题

[qiMengStars]

阿？这东西本来跑的时候就不会命中的，我暂时没装上0.2.x的版本（因为自动安装里没有），反正新的肯定不中，所以删了也不会有问题，我去找个0.2.x的再测测看

[qiMengStars]

按道理来说应该是没问题的，测试了cleanroom0.2.4在不命中的情况下能跑通
MMC的包导进去会报错，暂时懒得看发生了啥，我就在HMCL装了一个然后导出MCBBS去测了
但是这样导出之后Name是个null，这if分支更不可能中了（）
刚好我那pr还没合，要删了吗
需要提个新PR给删了吗

[Pigeon0v0]

按道理来说应该是没问题的，测试了cleanroom0.2.4在不命中的情况下能跑通

MMC的包导进去会报错，暂时懒得看发生了啥，我就在HMCL装了一个然后导出MCBBS去测了

但是这样导出之后Name是个null，这if分支更不可能中了（）

刚好我那pr还没合，要删了吗

需要提个新PR给删了吗

那就是其他地方哪里 patch 过了，但是当时写的时候脑子一糊涂给这里忘了...（

感谢 review，删了吧