Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add streaming mode to character set conversion #7

Closed
thiloplanz opened this issue Jan 4, 2011 · 12 comments
Closed

Add streaming mode to character set conversion #7

thiloplanz opened this issue Jan 4, 2011 · 12 comments
Labels

Comments

@thiloplanz
Copy link

It would be nice to be able to use iconv in a streaming fashion. This would do away with the need to collect all input data in a (potentially big) buffer and retrieve the output data in another buffer.

With multi-byte character sets, this is probably no easy task, though, as the partial input could stop in the middle of a sequence.

@aklt
Copy link

aklt commented May 30, 2011

+1

@aklt
Copy link

aklt commented May 31, 2011

Have you encountered problems in implementing this?

@bnoordhuis
Copy link
Owner

Not exactly problems but there are a lot of corner cases that need to be taken into account. I've been working on and off on a prototype, I'll try to finish it up this week.

@aklt
Copy link

aklt commented May 31, 2011

Uh, that sounds nice!

I am currently looking into how this and I think that it would make
sense to make node-iconv be a stream, or at least have the same interface as streams.

Curious what you come up with :-)

@papandreou
Copy link

Eagerly awaiting this... :)

@assistunion
Copy link

It would be nice! awaiting this feature :]

@Artazor
Copy link

Artazor commented Apr 22, 2012

+1

@magnumripper
Copy link

I too would appreciate this a lot. I exercised my swap file for a good while before I realised what was the problem.

@lanzz
Copy link

lanzz commented Oct 15, 2012

+1

@superfeedr
Copy link

That'd be indeed very very good! +1

@noahadams
Copy link

+1

@bnoordhuis
Copy link
Owner

Fixed in 1ae031e.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

9 participants