Yoann Padioleau

Results 358 comments of Yoann Padioleau

I see. You're right it's incorrect, but semgrep is not trying to be a syntax checker/type checker/interpreter, so for matching purpose, the transformation is fine. The only issue is that...

Oh sorry, I thought I passed to semgrep a JS files in my command above. I read it as misc_bang.js, not rs. Hmm then maybe we should not allow it...

Note that I used -e '$X == $X' -l js, so there's just one language here (not python).

I think when we run semgrep on 0 files, we should try to help the user, to explain why so (the explanation on what is skipped is useful, but here...

Did you manage to fix the segfault issue?

or to know where did it come from?

need to merge again to resolve the conflict

I've fixed the CI error in the pfff PR. You should be able to use the latest pfff now and it should work.

Can you rerun with --debug and copy-paste the output here?