Skip to content

when read fails due to exceeding FISH_READ_BYTE_LIMIT should the var be defined? #4180

Closed
@krader1961

Description

@krader1961

While working on making our unit tests compatible with issue #4163 I noticed that read x does not define x if the read fails because too much data was read. This wasn't noticed by the unit tests because they relied on string length "$x" to report zero if the var is empty or undefined. It seems to me the var should be defined but with no elements, not even a single empty string, when the read fails in this manner.

Does anyone have a different opinion?

Metadata

Metadata

Assignees

Labels

bugSomething that's not working as intended

Type

No type

Projects

No projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions