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

Major issue with Graphing #15

Open
nielswh opened this issue Jan 10, 2013 · 3 comments
Open

Major issue with Graphing #15

nielswh opened this issue Jan 10, 2013 · 3 comments

Comments

@nielswh
Copy link

nielswh commented Jan 10, 2013

Hi There,

I ran into an issue with the line plot and brought my app to its knees performance wise. I finally determined where the issue was. Its on line 73 of LineGraphView.m You make a call to
UIGraphicsPushContext(ctx); but then never call UIGraphicsPopContext();
I'm not 100% sure if you need either though.

Thanks,
Niels

@honcheng
Copy link
Owner

So it works fine when u take it out?

Sent from my iPad

On 11 Jan, 2013, at 1:50 AM, Niels Hansen [email protected] wrote:

Hi There,

I ran into an issue with the line plot and brought my app to its knees performance wise. I finally determined where the issue was. Its on line 73 of LineGraphView.m You make a call to
UIGraphicsPushContext(ctx); but then never call UIGraphicsPopContext();
I'm not 100% sure if you need either though.

Thanks,
Niels


Reply to this email directly or view it on GitHub.

@nielswh
Copy link
Author

nielswh commented Jan 11, 2013

I just added UIGraphicsPopContext(); at the end and then it worked great. Need the pop after the push :)

Sent from my iPhone

On Jan 10, 2013, at 4:47 PM, honcheng [email protected] wrote:

So it works fine when u take it out?

Sent from my iPad

On 11 Jan, 2013, at 1:50 AM, Niels Hansen [email protected] wrote:

Hi There,

I ran into an issue with the line plot and brought my app to its knees performance wise. I finally determined where the issue was. Its on line 73 of LineGraphView.m You make a call to
UIGraphicsPushContext(ctx); but then never call UIGraphicsPopContext();
I'm not 100% sure if you need either though.

Thanks,
Niels


Reply to this email directly or view it on GitHub.


Reply to this email directly or view it on GitHub.

@honcheng
Copy link
Owner

Thanks. Can you send me a pull request, just so I can credit you for the fix?

Sent from my iPad

On 11 Jan, 2013, at 8:03 AM, Niels Hansen [email protected] wrote:

I just added UIGraphicsPopContext(); at the end and then it worked great. Need the pop after the push :)

Sent from my iPhone

On Jan 10, 2013, at 4:47 PM, honcheng [email protected] wrote:

So it works fine when u take it out?

Sent from my iPad

On 11 Jan, 2013, at 1:50 AM, Niels Hansen [email protected] wrote:

Hi There,

I ran into an issue with the line plot and brought my app to its knees performance wise. I finally determined where the issue was. Its on line 73 of LineGraphView.m You make a call to
UIGraphicsPushContext(ctx); but then never call UIGraphicsPopContext();
I'm not 100% sure if you need either though.

Thanks,
Niels


Reply to this email directly or view it on GitHub.


Reply to this email directly or view it on GitHub.


Reply to this email directly or view it on GitHub.

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

No branches or pull requests

2 participants