search

AdamNiederer / cov

An emacs extension for displaying coverage data on your code
GNU General Public License v3.0
76 stars 16 forks source link

Better multifile coverage handling #18

Closed xendk closed 6 years ago

xendk commented 6 years ago

Bit all over the place, this PR:

Cleaned up documentation strings, updated readme, simplified the parser interface (no need to pass the buffer as argument as it's already current, and supply the filename as a buffer local variable, as not all parsers need it), documented it, implemented a global storage og coverage, fixed coverage for files in sub-directories, implemented a simple file time check for reloading coverage when file changes and added updating of overlays in all buffers that used the reloaded coverage.

Now I just need to implement inotify watching of coverage files, and purging of unused coverage data, and we're in a pretty sweet spot. Maybe an cov-mode-if-found function so one can just add that to the mode-hook of supported languages.

AdamNiederer commented 6 years ago

Hey, thanks again for the patches. Looks like the travis failure is still spurious. Want me to merge this in now or wait for everything to land?

xendk commented 6 years ago

Yeah, looks like some TLS trouble.

You can just merge if you're happy about it.

AdamNiederer commented 6 years ago

This looks good; thanks again!