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

Unneeded %Q Cop doesn't deal with escape codes #1210

Closed
mipearson opened this issue Jul 15, 2014 · 2 comments
Closed

Unneeded %Q Cop doesn't deal with escape codes #1210

mipearson opened this issue Jul 15, 2014 · 2 comments
Assignees
Labels

Comments

@mipearson
Copy link

Example during CSV output testing:

%Q(sep=,\r\nAd,Brand\r\n1,"Tre""k"\r\n)

will trigger the cop, even though writing this will quotes ends up with a backslashed mess:

"sep=,\r\nAd,Brand\r\n1,\"Tre\"\"k\"\r\n"
@mipearson
Copy link
Author

Just realised that this may be a dup of #1112

@jonas054
Copy link
Collaborator

@mipearson You're right. That should not be reported as an offense.

I'll fix it. But I don't think it's a duplicate, so we keep both issues open.

@jonas054 jonas054 self-assigned this Jul 15, 2014
@jonas054 jonas054 added the bug label Jul 15, 2014
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

2 participants