New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove BasicFilter.h #19381
Remove BasicFilter.h #19381
Conversation
This file tries to define a "vitrual"[sic] function and wasn't touched in years, so I think we can remove this.
Part of the work going on regarding the C++ modules migration of CMSSW (tracked as issue #15248). This PR is not meant to refactor things, but just to make these headers compile. |
A new Pull Request was created by @Teemperor (Raphael Isemann) for master. It involves the following packages: PhysicsTools/UtilAlgos @cmsbuild, @monttj, @davidlange6 can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
The tests are being triggered in jenkins. |
This file BasicFilter is provided as a skeleton. |
Comparison job queued. |
merge |
Comparison is ready Comparison Summary:
|
Seems like this got already merged (sorry, hard to keep track with all the cmsbuild notifications). @slava77 Shall I make a new PR where I readd this file with the typo fixed? |
sorry I missed @slava77 comment. however EDFilter(Object)Wrapper do not directly use BasicFilter. So i don't see the relevance. |
as I said, I interpreted BasicFilter as a skeleton example. Perhaps I'm wrong and the other two files just have outdated edits referring to BasicFilter. |
This file tries to define a "vitrual"[sic] function and wasn't
touched in years, so I think we can remove this.