Color.g() - Code Metrics - Inspection of "Add Code Climate integration." - HexDecimal/libtcod-cffi - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 8f28b9...54dd4d )

by Kyle

created 2016-12-31 08:27 UTC

Color.g() A

↳ Parent: Color

Complexity

Conditions

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
c	0
b	0
f	0
dl	0
loc	4
rs	10

"""This module handles loading of the libtcod cffi API.
"""
from __future__ import absolute_import as _

import sys as _sys
import os as _os

import platform as _platform

from tcod import __path__

# add Windows dll's to PATH
if _sys.platform == 'win32':
    _bits, _linkage = _platform.architecture()
    _os.environ['PATH'] += (';' +
        _os.path.join(__path__[0], 'x86/' if _bits == '32bit' else 'x64'))


def _import_library_functions(lib):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    # imports libtcod namespace into thie module
    # does not override existing names
    g = globals()
    for name in dir(lib):
        if name[:5] == 'TCOD_':
            if (isinstance(getattr(lib, name), ffi.CData) and

                ffi.typeof(getattr(lib, name)) == ffi.typeof('TCOD_color_t')):

                g[name[5:]] = Color._new_from_cdata(getattr(lib, name))
            elif name.isupper():
                g[name[5:]] = getattr(lib, name) # const names
            #else:
            #    g[name[5:]] = getattr(lib, name) # function names
        elif name[:6] == 'TCODK_': # key name
            g['KEY_' + name[6:]] = getattr(lib, name)

NOISE_DEFAULT_HURST = 0.5
NOISE_DEFAULT_LACUNARITY = 2.0

def FOV_PERMISSIVE(p) :
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    return FOV_PERMISSIVE_0+p


def BKGND_ALPHA(a):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    return BKGND_ALPH | (int(a * 255) << 8)


def BKGND_ADDALPHA(a):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    return BKGND_ADDA | (int(a * 255) << 8)


class _MockFFI(object):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    def def_extern(self):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        return lambda func:func


if _os.environ.get('READTHEDOCS'):
    # Mock the lib and ffi objects needed to compile docs for readthedocs.io
    # Allows an import without building the cffi module first.
    lib = object()
    ffi = _MockFFI()
    RENDERER_SDL = 2
    FONT_LAYOUT_ASCII_INCOL = 1
    BKGND_SET = 1
    BKGND_DEFAULT = 13
    KEY_RELEASED = 2
    NOISE_DEFAULT = 0
    NOISE_SIMPLEX = 2
else:
    from tcod._libtcod import lib, ffi
# .scrutinizer.yml
before_commands:
    - sudo pip install abc # Python2
    - sudo pip3 install abc # Python3

class Color(list):
    """

    Args:
        r (int): Red value, from 0 to 255.
        g (int): Green value, from 0 to 255.
        b (int): Blue value, from 0 to 255.
    """

    def __init__(self, r=0, g=0, b=0):
class SomeParent:
    def __init__(self):
        self.x = 1

class SomeChild(SomeParent):
    def __init__(self):
        # Initialize the super class
        SomeParent.__init__(self)
        self[:] = (r & 0xff, g & 0xff, b & 0xff)

    @property
    def r(self):
        """int: Red value, always normalised to 0-255."""
        return self[0]

    @r.setter
    def r(self, value):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        self[0] = value & 0xff

    @property
    def g(self):
        """int: Green value, always normalised to 0-255."""
        return self[1]
    @g.setter
    def g(self, value):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        self[1] = value & 0xff

    @property
    def b(self):
        """int: Blue value, always normalised to 0-255."""
        return self[2]
    @b.setter
    def b(self, value):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        self[2] = value & 0xff

    @classmethod
    def _new_from_cdata(cls, cdata):
        """new in libtcod-cffi"""
        return cls(cdata.r, cdata.g, cdata.b)


    @classmethod
    def _new_from_int(cls, integer):
        """a TDL int color: 0xRRGGBB

        new in libtcod-cffi"""
        return cls(lib.TDL_color_from_int(integer))


    def __eq__(self, other):
        """Compare equality between colors."""
        return (isinstance(other, (Color)) and
                lib.TCOD_color_equals(self, other))


    def __add__(self, other):
        """Add two colors together."""
        return Color._new_from_cdata(lib.TCOD_color_add(self, other))


    def __sub__(self, other):
        """Subtract one color from another."""
        return Color._new_from_cdata(lib.TCOD_color_subtract(self, other))


    def __mul__(self, other):
        """Multiply with a scaler or another color."""
        if isinstance(other, (Color, list, tuple)):
            return Color._new_from_cdata(lib.TCOD_color_multiply(self, other))

        else:
            return Color._new_from_cdata(
                lib.TCOD_color_multiply_scalar(self, other))


    def __bytes__(self):
        """Return this color in a format suited for color control."""
        return b'%c%c%c' % tuple(self)

    def __str__(self):
        """Return this color in a format suited for color control."""
        return '%c%c%c' % tuple(self)

    def __repr__(self):
        """Return a printable representation of the current color."""
        return "%s(%i,%i,%i)" % (self.__class__.__name__,
                                 self.r, self.g, self.b)

    def __int__(self):
        """Return this color as an integer in 0xRRGGBB format."""
        return lib.TDL_color_RGB(*self)



