| Index: adblockplussbrowser/src/org/adblockplus/sbrowser/contentblocker/engine/DefaultSubscriptions.java |
| =================================================================== |
| --- a/adblockplussbrowser/src/org/adblockplus/sbrowser/contentblocker/engine/DefaultSubscriptions.java |
| +++ b/adblockplussbrowser/src/org/adblockplus/sbrowser/contentblocker/engine/DefaultSubscriptions.java |
| @@ -30,21 +30,21 @@ import javax.xml.parsers.SAXParser; |
| import javax.xml.parsers.SAXParserFactory; |
| import org.xml.sax.Attributes; |
| import org.xml.sax.SAXException; |
| import org.xml.sax.helpers.DefaultHandler; |
| final class DefaultSubscriptions |
| { |
| - private final List<DefaultSubscriptionInfo> subscriptions = new ArrayList<DefaultSubscriptionInfo>(); |
| - private final List<DefaultSubscriptionInfo> linearSubscriptions = new ArrayList<DefaultSubscriptionInfo>(); |
| - private final HashMap<String, DefaultSubscriptionInfo> urlMap = new HashMap<String, DefaultSubscriptionInfo>(); |
| - private final List<DefaultSubscriptionInfo> adsSubscriptions = new ArrayList<DefaultSubscriptionInfo>(); |
| - private final List<DefaultSubscriptionInfo> otherSubscriptions = new ArrayList<DefaultSubscriptionInfo>(); |
| + private final List<DefaultSubscriptionInfo> subscriptions = new ArrayList<>(); |
| + private final List<DefaultSubscriptionInfo> linearSubscriptions = new ArrayList<>(); |
| + private final HashMap<String, DefaultSubscriptionInfo> urlMap = new HashMap<>(); |
| + private final List<DefaultSubscriptionInfo> adsSubscriptions = new ArrayList<>(); |
| + private final List<DefaultSubscriptionInfo> otherSubscriptions = new ArrayList<>(); |
| private DefaultSubscriptions initialize() |
| { |
| this.listSubscriptions(this.linearSubscriptions); |
| for (final DefaultSubscriptionInfo s : this.linearSubscriptions) |
| { |
| final String url = s.getUrl(); |
| @@ -65,17 +65,17 @@ final class DefaultSubscriptions |
| } |
| } |
| return this; |
| } |
| public List<Subscription> createSubscriptions() throws IOException |
| { |
| - final ArrayList<Subscription> subs = new ArrayList<Subscription>(); |
| + final ArrayList<Subscription> subs = new ArrayList<>(); |
| for (DefaultSubscriptionInfo info : this.linearSubscriptions) |
| { |
| if (!info.getUrl().isEmpty()) |
| { |
| final Subscription sub = Subscription.create(info.getUrl()); |
| sub.putMeta(Subscription.KEY_TITLE, info.getTitle()); |
| subs.add(sub); |
| } |
| @@ -121,40 +121,35 @@ final class DefaultSubscriptions |
| { |
| final SAXParserFactory factory = SAXParserFactory.newInstance(); |
| factory.setValidating(false); |
| final SAXParser parser = factory.newSAXParser(); |
| final SubscriptionParser handler = new SubscriptionParser(); |
| parser.parse(in, handler); |
| return handler.subscriptions.initialize(); |
| } |
| - catch (final ParserConfigurationException e) |
| + catch (final ParserConfigurationException | SAXException e) |
| { |
| - throw new IOException("ParserConfigurationException: " + e.getMessage()); |
| - } |
| - catch (final SAXException e) |
| - { |
| - e.printStackTrace(); |
| - throw new IOException("SAXException: " + e.getMessage()); |
| + throw new IOException(e.getClass().getSimpleName() + ": " + e.getMessage()); |
| } |
| } |
| private static class SubscriptionParser extends DefaultHandler |
| { |
| private boolean inSubscriptions = false; |
| private boolean inVariants = false; |
| private final static String KEY_SUPPLEMENTS = "supplements"; |
| private final static String KEY_SUBSCRIPTIONS = "subscriptions"; |
| private final static String KEY_SUBSCRIPTION = "subscription"; |
| private final static String KEY_VARIANTS = "variants"; |
| private final static String KEY_VARIANT = "variant"; |
| private final DefaultSubscriptions subscriptions = new DefaultSubscriptions(); |
| - private final LinkedList<DefaultSubscriptionInfo> subscriptionStack = new LinkedList<DefaultSubscriptionInfo>(); |
| + private final LinkedList<DefaultSubscriptionInfo> subscriptionStack = new LinkedList<>(); |
| private DefaultSubscriptionInfo subscription = null; |
| private DefaultSubscriptionInfo variant = null; |
| @Override |
| public void startElement(final String uri, final String localName, final String qName, |
| final Attributes attributes) throws SAXException |
| { |
| super.startElement(uri, localName, qName, attributes); |