Commite674941

jcarlosgarciasegovia

authored and

gsnedders

committed

Google Code Issue 215: Properly detect seekable streams

Removes the hack that tests for sys.stdin to determine if the streamis seekable (it has tell() and seek() but it is not seekable) byactually calling tell() and seek().This also removes the wrapping in HTMLUnicodeInputStream, where wedo not need the ability to seek.

1 parent52c37b6 commite674941Copy full SHA for e674941

File tree

1 file changed

-9

lines changed

html5lib
- inputstream.py

1 file changed

-9

lines changed

`‎html5lib/inputstream.py`

Lines changed: 3 additions & 9 deletions

Original file line number	Diff line number	Diff line change
`@@ -3,7 +3,6 @@`
`3`	`3`
`4`	`4`	`importcodecs`
`5`	`5`	`importre`
`6`		`-importsys`
`7`	`6`
`8`	`7`	`from .constantsimportEOF,spaceCharacters,asciiLetters,asciiUppercase`
`9`	`8`	`from .constantsimportencodings,ReparseException`
`@@ -202,12 +201,6 @@ def openStream(self, source):`
`202`	`201`	`else:`
`203`	`202`	`stream=StringIO(source)`
`204`	`203`
`205`		`-if (# not isinstance(stream, BufferedIOBase) and`
`206`		`-not(hasattr(stream,"tell")and`
`207`		`-hasattr(stream,"seek"))or`
`208`		`-streamissys.stdin):`
`209`		`-stream=BufferedStream(stream)`
`210`		`-`
`211`	`204`	`returnstream`
`212`	`205`
`213`	`206`	`def_position(self,offset):`
`@@ -437,8 +430,9 @@ def openStream(self, source):`
`437`	`430`	`else:`
`438`	`431`	`stream=BytesIO(source)`
`439`	`432`
`440`		`-if (not(hasattr(stream,"tell")andhasattr(stream,"seek"))or`
`441`		`-streamissys.stdin):`
	`433`	`+try:`
	`434`	`+stream.seek(stream.tell())`
	`435`	`+except:`
`442`	`436`	`stream=BufferedStream(stream)`
`443`	`437`
`444`	`438`	`returnstream`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commite674941

File tree

1 file changed

1 file changed

`‎html5lib/inputstream.py`

0 commit comments