search

NRajkumar / pdfium

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

CPDF_ICCBasedCS::GetDefaultValue() doesn't override and is unused. #170

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 
Looks to be a signature mismatch between:
CPDF_ICCBasedCS::GetDefaultValue(int i, FX_FLOAT& min, FX_FLOAT& max) const
CPDF_ColorSpace::GetDefaultValue(int iComponent, FX_FLOAT& value, FX_FLOAT& 
min, FX_FLOAT& max) const

Which was noticed while adding some "overrides".  Not clear what the default 
value should be or if fixing this will break anything.

Original issue reported on code.google.com by tsepez@chromium.org on 13 Jun 2015 at 12:32

GoogleCodeExporter commented 9 years ago 
Hey Jun, can you ping your developers and see if they have an opinion.  Thanks.

Original comment by tsepez@chromium.org on 13 Jun 2015 at 12:37

GoogleCodeExporter commented 9 years ago 
Same with CPDF_IndexedCS::GetDefaultValue()

Original comment by tsepez@chromium.org on 13 Jun 2015 at 12:38