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

save to image #21

Merged
merged 1 commit into from
Jul 23, 2012
Merged

save to image #21

merged 1 commit into from
Jul 23, 2012

Conversation

dk
Copy link
Contributor

@dk dk commented Jul 23, 2012

Hi David,

i've fixed a minor thing with regard to saving to an image, i hope it makes sense

/dk

run4flat added a commit that referenced this pull request Jul 23, 2012
Fixes the save-to-image behavior to do what I always meant it to do, but couldn't figure out how to do!
@run4flat run4flat merged commit b4ca102 into PDLPorters:master Jul 23, 2012
@run4flat
Copy link
Member

Dmitry -

This is fantastic, and it lays the groundwork for copy-to-clipboard
behavior on Macs, which before just put a black rectangle on the
clipboard. For some reason, it's not working on my mac yet, but it's
much close to working, I think.

Low-hanging fruit, but very important! Thanks!
David

On Mon, Jul 23, 2012 at 5:27 AM, Dmitry Karasik
reply@reply.github.com
wrote:

Hi David,

i've fixed a minor thing with regard to saving to an image, i hope it makes sense

/dk

You can merge this Pull Request by running:

git pull https://github.com/dk/PDL-Graphics-Prima master

Or you can view, comment on it, or merge it online at:

#21

-- Commit Summary --

  • - minor optimizations

-- File Changes --

M lib/PDL/Graphics/Prima.pm (24)

-- Patch Links --

https://github.com/run4flat/PDL-Graphics-Prima/pull/21.patch
https://github.com/run4flat/PDL-Graphics-Prima/pull/21.diff


Reply to this email directly or view it on GitHub:
#21

"Debugging is twice as hard as writing the code in the first place.
Therefore, if you write the code as cleverly as possible, you are,
by definition, not smart enough to debug it." -- Brian Kernighan

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants