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

Double check va_list usage #12

Closed
basvodde opened this issue Aug 11, 2012 · 5 comments
Closed

Double check va_list usage #12

basvodde opened this issue Aug 11, 2012 · 5 comments
Labels

Comments

@basvodde
Copy link
Member

The va_list usage ought to be checked as it has led to some problems in the past.

@arstrube
Copy link
Contributor

Is this still relevant? Exactly what needs to be checked?

@basvodde
Copy link
Member Author

basvodde commented Aug 8, 2014

I'm not sure. What needed to be checked is that is used va_start and va_end properly I believe. It hasn't been a problem for a while though. @jwgrenning did you have an issue with this recently?

@arstrube
Copy link
Contributor

arstrube commented Aug 8, 2014

You mean whether CppUTest uses va_start and va_end properly? I wasn't under the impression that it doesn't?

@jwgrenning
Copy link
Member

I have not done anything with it lately.

On 8 Aug 2014, at 1:20, Bas Vodde wrote:

I'm not sure. What needed to be checked is that is used va_start and
va_end properly I believe. It hasn't been a problem for a while
though. @jwgrenning did you have an issue with this recently?


Reply to this email directly or view it on GitHub:
#12 (comment)

@basvodde
Copy link
Member Author

basvodde commented Aug 9, 2014

Let me then close it. It has caused problems in the past, but not anymore in the last 4 years or so, I guess.

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

3 participants