verilog_systemverilog.vim icon indicating copy to clipboard operation
verilog_systemverilog.vim copied to clipboard

Add support for folding of OVM/UVM macros

Open jrudess opened this issue 7 years ago • 3 comments

This commit adds a new folding option 'ovm' which will fold ovm/uvm macro constructs that have a begin/end

I originally wanted to name the new dictionary entry 'verif' since it encompasses both OVM and UVM, but for some reason the dictionary entry would not work when I named it either 'verif' or 'uvm'. But naming it 'ovm' does work. I can't explain that. 'ovm' isn't a terrible name but it's a deprecated library, so it's not ideal.

Let me know if you would like additions to the test cases to cover this change.

jrudess avatar Aug 02 '18 01:08 jrudess

Would you mind including a test case in test/folding.v?

vhda avatar Aug 12 '18 18:08 vhda

Don't you want to update, rebase on top of master and force push this branch?

vhda avatar Aug 20 '18 07:08 vhda

Haven't forgotten this one, I just need to add the test case. I only haven't done that yet because I wanted to understand the test suite setup a little better before I started modfying it, and I also got massively sidetracked learning errorformat. :-)

I have another minor PR (randsequence support) to submit which will also benefit from some added cases.

jrudess avatar Aug 21 '18 01:08 jrudess