Палю годноту или OpenGL готов для десктопа

0

0

Тем, кто использует OpenGL, clang 3.2 и функции glVertexPointer/glNormalPointer, предлагаю заценить фишку. Вставьте это где-нибудь после включения <GL/gl.h> или <GL/glew.h> в том файле, где используется glVertexPointer или glNormalPointer

#if defined(__has_attribute)
extern void GLAPIENTRY glNormalPointer (GLenum type, GLsizei stride, const GLvoid *pointer)
        __attribute__((pointer_with_type_tag(opengl, 3, 1)));
extern void GLAPIENTRY glVertexPointer (GLint size, GLenum type, GLsizei stride, const GLvoid *pointer)
__attribute__((pointer_with_type_tag(opengl, 4, 2)));

GLenum opengl_predefined_t_double
__attribute__(( type_tag_for_datatype(opengl,const double) )) = GL_DOUBLE;
GLenum opengl_predefined_t_float
__attribute__(( type_tag_for_datatype(opengl,const float) )) = GL_FLOAT;
GLenum opengl_predefined_t_unsigned_byte
__attribute__(( type_tag_for_datatype(opengl,const unsigned char) )) = GL_UNSIGNED_BYTE;
GLenum opengl_predefined_t_unsigned_int
__attribute__(( type_tag_for_datatype(opengl,const unsigned int) )) = GL_UNSIGNED_INT;

#undef GL_DOUBLE
#undef GL_FLOAT
#undef GL_UNSIGNED_BYTE
#undef GL_UNSIGNED_INT
#define GL_DOUBLE opengl_predefined_t_double
#define GL_FLOAT opengl_predefined_t_float
#define GL_UNSIGNED_BYTE opengl_predefined_t_unsigned_byte
#define GL_UNSIGNED_INT opengl_predefined_t_unsigned_int

#endif

А потом попробуйте заменить работающий...

glNormalPointer(GL_DOUBLE, 0, m_normales.front());

...на заведомо бажный...

glNormalPointer(GL_FLOAT, 0, m_normales.front());

...и скомпилировать. В данном случае m_normales — массив векторов, а вектор имеет оператор приведения к типу к const double *.

Ссылка

←	Вышла новая минорная версия планировщика Fcron.

Gentoo: portconf-1.5.4

→

и что будет?
//вообще мимо проходил

~~Reinar~~ ★
(16.03.13 00:04:15 MSK)

Ответ на: комментарий от Reinar 16.03.13 00:04:15 MSK

AppDelegate.cpp:122: warning: argument type 'const double *' doesn't match specified 'opengl' type tag that requires 'const float *' [-Wtype-safety]
    glNormalPointer(GL_FLOAT, 0, m_normales.front());
                    ~~~~~~~~     ^~~~~~~~~~~~~~~~~~

quiet_readonly ★★★★
(16.03.13 00:06:38 MSK) автор топика

Ответ на: комментарий от Reinar 16.03.13 00:04:15 MSK

Запустится half-life 3. o.O

hope13 ★★★
(16.03.13 00:07:02 MSK)

Ссылка

Ответ на: комментарий от quiet_readonly 16.03.13 00:06:38 MSK

хм... я видимо чего то не понимаю в шлангюморе

x0r ★★★★★
(16.03.13 00:15:37 MSK)

Ответ на: комментарий от x0r 16.03.13 00:15:37 MSK

Это типобезопасность медленными шагами пробирается в C++.

note173 ★★★★★
(16.03.13 00:18:37 MSK)

Прикольно. Почти как в Ivy.

tailgunner ★★★★★
(16.03.13 00:19:18 MSK)

Ссылка

Ответ на: комментарий от note173 16.03.13 00:18:37 MSK

ну это не серьезно, весь фан теряется. к томуже typedef вертанет этот ворнинг на банане на раз-два.

quiet_readonly а проверь с typedef плз

x0r ★★★★★
(16.03.13 00:21:36 MSK)

Ответ на: комментарий от x0r 16.03.13 00:21:36 MSK

Варнинга нет

    typedef const float arr[3];
    arr array = {1, 2, 3};
    glVertexPointer(3, GL_FLOAT, 0, array);

Варнинг есть

    typedef const float arr[3];
    arr array = {1, 2, 3};
    glVertexPointer(3, GL_DOUBLE, 0, array);

AppDelegate.cpp:123: warning: argument type 'const float *' doesn't match specified 'opengl' type tag that requires 'const double *' [-Wtype-safety]
    glVertexPointer(3, GL_DOUBLE, 0, array);
                       ~~~~~~~~~     ^~~~~

quiet_readonly ★★★★
(16.03.13 00:31:37 MSK) автор топика

Ссылка

Ответ на: комментарий от quiet_readonly 16.03.13 00:06:38 MSK

это успех или фэйл?

BambarbiyaKirgudu ☆
(16.03.13 00:50:18 MSK)

Ссылка

А вообще — это же такие костыли. По стилю похоже на скалу, где на каждую задачу свой DSL изобретают.

note173 ★★★★★
(16.03.13 00:54:40 MSK)

Я не понимаю, вам код надо писать, или портить существующий?

#define TRUE FALSE

~~DELIRIUM~~ ☆☆☆☆☆
(16.03.13 01:50:20 MSK)

Ответ на: комментарий от note173 16.03.13 00:54:40 MSK

По стилю похоже на скалу, где на каждую задачу свой DSL изобретают.
По стилю похоже на руби, где на каждую задачу свой DSL изобретают.

удвоил

ZuBB ★★★★★
(16.03.13 01:51:35 MSK)

Ссылка

Хренотень какая-то. Если хочется проверки типов, достаточно использовать подобные обертки:

static inline void GL_VertexFloatPointer(GLint size, GLsizei stride, const GLfloat *pointer)
{   
    glVertexPointer(size, GL_FLOAT, sizeof(GLfloat) * stride, pointer); 
}

static inline void GL_VertexDoublePointer(GLint size, GLsizei stride, const GLdouble *pointer)
{   
    glVertexPointer(size, GL_DOUBLE, sizeof(GLdouble) * stride, pointer); 
}

Работать будет в любом компиляторе и не нужно ломать заголовочные файлы.

Deleted
(16.03.13 04:35:01 MSK)

Ответ на: комментарий от Deleted 16.03.13 04:35:01 MSK

Работать будет в любом компиляторе и не нужно ломать заголовочные файлы.

Одна беда — надо переписывать существующие исходники. Предлагаемое clang'ом расширение позволяет один раз добавить аттрибуты в системном заголовке и проверять типы всегда.

К сожалению, пока только две реализации MPI используют эту фичу. Попробую в будущем сделать патчик для GLEW хотя бы.

quiet_readonly ★★★★
(16.03.13 09:44:46 MSK) автор топика