[lldb-dev] Change for Review: warning cleanup in Linux ProcessMonitor

[Michael Sartain]

Minor warnings cleanup. Thanks.
 -Mike

mikesart at mikesart-rad:~/data/src/llvm.svn/llvm/tools/lldb$ svn diff
source/Plugins/Process/Linux/ProcessMonitor.cpp
Index: source/Plugins/Process/Linux/ProcessMonitor.cpp
===================================================================
--- source/Plugins/Process/Linux/ProcessMonitor.cpp     (revision 190896)
+++ source/Plugins/Process/Linux/ProcessMonitor.cpp     (working copy)
@@ -1610,14 +1610,14 @@
 {
     Log *log (ProcessPOSIXLog::GetLogIfAllCategoriesSet
(POSIX_LOG_PROCESS));
     if (log)
-        log->Printf ("ProcessMonitor::%s(%d) waiting for thread to
stop...", __FUNCTION__, tid);
+        log->Printf ("ProcessMonitor::%s(%" PRIu64 ") waiting for thread
to stop...", __FUNCTION__, tid);

     // Wait for the thread to stop
     while (true)
     {
         int status = -1;
         if (log)
-            log->Printf ("ProcessMonitor::%s(%d) waitpid...",
__FUNCTION__, tid);
+            log->Printf ("ProcessMonitor::%s(%" PRIu64 ") waitpid...",
__FUNCTION__, tid);
         lldb::pid_t wait_pid = waitpid(tid, &status, __WALL);
         if (status == -1)
         {
@@ -1628,13 +1628,13 @@
             else
             {
                 if (log)
-                    log->Printf("ProcessMonitor::%s(%d) waitpid error --
%s", __FUNCTION__, tid, strerror(errno));
+                    log->Printf("ProcessMonitor::%s(%" PRIu64 ") waitpid
error -- %s", __FUNCTION__, tid, strerror(errno));
                 return false; // This is bad, but there's nothing we can
do.
             }
         }

         if (log)
-            log->Printf ("ProcessMonitor::%s(%d) waitpid, status = %d",
__FUNCTION__, tid, status);
+            log->Printf ("ProcessMonitor::%s(%" PRIu64 ") waitpid, status
= %d", __FUNCTION__, tid, status);

         assert(wait_pid == tid);

@@ -1753,6 +1753,7 @@

         switch (message.GetKind())
         {
+            case ProcessMessage::eAttachMessage:
             case ProcessMessage::eInvalidMessage:
                 break;

@@ -1971,8 +1972,6 @@
 void
 ProcessMonitor::ServeOperation(OperationArgs *args)
 {
-    int status;
-
     ProcessMonitor *monitor = args->m_monitor;

     // We are finised with the arguments and are ready to go.  Sync with
the
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-dev/attachments/20130917/d3320371/attachment.html>