Trevor Smith

Maybe we could add one or the other as a context menu item? "Copy as Plain Text" or "Copy as Diff" perhaps.

I agree. If the heuristic fails even 1% of the time it would be confusing and frustrating. As for the label, I haven been able to think of anything better...

I do like this idea as an additional feature. For this use case, however - manually cherry picking a few random lines - having to jump to another file is...

I just discovered that the Stage view already implements a Copy button that copies lines without the diff markers.

How often are you needing to copy lines with the markers, and where are you using them? It's currently not possible to copy the entire patch from the stage view,...

I like that solution. It is less bitty, but using a GUI is arguably not very gritty either. So my vote is for the markerless diffs, but I'm open to...

I don't have very deep knowledge of `Domain`, but now that you're not attaching an error listener is the domain necessary? We too were (unintentionally) relying on the previous behavior....

I've created a PR to fix this issue: #264

I just ran across this by chance while searching for a related Redshift error. The current_schema error can be resolved by changing `select current_schema` to `select current_schema()`. Redshift thinks it...