CPD - filter out files

A source code analyzer

Brought to you by: adangel, juansotuyo

#272 CPD - filter out files

Milestone: Unstable_(example)

Status: closed

Owner: Romain PELISSE

Labels: pmd (28)

Priority: 5

Updated: 2013-09-29

Created: 2012-03-05

Creator: Delmas

Private: No

See https://sourceforge.net/projects/pmd/forums/forum/188192/topic/5083509 for more information.

Note the use of getAbsolutePath() in order to properly exclude the kind of construct:
--file src/ --exclude pwd/src/subfolder

Discussion

Delmas - 2012-03-05

exclude_file.diff

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

mikolg - 2012-03-13

Believe it would be more useful if exclude filter supports regular expression or file mask.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Delmas - 2012-03-13

The method used is the same that is used to process the "--file" option. I do not think it would make sense to do it differently, for consistency
Another option would be to support regexp/globs for both the --file and --exclude options.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Delmas - 2012-03-21

Hi,

I tend to disagree. The mechanism is the same as for the --file option, so enhancing one of the two should benefit to the other option as well, in my opinion. Thus, this proposal belongs to a different change.

Cheers,
Remi

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Jan Rüegg - 2013-08-21

Is this patch now in the source code? If not, whats missing?

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Andreas Dangel - 2013-09-29

Well, what's missing? Bascially only time to adapt this patch to the current code.
It's now applied and will be included with the next 5.1 release.
Please let me know, if you wanted to have it in a maintenance release - but I tend not to add features to maintenance release...

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Andreas Dangel - 2013-09-29

status: open --> closed

Group: --> Unstable_(example)
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Log in to post a comment.