_import_library_functions(lib)

__all__ = [_name for _name in list(globals()) if _name[0] != '_']


1			"""This module handles loading of the libtcod cffi API.
2			"""
3			from __future__ import absolute_import as _
4
5			import sys as _sys
6			import os as _os
7
8			import platform as _platform
9
10			from tcod import __path__
11
12			# add Windows dll's to PATH
13			if _sys.platform == 'win32':
14			_bits, _linkage = _platform.architecture()
15			_os.environ['PATH'] += (';' +
16			_os.path.join(__path__[0], 'x86/' if _bits == '32bit' else 'x64'))
17
18
19			def _import_library_functions(lib):
			0 ignored issues – show Coding Style introduced 2016-10-14 03:53 UTC by Report Bug Copy Issue Report This function should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history... Comprehensibility Bug introduced 2016-10-14 03:53 UTC by Report Bug Copy Issue Report `lib` is re-defining a name which is already available in the outer-scope (previously defined on line `54`). It is generally a bad practice to shadow variables from the outer-scope. In most cases, this is done unintentionally and might lead to unexpected behavior: param = 5 class Foo: def __init__(self, param): # "param" would be flagged here self.param = param Loading history...
20			# imports libtcod namespace into thie module
21			# does not override existing names
22			g = globals()
23			for name in dir(lib):
24			if name[:5] == 'TCOD_':
25			if (isinstance(getattr(lib, name), ffi.CData) and
			0 ignored issues – show Bug introduced 2016-10-16 23:42 UTC by Report Bug Copy Issue Report The Instance of `_MockFFI` does not seem to have a member named `CData`. This check looks for calls to members that are non-existent. These calls will fail. The member could have been renamed or removed. Loading history...
26			ffi.typeof(getattr(lib, name)) == ffi.typeof('TCOD_color_t')):
			0 ignored issues – show Bug introduced 2016-10-16 23:42 UTC by Report Bug Copy Issue Report The Instance of `_MockFFI` does not seem to have a member named `typeof`. This check looks for calls to members that are non-existent. These calls will fail. The member could have been renamed or removed. Loading history...
27			g[name[5:]] = Color._new_from_cdata(getattr(lib, name))
28			elif name.isupper():
29			g[name[5:]] = getattr(lib, name) # const names
30			#else:
31			# g[name[5:]] = getattr(lib, name) # function names
32			elif name[:6] == 'TCODK_': # key name
33			g['KEY_' + name[6:]] = getattr(lib, name)
34
35			NOISE_DEFAULT_HURST = 0.5
36			NOISE_DEFAULT_LACUNARITY = 2.0
37
38			def FOV_PERMISSIVE(p) :
			0 ignored issues – show Coding Style introduced 2016-10-14 03:53 UTC by Report Bug Copy Issue Report No space allowed before : def FOV_PERMISSIVE(p) : ^ Loading history... Coding Style introduced 2016-10-14 03:53 UTC by Report Bug Copy Issue Report This function should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history...
39			return FOV_PERMISSIVE_0+p
			0 ignored issues – show Comprehensibility Best Practice introduced 2016-10-14 03:53 UTC by Report Bug Copy Issue Report Undefined variable 'FOV_PERMISSIVE_0' Loading history...
40
41			def BKGND_ALPHA(a):
			0 ignored issues – show Coding Style introduced 2016-10-14 03:53 UTC by Report Bug Copy Issue Report This function should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history...
42			return BKGND_ALPH \| (int(a * 255) << 8)
			0 ignored issues – show Comprehensibility Best Practice introduced 2016-10-14 03:53 UTC by Report Bug Copy Issue Report Undefined variable 'BKGND_ALPH' Loading history...
