removed some unnecesary code #39

Merged

davidpanic merged 1 commits from dev into dev 2016-03-26 21:08:34 +00:00

Conversation 3 Commits 1 Files Changed 1 -2

davidpanic commented 2016-03-26 20:59:26 +00:00 (Migrated from github.com)

Copy Link

The below if statement is always true and the value of timeout_on isn't used before a return statement.
timeout_on = False
if timeout_on == False:

The below if statement is always true and the value of timeout_on isn't used before a return statement. timeout_on = False if timeout_on == False:

Dico200 commented 2016-03-27 00:13:01 +00:00 (Migrated from github.com)

Copy Link

there's a bit more wrong with that code
60 => timeout_on <= -2
lol
Well there's no such thing in toolbox right now :)

there's a bit more wrong with that code `60 => timeout_on <= -2` lol Well there's no such thing in toolbox right now :)

jomo commented 2016-03-27 00:15:23 +00:00 (Migrated from github.com)

Copy Link

I assume it should be:

-2 <= timeout_on <= 60

I assume it should be: ``` Python -2 <= timeout_on <= 60 ```

Dico200 commented 2016-03-27 00:15:27 +00:00 (Migrated from github.com)

Copy Link

Looks like the indentation of lines 39-41 is also wrong

Looks like the indentation of lines 39-41 is also wrong

This repo is archived. You cannot comment on pull requests.

Reviewers

No reviewers

Labels

Clear labels   

bug

  

duplicate

  

enhancement

  

help wanted

  

invalid

  

question

  

wontfix

No Label

bug

duplicate

enhancement

help wanted

invalid

question

wontfix

Milestone

Clear milestone

No items

No Milestone

Assignees

Clear assignees

No Assignees

1 Participants

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: Redstoner/redstoner-utils#39

No description provided.