throught2 : what did I do wrong

62 views
Skip to first unread message

Roelof Wobben

unread,
Jan 18, 2015, 4:53:45 AM1/18/15
to nod...@googlegroups.com
Hello,

I try to make a function which uppercase a stream by using through2.

So I did this :

~~~

var through = require('through2');
var tr = through(function (chunk, enc,callback) {
    chunk. toString().toUpperCase();
    this.push(chunk);
    callback();
});

process.stdin.pipe(tr).pipe(process.stdout);

~~~

But the output is not uppercased . Where did I do things wrong ?

Roelof

Aria Stewart

unread,
Jan 18, 2015, 10:17:39 AM1/18/15
to nod...@googlegroups.com

> On 18 Jan 2015, at 04:53, Roelof Wobben <rwo...@hotmail.com> wrote:
>
> Hello,
>
> I try to make a function which uppercase a stream by using through2.
>
> So I did this :
>
> ~~~
>
> var through = require('through2');
> var tr = through(function (chunk, enc,callback) {
> chunk. toString().toUpperCase();
> this.push(chunk);

try this.push(chunk.toString().toUpperCase());

String methods in javascript return new strings, rather than modifying the string.

> callback();
> });
>
> process.stdin.pipe(tr).pipe(process.stdout);
>
> ~~~
>
> But the output is not uppercased . Where did I do things wrong ?
>
> Roelof
>
>
> --
> Job board: http://jobs.nodejs.org/
> New group rules: https://gist.github.com/othiym23/9886289#file-moderation-policy-md
> Old group rules: https://github.com/joyent/node/wiki/Mailing-List-Posting-Guidelines
> ---
> You received this message because you are subscribed to the Google Groups "nodejs" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to nodejs+un...@googlegroups.com.
> To post to this group, send email to nod...@googlegroups.com.
> To view this discussion on the web visit https://groups.google.com/d/msgid/nodejs/9486172e-e0b4-4853-a6b5-4d95ca36ce22%40googlegroups.com.
> For more options, visit https://groups.google.com/d/optout.

Roelof Wobben

unread,
Jan 18, 2015, 10:54:49 AM1/18/15
to nod...@googlegroups.com

Thanks, another point learned.

Roelof

Op zondag 18 januari 2015 16:17:39 UTC+1 schreef Aria Stewart:

Shripad K

unread,
Jan 18, 2015, 2:48:24 PM1/18/15
to nodejs
    chunk. toString().toUpperCase(); 

Strings are immutable.
Reply all
Reply to author
Forward
0 new messages