Loading...
iokit/Kernel/IOPMPowerSourceList.cpp xnu-792.25.20 xnu-201.19
--- xnu/xnu-792.25.20/iokit/Kernel/IOPMPowerSourceList.cpp
+++ xnu/xnu-201.19/iokit/Kernel/IOPMPowerSourceList.cpp
@@ -19,6 +19,7 @@
  * 
  * @APPLE_LICENSE_HEADER_END@
  */
+#include <stddef.h> 
 #include <IOKit/pwr_mgt/IOPM.h>
 #include <IOKit/pwr_mgt/IOPMPowerSourceList.h>
 #include <IOKit/pwr_mgt/IOPMPowerSource.h>
@@ -26,61 +27,54 @@
 #define super OSObject
 OSDefineMetaClassAndStructors(IOPMPowerSourceList,OSObject)
 
-//******************************************************************************
+//*********************************************************************************
 // init
 //
-//******************************************************************************
+//*********************************************************************************
 void IOPMPowerSourceList::initialize ( void )
 {
     firstItem = NULL;
     length = 0;
 }
 
-//******************************************************************************
+//*********************************************************************************
 // addToList
 //
-//******************************************************************************
+//*********************************************************************************
 
-IOReturn IOPMPowerSourceList::addToList(IOPMPowerSource *newPowerSource)
+IOReturn IOPMPowerSourceList::addToList ( IOPMPowerSource * newPowerSource )
 {
     IOPMPowerSource * nextPowerSource;
-
-    // Is new object already in the list?
-    nextPowerSource = firstItem;
-    while (  nextPowerSource != NULL ) 
-    {
-        if ( nextPowerSource == newPowerSource ) 
-        {
-            // yes, just return
-            return IOPMNoErr;				
+    nextPowerSource = firstItem;				// Is new object already in the list?
+    while (  nextPowerSource != NULL ) {
+        if ( nextPowerSource == newPowerSource ) {
+            return IOPMNoErr;				// yes, just return
         }
         nextPowerSource = nextInList(nextPowerSource);
     }
-
-    // add it to list
-    newPowerSource->nextInList = firstItem;
+    newPowerSource->nextInList = firstItem;		// add it to list
     firstItem = newPowerSource;
-    length++;
+    length += 1;
     return IOPMNoErr;
 }
 
 
-//******************************************************************************
+//*********************************************************************************
 // firstInList
 //
-//******************************************************************************
+//*********************************************************************************
 
 IOPMPowerSource * IOPMPowerSourceList::firstInList ( void )
 {
     return firstItem;
 }
 
-//******************************************************************************
+//*********************************************************************************
 // nextInList
 //
-//******************************************************************************
+//*********************************************************************************
 
-IOPMPowerSource * IOPMPowerSourceList::nextInList(IOPMPowerSource *currentItem)
+IOPMPowerSource * IOPMPowerSourceList::nextInList ( IOPMPowerSource * currentItem )
 {
     if ( currentItem != NULL ) {
        return (currentItem->nextInList);
@@ -88,56 +82,54 @@
     return NULL;
 }
 
-//******************************************************************************
+//*********************************************************************************
 // numberOfItems
 //
-//******************************************************************************
+//*********************************************************************************
 
 unsigned long IOPMPowerSourceList::numberOfItems ( void )
 {
     return length;
 }
 
-//******************************************************************************
+//*********************************************************************************
 // removeFromList
 //
 // Find the item in the list, unlink it, and free it.
-//******************************************************************************
+//*********************************************************************************
 
 IOReturn IOPMPowerSourceList::removeFromList ( IOPMPowerSource * theItem )
 {
     IOPMPowerSource * item = firstItem;
     IOPMPowerSource * temp;
 
-    if ( NULL == item) goto exit;
-    
-    if ( item  == theItem ) {
-        firstItem = item->nextInList;
-        length--;
-        item->release();
-        return IOPMNoErr;
-    }
-    while ( item->nextInList != NULL ) {
-        if ( item->nextInList == theItem ) {
-            temp = item->nextInList;
-            item->nextInList = temp->nextInList;
+    if ( item != NULL ) {
+        if ( item  == theItem ) {
+            firstItem = item->nextInList;
             length--;
-            temp->release();
+            item->release();
             return IOPMNoErr;
         }
-        item = item->nextInList;
+        while ( item->nextInList != NULL ) {
+            if ( item->nextInList == theItem ) {
+                temp = item->nextInList;
+                item->nextInList = temp->nextInList;
+                length--;
+                temp->release();
+                return IOPMNoErr;
+            }
+            item = item->nextInList;
+        }
     }
-
-exit:
     return IOPMNoErr;
 }
 
 
-//******************************************************************************
+//*********************************************************************************
 // free
 //
 // Free all items in the list, and then free the list itself
-//******************************************************************************
+//*********************************************************************************
 
 void IOPMPowerSourceList::free (void )
 {
@@ -149,7 +141,7 @@
         next->release();
         next = firstItem;        
     }
-    super::free();
+super::free();
 }