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

Inital port for linux #2

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Inital port for linux #2

wants to merge 8 commits into from

Conversation

tyabus
Copy link

@tyabus tyabus commented May 31, 2019

No description provided.

@@ -454,7 +454,7 @@ int Q_vsnprintf( char *buffer, size_t buffersize, const char *format, va_list ar
{
size_t result;

result = _vsnprintf( buffer, buffersize, format, args );
result = vsnprintf( buffer, buffersize, format, args );
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Лучше такое в #ifdef _WIN32 оборачивать.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

я знаю, но там всего лишь warning

Copy link
Member

@nekonomicon nekonomicon May 31, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Под msvc6?
Это расширение MS.
vsnprintf в С99 ввели.
Можно обойтись дефайном же.

#ifndef _WIN32
#define _vsnprintf vsnprintf
#endif // _WIN32

или опцию сборки добавить -D_vsnprintf=vsnprintf.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Это win32-only варнинг

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Интересно, какая реакция будет на warning о пропущенном виртуальном деструкторе или вызове delete без квадратных скобок для динамического массива?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Интересно, какая реакция будет на warning о пропущенном виртуальном деструкторе или вызове delete без квадратных скобок для динамического массива?

https://youtu.be/1ExYXiRuG6o

Android.mk Outdated Show resolved Hide resolved
@a1batross
Copy link
Member

Вообще у меня уже давно порт был. Просто я его не выклвдывал, да и он чутка грязноват -- я начал делать VBO рендерер по просьбе ДМ и так не осилил доделать.

Android.mk Outdated Show resolved Hide resolved
@tyabus
Copy link
Author

tyabus commented May 31, 2019

Done.

@a1batross
Copy link
Member

pyc закидывать не надо. Это файлы создаваемые питоном в процессе работы. Удали и в .gitignore.

@tyabus
Copy link
Author

tyabus commented Jun 1, 2019

pyc закидывать не надо. Это файлы создаваемые питоном в процессе работы. Удали и в .gitignore.

Я уже удалил, щас закину в .gitignore.

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.

3 participants