From f18ee8e83d048ac3fb062c120387c546e47e887a Mon Sep 17 00:00:00 2001
From: litetex <40789489+litetex@users.noreply.github.com>
Date: Fri, 1 Oct 2021 18:44:47 +0200
Subject: [PATCH] Added a bit more documentation

---
 .../fragments/detail/VideoDetailPlayerCrasher.java        | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/app/src/main/java/org/schabi/newpipe/fragments/detail/VideoDetailPlayerCrasher.java b/app/src/main/java/org/schabi/newpipe/fragments/detail/VideoDetailPlayerCrasher.java
index d8a4a99d1..a7c382f33 100644
--- a/app/src/main/java/org/schabi/newpipe/fragments/detail/VideoDetailPlayerCrasher.java
+++ b/app/src/main/java/org/schabi/newpipe/fragments/detail/VideoDetailPlayerCrasher.java
@@ -27,8 +27,14 @@ import java.util.Map;
 import java.util.concurrent.TimeoutException;
 import java.util.function.Supplier;
 
+/**
+ * Outsourced logic for crashing the player in the {@link VideoDetailFragment}.
+ */
 public class VideoDetailPlayerCrasher {
 
+    // This has to be <= 23 chars on devices running Android 7 or lower (API <= 25)
+    // or it fails with an IllegalArgumentException
+    // https://stackoverflow.com/a/54744028
     private static final String TAG = "VideoDetPlayerCrasher";
 
     @NonNull
@@ -109,6 +115,8 @@ public class VideoDetailPlayerCrasher {
             return;
         }
 
+        // -- Build the dialog/UI --
+
         final Context themeWrapperContext = getThemeWrapperContext();
 
         final LayoutInflater inflater = LayoutInflater.from(themeWrapperContext);