Always call setCurrentItem from updateCurrentItem

Previously updateCurrentItem() was only called if pager.setItem()
returned true which would not be the case when the pager was manually
swiped since the position had already been set to the new value.

Closes pockethub#352

Author: kevinsawicki

app/src/main/java/com/github/mobile/ui/TabPagerActivity.java

@@ -136,9 +136,10 @@ protected int getContentView() {
     }
 
     private void updateCurrentItem(final int newPosition) {
-        if (newPosition > -1 && newPosition < adapter.getCount()
-                && pager.setItem(newPosition))
+        if (newPosition > -1 && newPosition < adapter.getCount()) {
+            pager.setItem(newPosition);
             setCurrentItem(newPosition);
+        }
     }
 
     private void createPager() {

app/src/main/java/com/github/mobile/ui/comment/CommentPreviewPagerAdapter.java

@@ -16,7 +16,6 @@
 package com.github.mobile.ui.comment;
 
 import android.support.v4.app.Fragment;
-
 import com.actionbarsherlock.app.SherlockFragmentActivity;
 import com.github.mobile.ui.FragmentPagerAdapter;
 

app/src/main/java/com/github/mobile/ui/comment/CreateCommentActivity.java

@@ -61,7 +61,6 @@ public void invalidateOptionsMenu() {
         if (applyItem != null)
             applyItem.setEnabled(adapter != null
                     && !TextUtils.isEmpty(adapter.getCommentText()));
-
     }
 
     @Override