Skip to content

Commit

Permalink
Merge pull request #73 from doronz88/bugfix/objc-monitor
Browse files Browse the repository at this point in the history
symbols_jar: make `monitor()` use `$argX` instead (#72)
  • Loading branch information
doronz88 authored Jul 22, 2024
2 parents f2b1bb2 + 4a80269 commit fcab630
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions hilda/symbols_jar.py
Original file line number Diff line number Diff line change
Expand Up @@ -139,15 +139,15 @@ def monitor(self, **args):
:param args: given arguments for monitor command
"""
for name, address in self.items():
args_c = args.copy()
options = args.copy()
if name == '_client':
continue
if self.__dict__['_client'].configs.objc_verbose_monitor:
arg_count = name.count(':')
if arg_count > 0:
args_c['regs'] = {f'x{i + 2}': 'po' for i in range(arg_count)}
name = args_c.get('name', name)
address.monitor(name=name, **args_c)
options['expr'] = {f'$arg{i + 3}': 'po' for i in range(arg_count)}
name = options.get('name', name)
address.monitor(name=name, **options)

def startswith(self, exp, case_sensitive=True):
"""
Expand Down

0 comments on commit fcab630

Please sign in to comment.