-
Notifications
You must be signed in to change notification settings - Fork 7
Profiler #1
Copy link
Copy link
Open
Description
@secabstraction, I've scanned through the code and have few questions. The Get-ProcessTrace does not parse correctly right now - is everything committed? I would also like to put the PSReflect module as a dependency module (as it's done in PowershellArsenal) and remove it from the project code. Also, I think it would be nice to split the code of Get-StackTrace and Get-ProcessTrace among several files - > 1000 lines of code is extremely hard to support and develop. What do you think? I could start working on that in my fork, but would like to first discuss the changes with you. Also, what is your plan for the profiler? I guess the current goal is to make Get-ProcessTrace work, parsing both managed and native stacks, am I right?
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels