Commit 5556eec
authored
Merge pull request #4037 from jmackay2:fix_clahe_build
Fix clahe build
### Pull Request Readiness Checklist
Fix the build related to this [PR](opencv/opencv#28014). This inherits from CLAHE, so the `setBitShift` and `getBitShift` functions need to be defined.
Error this fixes:
```
error: invalid new-expression of abstract class type '{anonymous}::CLAHE_Impl'
opencv/contrib/modules/cudaimgproc/src/histogram.cpp:150:11: note: because the following virtual functions are pure within '{anonymous}::CLAHE_Impl':
class CLAHE_Impl : public cv::cuda::CLAHE
In file included from opencv_contrib/modules/cudaimgproc/include/opencv2/cudaimgproc.hpp:51
opencv/modules/imgproc/include/opencv2/imgproc.hpp:1074:26: note: 'virtual void cv::CLAHE::setBitShift(int)'
CV_WRAP virtual void setBitShift(int bitShift) = 0;
opencv/modules/imgproc/include/opencv2/imgproc.hpp:1080:25: note: 'virtual int cv::CLAHE::getBitShift() const'
CV_WRAP virtual int getBitShift() const = 0;
```
This change was tested on Ubuntu 24.04 with the latest updates from the opencv and opencv_contrib repositories. This resulted in the build being successful.
- [x] I agree to contribute to the project under Apache 2 License.
- [x] To the best of my knowledge, the proposed patch is not based on a code under GPL or another license that is incompatible with OpenCV
- [x] The PR is proposed to the proper branch
- [x] There is a reference to the original bug report and related work
- [x] There is accuracy test, performance test and test data in opencv_extra repository, if applicable
Patch to opencv_extra has the same branch name.
- [x] The feature is well documented and sample code can be built with the project CMake1 parent 6a0c82a commit 5556eec
1 file changed
+12
-0
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
64 | 64 | | |
65 | 65 | | |
66 | 66 | | |
| 67 | + | |
| 68 | + | |
67 | 69 | | |
68 | 70 | | |
69 | 71 | | |
| |||
163 | 165 | | |
164 | 166 | | |
165 | 167 | | |
| 168 | + | |
| 169 | + | |
| 170 | + | |
| 171 | + | |
| 172 | + | |
| 173 | + | |
| 174 | + | |
| 175 | + | |
| 176 | + | |
| 177 | + | |
166 | 178 | | |
167 | 179 | | |
168 | 180 | | |
| |||
0 commit comments