ulfsri/pyAlicat

Eliminate setting "c" to None call

Opened this issue ยท 1 comments

๐Ÿš€ Feature Request

I believe we can eliminate this initial setting of "c" to None before reading. The initial use of this was so that we could break if the read timed out. As long as we return None through the read() function on timeout though we should be able to do without this.

๐Ÿ”ˆ Motivation

Eliminate needless setting of variable.

๐Ÿ›ฐ Alternatives

๐Ÿ“Ž Additional context

image

Nevermind. This may not be true if the read_line() function times out before the read() function. Let's leave as-is for now because it's incredibly minor and maybe revisit later.