Small changes...

* Suggest import when a function is not found.
* gcd.py better error message
* info frame: change field name

Author: rocky

test/example/gcd.py

@@ -13,7 +13,7 @@
 def check_args():
     if len(sys.argv) != 3:
         # Rather than use sys.exit let's just raise an error
-        raise Exception("Need to give two numbers")
+        raise Exception(f"Need to give two numbers; got: {sys.argv}")
     for i in range(2):
         try:
             sys.argv[i+1] = int(sys.argv[i+1])

trepan/processor/command/info_subcmd/frame.py

@@ -154,9 +154,9 @@ def run(self, args):
         if f_lasti >= 0:
             opname = opc.opname[code.co_code[f_lasti]]
             opname_formatted = format_token(Keyword, opname, style=style)
-            self.msg(f"  last instruction: {frame.f_lasti} {opname_formatted}")
+            self.msg(f"  instruction offset and opname: {frame.f_lasti} {opname_formatted}")
         else:
-            self.msg(f"  last instruction: {frame.f_lasti}")
+            self.msg(f"  instruction offset: {frame.f_lasti}")
 
         self.msg(f"  code: {format_code(code, style)}")
         if frame.f_back:

trepan/processor/location.py

@@ -79,11 +79,16 @@ def resolve_location(proc, location) -> Optional[Location]:
                     return INVALID_LOCATION
 
         if mod_func is None:
-            msg = f"Object {location_method} is not known yet as a function, "
+            # [1] DRY similar code [2] below
+            msg = f"Object {location_method} is not known yet as a function."
             try:
                 mod_func = eval(location_method, g, locals_dict)
             except Exception:
                 proc.errmsg(msg)
+                split_names = location_method.split(".")
+                if len(split_names) > 1:
+                    proc.msg(f'Try importing {".".join(split_names[:-1])}?')
+
                 return INVALID_LOCATION
 
             try:
@@ -202,10 +207,14 @@ def resolve_address_location(proc, location) -> Optional[Location]:
     if location.method:
         # Validate arguments that can't be done in parsing
         filename = offset = None
-        msg = f"Object {location.method} is not known yet as a function, "
+        msg = f"Object {location.method} is not known yet as a function."
+        # [2] DRY simlar code above [1]
         try:
             mod_func = eval(location.method, g, locals_dict)
         except Exception:
+            split_names = location_method.split(".")
+            if len(split_names) > 1:
+                proc.msg(f'Try importing {".".join(split_names[:-1])}?')
             proc.errmsg(msg)
             return INVALID_LOCATION