Closed Bug 1702012 Opened 5 years ago Closed 4 years ago

move symbolic-utils functions to new module

Categories

(Eliot :: General, task, P3)

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: willkg, Assigned: willkg)

References

Details

Attachments

(1 file)

The symbolicate_resource.py module has a lot of stuff in it and some of it would be better in a separate module of Breakpad SYM and symbolic utility functions.

  • bytes_split_generator
  • get_module_filename
  • converting from a debug_filename to the sym filename
  • converting from sym to symcache
  • converting from a Lineinfo to a filepath

Probably some other things, too.

Assignee: nobody → willkg
Status: NEW → ASSIGNED

willkg merged PR #2396: "bug 1702012: split out symbolic wrapper" in cff2c20.

This is in the eliot-dev branch. After codefreeze is over, I'll merge it to main.

One note is that I didn't move the lineinfo -> data frame code over. The shape of the data frame is specific to the API version, so it seemed like that belongs in the view and not the wrapper.

This merged to main and deployed to stage so we're good here.

Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Component: Symbolication → General
Product: Tecken → Eliot
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: