esgraph icon indicating copy to clipboard operation
esgraph copied to clipboard

Added option to omit excpetion edges in CFG

Open ksons opened this issue 11 years ago • 5 comments

Pass { omitExceptions: true } as second argument to esgraph ControlFlowGraph

It's usefull if excpetions should not be considered, either by the analysis or just for display purposes.

ksons avatar Oct 25 '13 06:10 ksons

Coverage Status

Coverage remained the same when pulling 5384f3aa630d6a7bc4822f2415de7a7e4d4d76e1 on ksons:master into a26fd60701351e31fc324715098bcb71d4a61925 on Swatinem:master.

coveralls avatar Oct 25 '13 06:10 coveralls

Coverage Status

Coverage remained the same when pulling ec20bf90fb9b13ee215aff53e2cccf450f7f74c9 on ksons:master into a26fd60701351e31fc324715098bcb71d4a61925 on Swatinem:master.

coveralls avatar Oct 25 '13 07:10 coveralls

Thanks a lot for taking an interest in this project! :-)

It would be nice to have some tests for this feature. Lets make the testing framework recognize an additional comment with a JSON options in there.

Also, I wanted to avoid pulling in escodegen as dependency. dot should take the original source as label if you have range tracking enabled. Isn’t that working for you?

Swatinem avatar Oct 25 '13 12:10 Swatinem

Hi,

actually I didn't want to pull that commit, it was done automatically (do I have to close the pull request to avoid that further commits end up in the pull request? I see the point having another dependency.

ksons avatar Oct 25 '13 12:10 ksons

If you want you can open another one with added tests :-)

Swatinem avatar Oct 25 '13 12:10 Swatinem