Add SetExceptionBreakpoint Optional Args

[blakekrpec]

In nvim-dap, set_exception_breakpoints() exposes one of two optional args in the DAP spec (ExceptionOptions), and it is always nil. This PR adds the other optional arg (ExceptionFilterOptions) and exposes both optional args in defaults.

This allows for non nil definitions of both optional args in the default table if needed/desired, otherwise they will default to nil.

Tested with debugpy, codelldb, and the Unity debugger.

[Erifirin]

Oh yeah! 'filterOptions' is required for Unity debugger. It would be great to merge this PR.

[mfussenegger]

Oh yeah! 'filterOptions' is required for Unity debugger. It would be great to merge this PR.

It will be some time before I get to this, because I'll want to make sure the API fits with what I have in mind for #1388 and also see how to deal with #494 . Before that I won't be making any other public API changes to these methods.

In the meantime you could bully ask the billion $ company to fix their adapter to comply with the specification. filterOptions are supposed to be optional

[Erifirin]

@mfussenegger Yes, I understand. You are absolutely right. For now, I will continue to modify session.lua locally. Maybe this issue will be solved in the future.

[blakekrpec]

It will be some time before I get to this, because I'll want to make sure the API fits with what I have in mind for #1388 and also see how to deal with #494 . Before that I won't be making any other public API changes to these methods.

Sounds good. I am happy to come back and refactor this PR is need be after you make any major changes for #1388 and/or #494.

In the meantime you could bully ask the billion $ company to fix their adapter to comply with the specification. filterOptions are supposed to be optional

Agreed, they should adhere to the spec. I tried their support avenues, and never heard back. I assume I never will, but was worth a shot...

[blakekrpec]

I will continue to modify session.lua locally. Maybe this issue will be solved in the future.

@Erifirin I have my fork public and plan to keep it up while these changes aren't in nvim-dap. Feel free to use it if it saves you time over manually updating session.lua. Cheers.

[Erifirin]

In the meantime you could bully ask the billion $ company to fix their adapter to comply with the specification. filterOptions are supposed to be optional

I did so. At least they promised to fix the bug in the next version of visualstudiotoolsforunity.vstuc.