Merge pull request #1152 from lioncash/nullptr

Replace 0 literals with nullptr where applicable

src/citra_qt/debugger/profiler.cpp

@@ -150,7 +150,7 @@ void ProfilerWidget::setProfilingInfoUpdateEnabled(bool enable)
 
 class MicroProfileWidget : public QWidget {
 public:
-    MicroProfileWidget(QWidget* parent = 0);
+    MicroProfileWidget(QWidget* parent = nullptr);
 
 protected:
     void paintEvent(QPaintEvent* ev) override;

src/citra_qt/debugger/profiler.h

@@ -53,7 +53,7 @@ class MicroProfileDialog : public QWidget {
     Q_OBJECT
 
 public:
-    MicroProfileDialog(QWidget* parent = 0);
+    MicroProfileDialog(QWidget* parent = nullptr);
 
     /// Returns a QAction that can be used to toggle visibility of this dialog.
     QAction* toggleViewAction();

src/common/chunk_file.h

@@ -575,10 +575,10 @@ public:
     }
 
     template<class T, LinkedListItem<T>* (*TNew)(), void (*TFree)(LinkedListItem<T>*), void (*TDo)(PointerWrap&, T*)>
-    void DoLinkedList(LinkedListItem<T>*& list_start, LinkedListItem<T>** list_end=0)
+    void DoLinkedList(LinkedListItem<T>*& list_start, LinkedListItem<T>** list_end = nullptr)
     {
         LinkedListItem<T>* list_cur = list_start;
-        LinkedListItem<T>* prev = 0;
+        LinkedListItem<T>* prev = nullptr;
 
         while (true)
         {

src/common/memory_util.cpp

@@ -28,9 +28,9 @@
 void* AllocateExecutableMemory(size_t size, bool low)
 {
 #if defined(_WIN32)
-    void* ptr = VirtualAlloc(0, size, MEM_COMMIT, PAGE_EXECUTE_READWRITE);
+    void* ptr = VirtualAlloc(nullptr, size, MEM_COMMIT, PAGE_EXECUTE_READWRITE);
 #else
-    static char *map_hint = 0;
+    static char* map_hint = nullptr;
 #if defined(ARCHITECTURE_X64) && !defined(MAP_32BIT)
     // This OS has no flag to enforce allocation below the 4 GB boundary,
     // but if we hint that we want a low address it is very likely we will
@@ -49,9 +49,6 @@ void* AllocateExecutableMemory(size_t size, bool low)
         , -1, 0);
 #endif /* defined(_WIN32) */
 
-    // printf("Mapped executable memory at %p (size %ld)\n", ptr,
-    //    (unsigned long)size);
-
 #ifdef _WIN32
     if (ptr == nullptr)
     {
@@ -69,7 +66,6 @@ void* AllocateExecutableMemory(size_t size, bool low)
         {
             map_hint += size;
             map_hint = (char*)round_page(map_hint); /* round up to the next page */
-            // printf("Next map will (hopefully) be at %p\n", map_hint);
         }
     }
 #endif
@@ -85,18 +81,15 @@ void* AllocateExecutableMemory(size_t size, bool low)
 void* AllocateMemoryPages(size_t size)
 {
 #ifdef _WIN32
-    void* ptr = VirtualAlloc(0, size, MEM_COMMIT, PAGE_READWRITE);
+    void* ptr = VirtualAlloc(nullptr, size, MEM_COMMIT, PAGE_READWRITE);
 #else
-    void* ptr = mmap(0, size, PROT_READ | PROT_WRITE,
+    void* ptr = mmap(nullptr, size, PROT_READ | PROT_WRITE,
             MAP_ANON | MAP_PRIVATE, -1, 0);
 
     if (ptr == MAP_FAILED)
         ptr = nullptr;
 #endif
 
-    // printf("Mapped memory at %p (size %ld)\n", ptr,
-    //    (unsigned long)size);
-
     if (ptr == nullptr)
         LOG_ERROR(Common_Memory, "Failed to allocate raw memory");
 
@@ -117,9 +110,6 @@ void* AllocateAlignedMemory(size_t size,size_t alignment)
 #endif
 #endif
 
-    // printf("Mapped memory at %p (size %ld)\n", ptr,
-    //    (unsigned long)size);
-
     if (ptr == nullptr)
         LOG_ERROR(Common_Memory, "Failed to allocate aligned memory");
 
@@ -131,11 +121,8 @@ void FreeMemoryPages(void* ptr, size_t size)
     if (ptr)
     {
 #ifdef _WIN32
-
         if (!VirtualFree(ptr, 0, MEM_RELEASE))
             LOG_ERROR(Common_Memory, "FreeMemoryPages failed!\n%s", GetLastErrorMsg());
-        ptr = nullptr; // Is this our responsibility?
-
 #else
         munmap(ptr, size);
 #endif

src/core/core_timing.cpp

@@ -207,7 +207,7 @@ void ScheduleEvent_Threadsafe(s64 cycles_into_future, int event_type, u64 userda
     Event* new_event = GetNewTsEvent();
     new_event->time = GetTicks() + cycles_into_future;
     new_event->type = event_type;
-    new_event->next = 0;
+    new_event->next = nullptr;
     new_event->userdata = userdata;
     if (!ts_first)
         ts_first = new_event;