422620 - Remove NS_BrightenColor and NS_DarkenColor

Status: VERIFIED FIXED

(Core Graveyard :: GFX, defect)

Description

[Alfred Kayser]

Attachment: Clean and simple removal of unused functions

Remove NS_BrightenColor and NS_DarkenColor, as they are defined in nsColor, but nowhere used. Also they use a fixed hardcode factor to the adjustment, making them not useful generic functions to keep around.

Comment 1

[Alfred Kayser]

Attachment: V2: Also remove the ASCIIHex functions

Also NS_ASCIIHexToRGB and NS_RGBToASCIIHex are not used anywhere (anymore?)
(see http://lxr.mozilla.org/mozilla/ident?i=NS_ASCIIHexToRGB)
(see http://lxr.mozilla.org/mozilla/ident?i=NS_RGBToASCIIHex)

With this patch, the .obj size is 10K less.

Comment 2

[Damon Sicore (:damons)]

Comment on attachment 309077 [details] [diff] [review]
V2: Also remove the ASCIIHex functions

a1.9+=damons

Comment 3

[Reed Loden [:reed]]

Checking in gfx/public/nsColor.h;
/cvsroot/mozilla/gfx/public/nsColor.h,v  <--  nsColor.h
new revision: 1.26; previous revision: 1.25
done
Checking in gfx/src/nsColor.cpp;
/cvsroot/mozilla/gfx/src/nsColor.cpp,v  <--  nsColor.cpp
new revision: 3.36; previous revision: 3.35
done

Comment 4

[Alfred Kayser]

Verified:
 libgkgfx.so
  	Total:	       -608 (+9/-617)
  	Code:	       -617 (+0/+0)
  	Data:	         +9 (+9/-617)
  	       -608 (+9/-617)	text (DATA)
  		       -608 (+9/-617)	UNDEF:libgkgfx.so:text
  			         +9	.nosyms.text
  			        -19	nsAutoString::~nsAutoString()
  			        -19	NS_ConvertASCIItoUTF16::~NS_ConvertASCIItoUTF16()
  			        -28	nsAutoString::nsAutoString()
  			        -43	nsACString_internal::BeginWriting()
  			        -49	NS_ConvertASCIItoUTF16::NS_ConvertASCIItoUTF16(nsACString_internal const&)
  			        -69	NS_ASCIIHexToRGB(nsCString const&, unsigned int*)
  			        -88	NS_RGBToASCIIHex(unsigned int, nsCString&)
  			       -140	NS_DarkenColor(unsigned int)
  			       -162	NS_BrightenColor(unsigned int)