Re: [Lldb-commits] [PATCH] Add const versions of SBFrame::GetFunctionName and SBFrame::IsInlined.

2015-06-25 Thread Pavel Labath
lgtm. Potentially we can leave a note to the future maintainer to remove the 
non-const versions, when/if we decide to make a breaking API change.


http://reviews.llvm.org/D10711

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/



___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits


Re: [Lldb-commits] [PATCH] Add const versions of SBFrame::GetFunctionName and SBFrame::IsInlined.

2015-06-25 Thread Oleksiy Vyalov
AFFECTED FILES

  /lldb/trunk/include/lldb/API/SBFrame.h
  /lldb/trunk/scripts/interface/SBFrame.i
  /lldb/trunk/source/API/SBFrame.cpp

USERS

  ovyalov (Author)

http://reviews.llvm.org/rL240660


http://reviews.llvm.org/D10711

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/



___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits


Re: [Lldb-commits] [PATCH] Add const versions of SBFrame::GetFunctionName and SBFrame::IsInlined.

2015-06-24 Thread Jim Ingham
That seems fine.

Jim

 On Jun 24, 2015, at 2:05 PM, Oleksiy Vyalov ovya...@google.com wrote:
 
 Hi jingham, labath,
 
 Add const versions of SBFrame::GetFunctionName and SBFrame::IsInlined.
 
 http://reviews.llvm.org/D10711
 
 Files:
  include/lldb/API/SBFrame.h
  scripts/interface/SBFrame.i
  source/API/SBFrame.cpp
 
 Index: include/lldb/API/SBFrame.h
 ===
 --- include/lldb/API/SBFrame.h
 +++ include/lldb/API/SBFrame.h
 @@ -91,12 +91,18 @@
 const char *
 GetFunctionName();
 
 +const char *
 +GetFunctionName() const;
 +
 /// Return true if this frame represents an inlined function.
 ///
 /// See also GetFunctionName().
 bool
 IsInlined();
 -
 +
 +bool
 +IsInlined() const;
 +
 /// The version that doesn't supply a 'use_dynamic' value will use the
 /// target's default.
 lldb::SBValue
 Index: scripts/interface/SBFrame.i
 ===
 --- scripts/interface/SBFrame.i
 +++ scripts/interface/SBFrame.i
 @@ -128,14 +128,20 @@
 const char *
 GetFunctionName();
 
 +const char *
 +GetFunctionName() const;
 +
 %feature(docstring, 
 /// Return true if this frame represents an inlined function.
 ///
 /// See also GetFunctionName().
 ) IsInlined;
 bool
 IsInlined();
 -
 +
 +bool
 +IsInlined() const;
 +
 %feature(docstring, 
 /// The version that doesn't supply a 'use_dynamic' value will use the
 /// target's default.
 Index: source/API/SBFrame.cpp
 ===
 --- source/API/SBFrame.cpp
 +++ source/API/SBFrame.cpp
 @@ -1501,6 +1501,12 @@
 bool
 SBFrame::IsInlined()
 {
 +return static_castconst SBFrame*(this)-IsInlined();
 +}
 +
 +bool
 +SBFrame::IsInlined() const
 +{
 Log *log(GetLogIfAllCategoriesSet (LIBLLDB_LOG_API));
 ExecutionContext exe_ctx(m_opaque_sp.get());
 StackFrame *frame = NULL;
 @@ -1538,6 +1544,12 @@
 const char *
 SBFrame::GetFunctionName()
 {
 +return static_castconst SBFrame*(this)-GetFunctionName();
 +}
 +
 +const char *
 +SBFrame::GetFunctionName() const
 +{
 Log *log(lldb_private::GetLogIfAllCategoriesSet (LIBLLDB_LOG_API));
 const char *name = NULL;
 ExecutionContext exe_ctx(m_opaque_sp.get());
 @@ -1590,4 +1602,3 @@
 }
 return name;
 }
 -
 
 EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
 D10711.28400.patch

___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits


Re: [Lldb-commits] [PATCH] Add const versions of SBFrame::GetFunctionName and SBFrame::IsInlined.

2015-06-24 Thread Jim Ingham
That seems fine.

Jim


http://reviews.llvm.org/D10711

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/



___
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits