Issue 5237 - Correct filter lists not applied on other language installation

adblockplussbrowser/src/org/adblockplus/sbrowser/contentblocker/engine/DefaultSubscriptions.java

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-2016 Eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 75 matching lines @@
   public DefaultSubscriptionInfo getForUrl(final String url)
   {
     return this.urlMap.get(url);
   }
 
   public DefaultSubscriptionInfo getForUrl(final URL url)
   {
     return url != null ? this.getForUrl(url.toString()) : null;
   }
 
-  List<DefaultSubscriptionInfo> getAdsSubscriptions()

[anton, 2017/08/08 05:43:41]

why not public (package now)?

[jens, 2017/08/08 13:53:05]

I thought it was to keep this method package-private for now, as we currently
don't use it outside of the engine package.

[anton, 2017/08/11 11:25:07]

That looks strange. Please consider making it 'private' or 'public' unless
'package' is common used in other classes. 

[jens, 2017/08/11 13:27:23]

Acknowledged.

+  public List<DefaultSubscriptionInfo> getAdsSubscriptions()
   {
     return adsSubscriptions;
   }
 
   private void listSubscriptions(final List<DefaultSubscriptionInfo> output)
   {
     for (final DefaultSubscriptionInfo s : this.subscriptions)
     {
       this.listSubscriptions(s, output);
     }
@@ skipping 177 matching lines @@
       }
     }
   }
 
   @Override
   public String toString()
   {
     return this.linearSubscriptions.toString();
   }
 }