Skip to content

Commit

Permalink
Fix useless use of \E warning
Browse files Browse the repository at this point in the history
Surely I meant \Q.

Fixes #69
  • Loading branch information
semifor committed Jan 2, 2017
1 parent c3be648 commit 7621748
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion t/stringent-encoding.t
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ my $text = q[Bob's your !@##$%^&*(){}} uncle!];
$nt->new_direct_message({ screen_name => 'perl_api', text => $text });

my $encoded_text = Net::OAuth::Message::encode($text);
like $req->content, qr/\E$encoded_text/, 'properly encoded';
like $req->content, qr/\Q$encoded_text/, 'properly encoded';

This comment has been minimized.

Copy link
@toddr

toddr Jan 3, 2017

I think you need a \E at the end of your sequence to tell the regex that \Q is done:

qr/\Q$encoded_text\E/

This comment has been minimized.

Copy link
@semifor

semifor via email Jan 3, 2017

Author Owner

This comment has been minimized.

Copy link
@toddr

toddr Jan 3, 2017

Yeah, you're right. I'm sure I just need to up my OCD medicine :)

This comment has been minimized.

Copy link
@semifor

semifor via email Jan 3, 2017

Author Owner

my $uri = URI->new($req->uri);
$uri->query($req->content);
Expand Down

0 comments on commit 7621748

Please sign in to comment.