commit fa319e6d918b84f93fb5457af5d1cca6421ac517
author Glenn Kasten <gkasten@google.com> Mon Jul 29 17:17:38 2013 -0700
committer Glenn Kasten <gkasten@google.com> Mon Jul 29 17:20:47 2013 -0700
tree 2ccff564a324d72c8bd5d872a220f56b26431203
parent 548efc94813c1dec6e8cf6c085ae41ccb04827f1

Remove CC_LIKELY and CC_UNLIKELY where not needed

Only keep them in performance-sensitive code

Change-Id: Ib257ddd7bc39ce9896997ffae008c524ac743d01

services/audioflinger/Threads.cpp

diff --git a/services/audioflinger/Threads.cpp b/services/audioflinger/Threads.cpp
index 3d8fd45..e71c66e 100644
--- a/services/audioflinger/Threads.cpp
+++ b/services/audioflinger/Threads.cpp
@@ -25,7 +25,6 @@
 #include <fcntl.h>
 #include <sys/stat.h>
 #include <cutils/properties.h>
-#include <cutils/compiler.h>
 #include <media/AudioParameter.h>
 #include <utils/Log.h>
 #include <utils/Trace.h>
@@ -1754,7 +1753,7 @@
         const Vector< sp<Track> >& tracksToRemove)
 {
     size_t count = tracksToRemove.size();
-    if (CC_UNLIKELY(count)) {
+    if (count) {
         for (size_t i = 0 ; i < count ; i++) {
             const sp<Track>& track = tracksToRemove.itemAt(i);
             if (!track->isOutputTrack()) {
@@ -2282,7 +2281,7 @@
 void AudioFlinger::PlaybackThread::removeTracks_l(const Vector< sp<Track> >& tracksToRemove)
 {
     size_t count = tracksToRemove.size();
-    if (CC_UNLIKELY(count)) {
+    if (count) {
         for (size_t i=0 ; i<count ; i++) {
             const sp<Track>& track = tracksToRemove.itemAt(i);
             mActiveTracks.remove(track);
@@ -3551,7 +3550,7 @@
         AudioBufferProvider::Buffer buffer;
         buffer.frameCount = frameCount;
         mActiveTrack->getNextBuffer(&buffer);
-        if (CC_UNLIKELY(buffer.raw == NULL)) {
+        if (buffer.raw == NULL) {
             memset(curBuf, 0, frameCount * mFrameSize);
             break;
         }
@@ -3852,7 +3851,7 @@
                 track->mFillingUpStatus = Track::FS_ACTIVE;
                 mLeftVolFloat = mRightVolFloat = 0;
                 if (track->mState == TrackBase::RESUMING) {
-                    if (CC_UNLIKELY(mPausedBytesRemaining)) {
+                    if (mPausedBytesRemaining) {
                         // Need to continue write that was interrupted
                         mCurrentWriteLength = mPausedWriteLength;
                         mBytesRemaining = mPausedBytesRemaining;
@@ -4274,7 +4273,7 @@
 
             buffer.frameCount = mFrameCount;
             status_t status = mActiveTrack->getNextBuffer(&buffer);
-            if (CC_LIKELY(status == NO_ERROR)) {
+            if (status == NO_ERROR) {
                 readOnce = true;
                 size_t framesOut = buffer.frameCount;
                 if (mResampler == NULL) {