[all-commits] [llvm/llvm-project] a69bbe: [LLDB][breakpoints] ArgInfo::count -> ArgInfo::max...

[smoofra via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: a69bbe02a2352271e8b14542073f177e24c499c1
      https://github.com/llvm/llvm-project/commit/a69bbe02a2352271e8b14542073f177e24c499c1
  Author: Lawrence D'Anna <lawrence_danna at apple.com>
  Date:   2019-10-29 (Tue, 29 Oct 2019)

  Changed paths:
    M lldb/include/lldb/Interpreter/ScriptInterpreter.h
    M lldb/packages/Python/lldbsuite/test/functionalities/breakpoint/breakpoint_command/TestBreakpointCommandsFromPython.py
    M lldb/scripts/Python/python-wrapper.swig
    M lldb/source/Plugins/ScriptInterpreter/Python/ScriptInterpreterPython.cpp
    M lldb/source/Plugins/ScriptInterpreter/Python/ScriptInterpreterPythonImpl.h
    M lldb/unittests/ScriptInterpreter/Python/PythonTestSuite.cpp

  Log Message:
  -----------
  [LLDB][breakpoints] ArgInfo::count -> ArgInfo::max_positional_args

Summary:
Move breakpoints from the old, bad ArgInfo::count to the new, better
ArgInfo::max_positional_args.   Soon ArgInfo::count will be no more.

It looks like this functionality is already well tested by
`TestBreakpointCommandsFromPython.py`, so there's no need to write
additional tests for it.

Reviewers: labath, jingham, JDevlieghere

Reviewed By: labath

Subscribers: lldb-commits

Tags: #lldb

Differential Revision: https://reviews.llvm.org/D69468