Bump ASM and fix providers with comments

build.gradle

@@ -31,6 +31,7 @@ dependencies {
 
 configurations.all {
  outgoing {
+ capability("net.minecraftforge:$project.name:$version") // For sub modules, they require the exact casing of the name
  capability("net.minecraftforge:securemodules:$version")
  capability("cpw.mods:securejarhandler:$version")
  }

settings.gradle

@@ -12,7 +12,7 @@ plugins {
 dependencyResolutionManagement {
  versionCatalogs {
  libs {
- version('asm', '9.6')
+ version('asm', '9.7')
  library('asm', 'org.ow2.asm', 'asm' ).versionRef('asm')
  library('asm-tree', 'org.ow2.asm', 'asm-tree' ).versionRef('asm')
  library('asm-commons', 'org.ow2.asm', 'asm-commons').versionRef('asm')

src/main/java/cpw/mods/jarhandling/impl/Jar.java

@@ -292,11 +292,23 @@ private List<Provider> gatherProviders(BiPredicate<String, String> filter) {
  public static Provider getProvider(Path path, BiPredicate<String, String> filter) {
  var sname = path.getFileName().toString();
  try {
- var entries = Files.readAllLines(path).stream()
- .map(String::trim)
- .filter(l -> l.length() > 0 && !l.startsWith("#"))
- .filter(p -> filter == null || filter.test(p.replace('.', '/'), ""))
- .toList();
+ var entries = new ArrayList<String>();
+ for (var line : Files.readAllLines(path)) {
+ int idx = line.indexOf('#');
+ if (idx != -1)
+ line = line.substring(0, idx);
+ line = line.trim();
+
+ if (line.isEmpty())
+ continue;
+
+ if (filter != null && !filter.test(line.replace('.', '/'), ""))
+ continue;
+
+ entries.add(line);
+
+ }
+
  return new Provider(sname, entries);
  } catch (IOException e) {
  return sneak(e);