[lldb] Stop generating swig bindings for SBLaunchInfo copy constructor

[bulbazord]

Summary

Given the line

launch_info = lldb.SBLaunchInfo(None)

We see different behaviors across different versionf of swig. On some
older versions of swig (e.g. 3.0.2) this line fails because it attempts
to use the copy constructor and blows up with an invalid null reference.
On newer versions of swig, this is correctly routed to the constructor
taking a pointer.

Prior to generating the swig bindings with the API headers,
SBLaunchInfo's copy constructor was not exposed so we're effectively
going back to the old behavior anyway.

[pranavk]

I tested this and this does fix the issue with older swig version. Thanks for working on it.

[saugustine]

Thanks for the quick fix.

lldb/include/lldb/API/SBLaunchInfo.h

 class SBTarget;
 
 class LLDB_API SBLaunchInfo {
 public:
   SBLaunchInfo(const char **argv);
 
   ~SBLaunchInfo();
 
+#ifndef SWIG
+  // The copy constructor for SBLaunchInfo presents some problems on some
+  // supported versions of swig (e.g. 3.0.2). When trying to create an
+  // SBLaunchInfo from python with the argument `None`, swig will try to call
+  // the copy constructor instead of SBLaunchInfo(const char **). For that
+  // reason, we avoid exposing the copy constructor to python.
   SBLaunchInfo(const SBLaunchInfo &rhs);
+#endif
 
   SBLaunchInfo &operator=(const SBLaunchInfo &rhs);
 
   lldb::pid_t GetProcessID();
 
   uint32_t GetUserID();
 
   uint32_t GetGroupID();