parfumix / gource

Automatically exported from code.google.com/p/gource
0 stars 0 forks source link

Improvement to the file-filter Option #126

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago
The file-filter option is OK, but I have over 400 extensions in one of my 
projects, and only want to show around 20 of them. It's extremely difficult to 
write a regex that filters out all but those 20. But, it's child's play to 
write a regex that matches them.

I suggest that the file-filter option be renamed to file-hide-filter, and that 
a new option be added called file-show-filter. So, if gource were run with this 
new option as shown:

gource --file-show-filter 
.+\.(c(c|pp|s(proj)?|xx)?|d(ll|sp)|exe|filters|h(pp)?|i(n(c|l)|pp)|lib|sln|vcx?p
roj)$

all of the following files will be shown: *.c, *.cc, *.cpp, *.cs, *.csproj, 
*.cxx, *.dll, *.dsp, *.exe, *.filters, *.h, *.hpp, *.inc, *.inl, *.ipp, *.lib, 
*.sln, *.vcproj, and *.vcxproj.

Of course, file-filter could be left as is for backward-compatibility. The new 
file-show-filter option would be easy to implement because it's the inverse of 
file-filter. Instead of removing the files that are matched, the files that 
aren't matched are removed.

Just my two cents.

- Jordan

Original issue reported on code.google.com by skoobiedu@gmail.com on 17 Apr 2011 at 4:50

GoogleCodeExporter commented 8 years ago
Sounds reasonably simple to add this.

Cheers

Andrew

Original comment by acaudw...@gmail.com on 18 Apr 2011 at 7:41