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

Fixed compilation error on function "conn_escape" on Visual Studio 2008. #13

Merged
merged 5 commits into from Mar 22, 2016

Conversation

echiesse
Copy link
Contributor

Now variable "to" is dynamically allocated to avoid a compilation error when trying to allocate an array on the stack with a non constant expression which is not supported by all C standards.

…" no visual studio 2008.

Alocação da variável "to" passa a ser dinâmica para evitar erro de compilação ao usar alocação na pilha (array) com expressão não constante.
Now variable "to" is dynamically allocated to avoid a compilation error when trying to allocate an array on the stack with a non constant expression which is not supported by all C standards.
@hishamhm
Copy link
Member

Github's "Filed changed" tab may be fooling me, but from the looks of it the indentation seems incorrect, isn't it?

This was referenced Mar 21, 2016
@echiesse
Copy link
Contributor Author

Hi Hisham, please check the correction.
In the middle of the discution of "out of memory safety" I really didn't realize your comment.

hishamhm added a commit that referenced this pull request Mar 22, 2016
Fixed compilation error on function "conn_escape" on Visual Studio 2008.
@hishamhm hishamhm merged commit 25eebb2 into lunarmodules:master Mar 22, 2016
@hishamhm
Copy link
Member

Merged, thanks!

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

Successfully merging this pull request may close these issues.

None yet

2 participants