From c1d39a2fce4baf344e69cd7d32670274d3eaa286 Mon Sep 17 00:00:00 2001
From: Flyte <flyte@failcode.co.uk>
Date: Thu, 18 Feb 2016 16:57:32 +0000
Subject: [PATCH] Remove unnecessary top-level TCP component. Fix order of
 inheritance on TCP BinarySensor.

---
 homeassistant/components/binary_sensor/tcp.py |  5 ++--
 homeassistant/components/sensor/tcp.py        | 21 +++++++++++------
 homeassistant/components/tcp.py               | 23 -------------------
 3 files changed, 16 insertions(+), 33 deletions(-)
 delete mode 100644 homeassistant/components/tcp.py

diff --git a/homeassistant/components/binary_sensor/tcp.py b/homeassistant/components/binary_sensor/tcp.py
index e6d01b893df..1f8bf2387c9 100644
--- a/homeassistant/components/binary_sensor/tcp.py
+++ b/homeassistant/components/binary_sensor/tcp.py
@@ -6,8 +6,7 @@ Provides a binary_sensor which gets its values from a TCP socket.
 import logging
 
 from homeassistant.components.binary_sensor import BinarySensorDevice
-from homeassistant.components.tcp import DOMAIN, CONF_VALUE_ON
-from homeassistant.components.sensor.tcp import Sensor
+from homeassistant.components.sensor.tcp import Sensor, DOMAIN, CONF_VALUE_ON
 
 
 DEPENDENCIES = [DOMAIN]
@@ -22,7 +21,7 @@ def setup_platform(hass, config, add_entities, discovery_info=None):
     add_entities((BinarySensor(hass, config),))
 
 
-class BinarySensor(Sensor, BinarySensorDevice):
+class BinarySensor(BinarySensorDevice, Sensor):
     """ A binary sensor which is on when its state == CONF_VALUE_ON. """
     required = (CONF_VALUE_ON,)
 
diff --git a/homeassistant/components/sensor/tcp.py b/homeassistant/components/sensor/tcp.py
index 29d6c260fdd..ad27ddef0bd 100644
--- a/homeassistant/components/sensor/tcp.py
+++ b/homeassistant/components/sensor/tcp.py
@@ -11,14 +11,22 @@ from homeassistant.const import CONF_NAME, CONF_HOST
 from homeassistant.util import template
 from homeassistant.exceptions import TemplateError
 from homeassistant.helpers.entity import Entity
-from homeassistant.components.tcp import (
-    DOMAIN, CONF_PORT, CONF_TIMEOUT, CONF_PAYLOAD, CONF_UNIT, CONF_VALUE_REGEX,
-    CONF_VALUE_TEMPLATE, CONF_VALUE_ON, CONF_BUFFER_SIZE, DEFAULT_TIMEOUT,
-    DEFAULT_BUFFER_SIZE
-)
 
 
-DEPENDENCIES = [DOMAIN]
+# DEPENDENCIES = [DOMAIN]
+
+DOMAIN = "tcp"
+
+CONF_PORT = "port"
+CONF_TIMEOUT = "timeout"
+CONF_PAYLOAD = "payload"
+CONF_UNIT = "unit"
+CONF_VALUE_TEMPLATE = "value_template"
+CONF_VALUE_ON = "value_on"
+CONF_BUFFER_SIZE = "buffer_size"
+
+DEFAULT_TIMEOUT = 10
+DEFAULT_BUFFER_SIZE = 1024
 
 _LOGGER = logging.getLogger(__name__)
 
@@ -44,7 +52,6 @@ class Sensor(Entity):
             CONF_TIMEOUT: config.get(CONF_TIMEOUT, DEFAULT_TIMEOUT),
             CONF_PAYLOAD: config[CONF_PAYLOAD],
             CONF_UNIT: config.get(CONF_UNIT),
-            CONF_VALUE_REGEX: config.get(CONF_VALUE_REGEX),
             CONF_VALUE_TEMPLATE: config.get(CONF_VALUE_TEMPLATE),
             CONF_VALUE_ON: config.get(CONF_VALUE_ON),
             CONF_BUFFER_SIZE: config.get(
diff --git a/homeassistant/components/tcp.py b/homeassistant/components/tcp.py
deleted file mode 100644
index c82bf935910..00000000000
--- a/homeassistant/components/tcp.py
+++ /dev/null
@@ -1,23 +0,0 @@
-"""
-homeassistant.components.tcp
-~~~~~~~~~~~~~~~~~~~~~~~~~~~~
-A generic TCP socket component.
-"""
-DOMAIN = "tcp"
-
-CONF_PORT = "port"
-CONF_TIMEOUT = "timeout"
-CONF_PAYLOAD = "payload"
-CONF_UNIT = "unit"
-CONF_VALUE_REGEX = "value_regex"
-CONF_VALUE_TEMPLATE = "value_template"
-CONF_VALUE_ON = "value_on"
-CONF_BUFFER_SIZE = "buffer_size"
-
-DEFAULT_TIMEOUT = 10
-DEFAULT_BUFFER_SIZE = 1024
-
-
-def setup(hass, config):
-    """ Nothing to do! """
-    return True