From d208cc38693381abfb4411abaca637ecb0959cc0 Mon Sep 17 00:00:00 2001
From: Phillip Hsu <philliphsu@outlook.com>
Date: Fri, 23 Sep 2016 02:17:38 -0700
Subject: [PATCH] Remove unused and outdated code

---
 .../com/philliphsu/clock2/MainActivity.java   | 59 +------------------
 app/src/main/res/layout/fragment_main.xml     | 16 -----
 2 files changed, 1 insertion(+), 74 deletions(-)
 delete mode 100644 app/src/main/res/layout/fragment_main.xml

diff --git a/app/src/main/java/com/philliphsu/clock2/MainActivity.java b/app/src/main/java/com/philliphsu/clock2/MainActivity.java
index e028c8e..a5c414a 100644
--- a/app/src/main/java/com/philliphsu/clock2/MainActivity.java
+++ b/app/src/main/java/com/philliphsu/clock2/MainActivity.java
@@ -13,11 +13,9 @@ import android.support.v4.content.ContextCompat;
 import android.support.v4.view.ViewPager;
 import android.util.Log;
 import android.util.SparseArray;
-import android.view.LayoutInflater;
 import android.view.MenuItem;
 import android.view.View;
 import android.view.ViewGroup;
-import android.widget.TextView;
 
 import com.philliphsu.clock2.alarms.ui.AlarmsFragment;
 import com.philliphsu.clock2.list.RecyclerViewFragment;
@@ -222,14 +220,6 @@ public class MainActivity extends BaseActivity {
         mFab.setOnClickListener(new View.OnClickListener() {
             @Override
             public void onClick(View view) {
-//                Intent intent = new Intent(MainActivity.this, EditAlarmActivity.class);
-//                // Call Fragment#startActivityForResult() instead of Activity#startActivityForResult()
-//                // because we want the result to be handled in the Fragment, not in this Activity.
-//                // FragmentActivity does NOT deliver the result to the Fragment, i.e. your
-//                // Fragment's onActivityResult() will NOT be called.
-//                mSectionsPagerAdapter.getFragment()
-//                        .startActivityForResult(intent, AlarmsFragment.REQUEST_CREATE_ALARM);
-
                 Fragment f = mSectionsPagerAdapter.getFragment(mViewPager.getCurrentItem());
                 if (f instanceof RecyclerViewFragment) {
                     ((RecyclerViewFragment) f).onFabClick();
@@ -291,18 +281,6 @@ public class MainActivity extends BaseActivity {
         // http://stackoverflow.com/q/6147884/5055032
         // http://stackoverflow.com/a/24303360/5055032
         super.onActivityResult(requestCode, resultCode, data);
-        // This is a hacky workaround when you absolutely must have a Fragment
-        // handle the result, even when it was not the one to start the requested
-        // Activity. For example, the ExpandedAlarmViewHolder can start the ringtone
-        // picker dialog (which is an Activity) for a result; ExpandedAlarmViewHolder
-        // has no reference to the AlarmsFragment, but it does have a reference to a
-        // Context (which we can cast to Activity). Thus, ExpandedAlarmViewHolder
-        // uses Activity#startActivityForResult().
-
-        // THIS WAS ACTUALLY A BAD IDEA, ESPECIALLY FOR TIMERSFRAGMENT. THIS ENDS UP ADDING
-        // DUPLICATE TIMERS.
-//        mSectionsPagerAdapter.getFragment(mViewPager.getCurrentItem())
-//                .onActivityResult(requestCode, resultCode, data);
 
         switch (requestCode) {
             case REQUEST_THEME_CHANGE:
@@ -366,41 +344,6 @@ public class MainActivity extends BaseActivity {
         return mFab.getWidth() / 2f + margin;
     }
 
-    /**
-     * A placeholder fragment containing a simple view.
-     */
-    public static class PlaceholderFragment extends Fragment {
-        /**
-         * The fragment argument representing the section number for this
-         * fragment.
-         */
-        private static final String ARG_SECTION_NUMBER = "section_number";
-
-        public PlaceholderFragment() {
-        }
-
-        /**
-         * Returns a new instance of this fragment for the given section
-         * number.
-         */
-        public static PlaceholderFragment newInstance(int sectionNumber) {
-            PlaceholderFragment fragment = new PlaceholderFragment();
-            Bundle args = new Bundle();
-            args.putInt(ARG_SECTION_NUMBER, sectionNumber);
-            fragment.setArguments(args);
-            return fragment;
-        }
-
-        @Override
-        public View onCreateView(LayoutInflater inflater, ViewGroup container,
-                                 Bundle savedInstanceState) {
-            View rootView = inflater.inflate(R.layout.fragment_main, container, false);
-            TextView textView = (TextView) rootView.findViewById(R.id.section_label);
-            textView.setText(getString(R.string.section_format, getArguments().getInt(ARG_SECTION_NUMBER)));
-            return rootView;
-        }
-    }
-
     /**
      * A {@link FragmentPagerAdapter} that returns a fragment corresponding to
      * one of the sections/tabs/pages.
@@ -426,7 +369,7 @@ public class MainActivity extends BaseActivity {
                 case PAGE_STOPWATCH:
                     return new StopwatchFragment();
                 default:
-                    return PlaceholderFragment.newInstance(position + 1);
+                    throw new IllegalStateException("No fragment can be instantiated for position " + position);
             }
         }
 
diff --git a/app/src/main/res/layout/fragment_main.xml b/app/src/main/res/layout/fragment_main.xml
deleted file mode 100644
index dd914e3..0000000
--- a/app/src/main/res/layout/fragment_main.xml
+++ /dev/null
@@ -1,16 +0,0 @@
-<RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android"
-                xmlns:tools="http://schemas.android.com/tools"
-                android:layout_width="match_parent"
-                android:layout_height="match_parent"
-                android:paddingBottom="@dimen/activity_vertical_margin"
-                android:paddingLeft="@dimen/activity_horizontal_margin"
-                android:paddingRight="@dimen/activity_horizontal_margin"
-                android:paddingTop="@dimen/activity_vertical_margin"
-                tools:context="com.philliphsu.clock2.MainActivity$PlaceholderFragment">
-
-    <TextView
-        android:id="@+id/section_label"
-        android:layout_width="wrap_content"
-        android:layout_height="wrap_content"/>
-
-</RelativeLayout>