Make the platform process connect path less chatty.

[sas]

Summary

If a stream contains an empty string, no need to append it to the output
(otherwise we end up with a blank line). Also, no need to print a status
message when the state changes to connected, as this string brings no
information -- "Process 0" does not mean anything to the user, and the
process being connected has no meaning either.

[clayborg]

Fine as long as all tests are passing cleanly.

lldb/trunk/source/Commands/CommandObjectProcess.cpp

DoExecute (Args& launch_args, CommandReturnObject &result)
         Error error = target->Launch(m_options.launch_info, &stream);
         
         if (error.Success())
         {
             const char *archname = exe_module_sp->GetArchitecture().GetArchitectureName();
             ProcessSP process_sp (target->GetProcessSP());
             if (process_sp)
             {
-                if (stream.GetData())
+                const char *data = stream.GetData();
+                if (data && strlen(data) > 0)
                     result.AppendMessage(stream.GetData());
                 result.AppendMessageWithFormat ("Process %" PRIu64 " launched: '%s' (%s)\n", process_sp->GetID(), exe_module_sp->GetFileSpec().GetPath().c_str(), archname);
                 result.SetStatus (eReturnStatusSuccessFinishResult);
                 result.SetDidChangeProcessState (true);
             }
             else
             {
                 result.AppendError("no error returned from Target::Launch, and target has no process");

lldb/trunk/source/Target/Process.cpp

Process::HandleProcessStateChangedEvent (const EventSP &event_sp,
     StateType event_state = Process::ProcessEventData::GetStateFromEvent (event_sp.get());
     if (event_state == eStateInvalid)
         return false;
 
     switch (event_state)
     {
         case eStateInvalid:
         case eStateUnloaded:
-        case eStateConnected:
         case eStateAttaching:
         case eStateLaunching:
         case eStateStepping:
         case eStateDetached:
             {
                 if (stream)
                     stream->Printf ("Process %" PRIu64 " %s\n",
                                     process_sp->GetID(),
                                     StateAsCString (event_state));
 
                 if (event_state == eStateDetached)
                     pop_process_io_handler = true;
             }
             break;
 
+        case eStateConnected:
         case eStateRunning:
             // Don't be chatty when we run...
             break;
 
         case eStateExited:
             if (stream)
                 process_sp->GetStatus(*stream);
             pop_process_io_handler = true;