From b05fa45080a1f60b80a30e3ab51c53a9d0e3056f Mon Sep 17 00:00:00 2001
From: Ritvik Saraf <13ritvik@gmail.com>
Date: Fri, 28 Dec 2018 13:02:00 +0530
Subject: [PATCH] resolved review comments

---
 .../newpipe/extractor/StreamingService.java   |  4 +--
 .../services/youtube/YoutubeService.java      | 26 +++++++++----------
 .../extractors/YoutubeCommentsExtractor.java  |  1 +
 3 files changed, 15 insertions(+), 16 deletions(-)

diff --git a/extractor/src/main/java/org/schabi/newpipe/extractor/StreamingService.java b/extractor/src/main/java/org/schabi/newpipe/extractor/StreamingService.java
index 702c4f360..8818ea362 100644
--- a/extractor/src/main/java/org/schabi/newpipe/extractor/StreamingService.java
+++ b/extractor/src/main/java/org/schabi/newpipe/extractor/StreamingService.java
@@ -222,7 +222,7 @@ public abstract class StreamingService {
     public ChannelExtractor getChannelExtractor(ListLinkHandler linkHandler) throws ExtractionException {
         return getChannelExtractor(linkHandler, NewPipe.getPreferredLocalization());
     }
- 
+    
     public PlaylistExtractor getPlaylistExtractor(ListLinkHandler linkHandler) throws ExtractionException {
         return getPlaylistExtractor(linkHandler, NewPipe.getPreferredLocalization());
     }
@@ -300,8 +300,6 @@ public abstract class StreamingService {
 
 
 
-	/**
-     * figure out where the link is pointing to (a channel, video, playlist, etc.)
     /**
      * Figures out where the link is pointing to (a channel, a video, a playlist, etc.)
      * @param url the url on which it should be decided of which link type it is
diff --git a/extractor/src/main/java/org/schabi/newpipe/extractor/services/youtube/YoutubeService.java b/extractor/src/main/java/org/schabi/newpipe/extractor/services/youtube/YoutubeService.java
index 0ff98b4f9..e5569775d 100644
--- a/extractor/src/main/java/org/schabi/newpipe/extractor/services/youtube/YoutubeService.java
+++ b/extractor/src/main/java/org/schabi/newpipe/extractor/services/youtube/YoutubeService.java
@@ -1,10 +1,5 @@
 package org.schabi.newpipe.extractor.services.youtube;
 
-import static java.util.Arrays.asList;
-import static org.schabi.newpipe.extractor.StreamingService.ServiceInfo.MediaCapability.AUDIO;
-import static org.schabi.newpipe.extractor.StreamingService.ServiceInfo.MediaCapability.LIVE;
-import static org.schabi.newpipe.extractor.StreamingService.ServiceInfo.MediaCapability.VIDEO;
-
 import org.schabi.newpipe.extractor.StreamingService;
 import org.schabi.newpipe.extractor.SuggestionExtractor;
 import org.schabi.newpipe.extractor.channel.ChannelExtractor;
@@ -38,6 +33,10 @@ import org.schabi.newpipe.extractor.stream.StreamExtractor;
 import org.schabi.newpipe.extractor.subscription.SubscriptionExtractor;
 import org.schabi.newpipe.extractor.utils.Localization;
 
+import static java.util.Arrays.asList;
+import static org.schabi.newpipe.extractor.StreamingService.ServiceInfo.MediaCapability.AUDIO;
+import static org.schabi.newpipe.extractor.StreamingService.ServiceInfo.MediaCapability.LIVE;
+import static org.schabi.newpipe.extractor.StreamingService.ServiceInfo.MediaCapability.VIDEO;
 
 /*
  * Created by Christian Schabesberger on 23.08.15.
@@ -140,15 +139,16 @@ public class YoutubeService extends StreamingService {
         return new YoutubeSubscriptionExtractor(this);
     }
 
-	@Override
-	public ListLinkHandlerFactory getCommentsLHFactory() {
-		return YoutubeCommentsLinkHandlerFactory.getInstance();
-	}
+    @Override
+    public ListLinkHandlerFactory getCommentsLHFactory() {
+        return YoutubeCommentsLinkHandlerFactory.getInstance();
+    }
 
-	@Override
-	public CommentsExtractor getCommentsExtractor(ListLinkHandler urlIdHandler, Localization localization) throws ExtractionException {
-		return new YoutubeCommentsExtractor(this, urlIdHandler, localization);
-	}
+    @Override
+    public CommentsExtractor getCommentsExtractor(ListLinkHandler urlIdHandler, Localization localization)
+            throws ExtractionException {
+        return new YoutubeCommentsExtractor(this, urlIdHandler, localization);
+    }
 
     @Override
     public boolean isCommentsSupported() {
diff --git a/extractor/src/main/java/org/schabi/newpipe/extractor/services/youtube/extractors/YoutubeCommentsExtractor.java b/extractor/src/main/java/org/schabi/newpipe/extractor/services/youtube/extractors/YoutubeCommentsExtractor.java
index e0bbaa11f..85d150014 100644
--- a/extractor/src/main/java/org/schabi/newpipe/extractor/services/youtube/extractors/YoutubeCommentsExtractor.java
+++ b/extractor/src/main/java/org/schabi/newpipe/extractor/services/youtube/extractors/YoutubeCommentsExtractor.java
@@ -35,6 +35,7 @@ import com.grack.nanojson.JsonParser;
 
 public class YoutubeCommentsExtractor extends CommentsExtractor {
 
+    // using the mobile site for comments because it loads faster and uses get requests instead of post
     private static final String USER_AGENT = "Mozilla/5.0 (Android 8.1.0; Mobile; rv:62.0) Gecko/62.0 Firefox/62.0";
     private static final Pattern YT_CLIENT_NAME_PATTERN = Pattern.compile("INNERTUBE_CONTEXT_CLIENT_NAME\\\":(.*?)[,}]");