diff options
| author | Tor Lillqvist <tml@iki.fi> | 2011-07-23 16:54:08 +0300 |
|---|---|---|
| committer | Tor Lillqvist <tml@iki.fi> | 2011-07-23 17:00:11 +0300 |
| commit | 8622956e24a178637cf3d6d8e84188ebfd3500d4 (patch) | |
| tree | fb062d7c0679372afba67633455580d5f42566ce /vcl | |
| parent | d3deac524bffaf5811a6ea4b9a4e7ac6acf7913c (diff) | |
Bin unused symbolic constants
Is it really good Mac style programming practice to define your own
non-namespaced kThisAndThat constants that look like they might be
defined by some platform API?
And seriously, defining symbolic constants for small integers like
const kOneBit = 1 ? These were not used, but if they had been would
it really be cleaner to use the identifier "kOneBit" instead of a
plain number 1?
Cases where small integers have magic meaning and don't stand for just
themselves are obviously different; there it makes sense to use
symbolic names and not the integer values. But in the case of things
like kOneBit that is not the case, as one might guesss from the name
it was supposed to be used to indicate a bit depth of one. Etc.
Diffstat (limited to 'vcl')
| -rw-r--r-- | vcl/inc/aqua/salconst.h | 25 |
1 files changed, 0 insertions, 25 deletions
diff --git a/vcl/inc/aqua/salconst.h b/vcl/inc/aqua/salconst.h index 50971a819c09..87bdbf42d5e0 100644 --- a/vcl/inc/aqua/salconst.h +++ b/vcl/inc/aqua/salconst.h @@ -33,23 +33,6 @@ // - Constants - // ------------------- -static const unsigned short kByteMask = 0xFF; - -static const unsigned short kOneByte = 8; -static const unsigned short kTwoBytes = 16; - -static const unsigned short kOneBit = 1; -static const unsigned short kFiveBits = 5; -static const unsigned short kEightBits = 8; -static const unsigned short kTenBits = 10; -static const unsigned short kElevenBits = 11; - -static const unsigned short kBlackAndWhite = 1; -static const unsigned short kFourBitColor = 4; -static const unsigned short kEightBitColor = 8; -static const unsigned short kThousandsColor = 16; -static const unsigned short kTrueColor = 32; - static const unsigned long k16BitRedColorMask = 0x00007c00; static const unsigned long k16BitGreenColorMask = 0x000003e0; static const unsigned long k16BitBlueColorMask = 0x0000001f; @@ -58,14 +41,6 @@ static const unsigned long k32BitRedColorMask = 0x00ff0000; static const unsigned long k32BitGreenColorMask = 0x0000ff00; static const unsigned long k32BitBlueColorMask = 0x000000ff; -static const unsigned short kPixMapCmpSizeOneBit = 1; -static const unsigned short kPixMapCmpSizeFourBits = 4; -static const unsigned short kPixMapCmpSizeFiveBits = 5; -static const unsigned short kPixMapCmpSizeEightBits = 8; - -static const long kPixMapHRes = 72; -static const long kPixMapVRes = 72; - #endif // _SV_SALCONST_H /* vim:set shiftwidth=4 softtabstop=4 expandtab: */ |