Lines 665~667 of src/library/tools/R/QC.R have the following code:
ind <- grep(.S4_method_markup_regexp, functions)
functions <- functions[!ind]
The use of "any()" and "!ind" indicates that "ind" should be a logical vector. Hence grep() should be replaced by grepl().
You are right.
It may be that this code is never really called .. which would be strange.
If it *has* been used in the past, I do expect that _less_ functions are removed (from the list, in that code) and potentially new "codoc problems" should be found.
I'm committing a fix to R-devel only for now.
Thank you Martin. I remember that I detected this problem because at that time `R CMD check` gave me warnings when I wrote a class that had a plus sign in the class name, and the Rd files were not properly recognized.