43
44			def BKGND_ADDALPHA(a):
			0 ignored issues – show Coding Style introduced 2016-10-14 03:53 UTC by Report Bug Copy Issue Report This function should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history...
45			return BKGND_ADDA \| (int(a * 255) << 8)
			0 ignored issues – show Comprehensibility Best Practice introduced 2016-10-14 03:53 UTC by Report Bug Copy Issue Report Undefined variable 'BKGND_ADDA' Loading history...
46
47			class _MockFFI(object):
			0 ignored issues – show Coding Style introduced 2016-10-16 23:42 UTC by Report Bug Copy Issue Report This class should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history...
48			def def_extern(self):
			0 ignored issues – show Coding Style introduced 2016-10-16 23:42 UTC by Report Bug Copy Issue Report This method should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history... Coding Style introduced 2016-10-16 23:42 UTC by Report Bug Copy Issue Report This method could be written as a function/class method. If a method does not access any attributes of the class, it could also be implemented as a function or static method. This can help improve readability. For example class Foo: def some_method(self, x, y): return x + y; could be written as class Foo: @classmethod def some_method(cls, x, y): return x + y; Loading history...
49			return lambda func:func
			0 ignored issues – show Coding Style introduced 2016-10-16 23:42 UTC by Report Bug Copy Issue Report Exactly one space required after : return lambda func:func ^ Loading history...
50
51			if _os.environ.get('READTHEDOCS'):
52			# Mock the lib and ffi objects needed to compile docs for readthedocs.io
53			# Allows an import without building the cffi module first.
54			lib = object()
55			ffi = _MockFFI()
56			RENDERER_SDL = 2
57			FONT_LAYOUT_ASCII_INCOL = 1
58			BKGND_SET = 1
59			BKGND_DEFAULT = 13
60			KEY_RELEASED = 2
61			NOISE_DEFAULT = 0
62			NOISE_SIMPLEX = 2
63			else:
64			from tcod._libtcod import lib, ffi
			0 ignored issues – show Bug introduced 2016-10-14 03:53 UTC by Report Bug Copy Issue Report The name `_libtcod` does not seem to exist in module `tcod`. Loading history... Configuration introduced 2016-10-14 03:53 UTC by Report Bug Copy Issue Report The import `tcod._libtcod` could not be resolved. This can be caused by one of the following: 1. Missing Dependencies This error could indicate a configuration issue of Pylint. Make sure that your libraries are available by adding the necessary commands. # .scrutinizer.yml before_commands: - sudo pip install abc # Python2 - sudo pip3 install abc # Python3 Tip: We are currently not using virtualenv to run pylint, when installing your modules make sure to use the command for the correct version. 2. Missing __init__.py files This error could also result from missing `__init__.py` files in your module folders. Make sure that you place one file in each sub-folder. Loading history...
65
66			class Color(list):
67			"""
68
69			Args:
70			r (int): Red value, from 0 to 255.
71			g (int): Green value, from 0 to 255.
72			b (int): Blue value, from 0 to 255.
73			"""
74
75			def __init__(self, r=0, g=0, b=0):
			0 ignored issues – show Bug introduced 2016-10-25 03:04 UTC by Report Bug Copy Issue Report The `__init__` method of the super-class `list` is not called. It is generally advisable to initialize the super-class by calling its `__init__` method: class SomeParent: def __init__(self): self.x = 1 class SomeChild(SomeParent): def __init__(self): # Initialize the super class SomeParent.__init__(self) Loading history...
76			self[:] = (r & 0xff, g & 0xff, b & 0xff)
77
78			@property
79			def r(self):
80			"""int: Red value, always normalised to 0-255."""
81			return self[0]
82
83			@r.setter
84			def r(self, value):
			0 ignored issues – show Coding Style introduced 2016-10-25 03:04 UTC by Report Bug Copy Issue Report This method should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history...
85			self[0] = value & 0xff
86
87			@property
88			def g(self):
89			"""int: Green value, always normalised to 0-255."""
90			return self[1]
91			@g.setter
92			def g(self, value):
			0 ignored issues – show Coding Style introduced 2016-10-25 03:04 UTC by Report Bug Copy Issue Report This method should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history...
93			self[1] = value & 0xff
94
95			@property
96			def b(self):
97			"""int: Blue value, always normalised to 0-255."""
98			return self[2]
99			@b.setter
100			def b(self, value):
			0 ignored issues – show Coding Style introduced 2016-10-25 03:04 UTC by Report Bug Copy Issue Report This method should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history...
101			self[2] = value & 0xff
102
103			@classmethod
104			def _new_from_cdata(cls, cdata):
105			"""new in libtcod-cffi"""
106			return cls(cdata.r, cdata.g, cdata.b)
107
108
109			@classmethod
110			def _new_from_int(cls, integer):
111			"""a TDL int color: 0xRRGGBB
112
113			new in libtcod-cffi"""
114			return cls(lib.TDL_color_from_int(integer))
			0 ignored issues – show Bug introduced 2016-10-16 23:09 UTC by Report Bug Copy Issue Report The Instance of `object` does not seem to have a member named `TDL_color_from_int`. This check looks for calls to members that are non-existent. These calls will fail. The member could have been renamed or removed. Loading history...
115
116			def __eq__(self, other):
117			"""Compare equality between colors."""
118			return (isinstance(other, (Color)) and
119			lib.TCOD_color_equals(self, other))
			0 ignored issues – show Bug introduced 2016-10-25 03:04 UTC by Report Bug Copy Issue Report The Instance of `object` does not seem to have a member named `TCOD_color_equals`. This check looks for calls to members that are non-existent. These calls will fail. The member could have been renamed or removed. Loading history...
120
121			def __add__(self, other):
122			"""Add two colors together."""
123			return Color._new_from_cdata(lib.TCOD_color_add(self, other))
			0 ignored issues – show Bug introduced 2016-10-25 03:04 UTC by Report Bug Copy Issue Report The Instance of `object` does not seem to have a member named `TCOD_color_add`. This check looks for calls to members that are non-existent. These calls will fail. The member could have been renamed or removed. Loading history...
124
125			def __sub__(self, other):
126			"""Subtract one color from another."""
127			return Color._new_from_cdata(lib.TCOD_color_subtract(self, other))
			0 ignored issues – show Bug introduced 2016-10-25 03:04 UTC by Report Bug Copy Issue Report The Instance of `object` does not seem to have a member named `TCOD_color_subtract`. This check looks for calls to members that are non-existent. These calls will fail. The member could have been renamed or removed. Loading history...
128
129			def __mul__(self, other):
130			"""Multiply with a scaler or another color."""
131			if isinstance(other, (Color, list, tuple)):
132			return Color._new_from_cdata(lib.TCOD_color_multiply(self, other))
			0 ignored issues – show Bug introduced 2016-10-25 03:04 UTC by Report Bug Copy Issue Report The Instance of `object` does not seem to have a member named `TCOD_color_multiply`. This check looks for calls to members that are non-existent. These calls will fail. The member could have been renamed or removed. Loading history...
133			else:
134			return Color._new_from_cdata(
135			lib.TCOD_color_multiply_scalar(self, other))
			0 ignored issues – show Bug introduced 2016-10-31 12:29 UTC by Report Bug Copy Issue Report The Instance of `object` does not seem to have a member named `TCOD_color_multiply_scalar`. This check looks for calls to members that are non-existent. These calls will fail. The member could have been renamed or removed. Loading history...
136
137			def __bytes__(self):
138			"""Return this color in a format suited for color control."""
139			return b'%c%c%c' % tuple(self)
140
141			def __str__(self):
142			"""Return this color in a format suited for color control."""
143			return '%c%c%c' % tuple(self)
144
145			def __repr__(self):
146			"""Return a printable representation of the current color."""
147			return "%s(%i,%i,%i)" % (self.__class__.__name__,
148			self.r, self.g, self.b)
149
150			def __int__(self):
151			"""Return this color as an integer in 0xRRGGBB format."""
152			return lib.TDL_color_RGB(*self)
			0 ignored issues – show Bug introduced 2016-10-16 23:09 UTC by Report Bug Copy Issue Report The Instance of `object` does not seem to have a member named `TDL_color_RGB`. This check looks for calls to members that are non-existent. These calls will fail. The member could have been renamed or removed. Loading history...
153
154
155			_import_library_functions(lib)
156
157			__all__ = [_name for _name in list(globals()) if _name[0] != '_']
158

HexDecimal / libtcod-cffi

Push — master ( 8f28b9...54dd4d )

Color.g() A

Complexity

Size

Duplication

Importance

1. Missing Dependencies

2. Missing __init__.py files

Duplication Side-by-Side

Filter issues like

2. Missing init.py files