VC++ 2010 не даёт описать переменную в середине блока — ограничение Express-версии?

Question

[DWZ]

C++ из Visual Studio 2010 Express. Собираю обычный проект (безо всяких afx) https://github.com/gpac/gpac/

И тут VC++ неожиданно начинает ругаться на описание переменных в середине блока. То есть, если описать переменные сразу после "{", то всё нормально. Но если между перед описанием переменной что-нибудь вычисляется, то VC++ начинает ругаться. На типы, определённые через typedef, ругается, что их использование в выражениях недопустимо, а на встроенные и вовсе, что перед описанием не стоит ";", хотя она стоит. Переношу описание переменной в начало блока - и всё в порядке.

Несколько сервис-паков поставил - не помогло.

Это что, новый стандарт? Или ограничение Express-версии? Или где-то можно что-то включить, чтобы всё стало нормально?

Comments

[Станислав Макаров]

А фрагмент кода можно? Как можно создать такой вопрос и описывать ситуация словами, вместо того чтобы привести фрагмент кода?)

[Adamos]

Или где-то можно что-то включить, чтобы всё стало нормально?

Мозг. Где-то в голове.
Очевидно, что при дурных проблемах с кодом в Студии в первую очередь стоит искать проблемы в коде, а не в Студии.

[DWZ]

Adamos, Вас понял - дурачок написал свой ошибочный код и по глупости своей винит Студию, а надо бы по-хорошему, винить себя. Я Вас правильно понял?

Вот только код не мой, а солидной фирмы. И под Винду они его как-то скомпилировали. Так что Ваш совет не катит.

[Adamos]

DWZ, они его не только как-то скомпилировали, но и положили в том же репозитории инструкцию по компиляции, а в INSTALLME дали ссылку на статью по сборке в 2010 Студии...

[#]

DWZ,

... Я Вас правильно понял?

думаю абсолютно точно поняли

[DWZ]

Станислав Макаров, пример 1

// очень много всего
if (do_all)
        import.flags |= GF_IMPORT_KEEP_REFS;

    u32 nb_tracks = gf_isom_get_track_count(dest);

ругается на строчку с u32

c:\users\user\documents\visual studio 2010\projects\gpac-master\applications\mp4box\fileimport.c(797): error C2275: u32: недопустимое использование этого типа в качестве выражения

убираем "u32", оставляем только

nb_tracks = gf_isom_get_track_count(dest);

идём наверх, где описываются переменные и ставим там
u32 nb_tracks;

главное поставить до первого оператора присваивания - и ошибка исчезает.

Пример ошибки со встроенным типом сразу не найду, там всё аналогично, только ругается на ";"

[Армянское Радио]

DWZ, проблема в том, что истинная причина 95% ора на тему ААААА, ОШИБКА В КОНПЕЛЯТРЕ - это кривые руки источника ора - неправильные настройки, неправильный код и прочее. Нужно понимать, что у компилятора за плечами сотни тысяч часов эксплуатации, у большинства библиотек - тоже, а конкретно ваш код с вашими настройками сборки собираете только вы. Так что вероятность ошибки "кривые руки" - 95%

[DWZ]

Adamos, так я уже всё скомпилировал. Мне главное, вопрос закрыть, чтобы в следующий раз так не уродоваться.

но и положили в том же репозитории инструкцию по компиляции, а в INSTALLME дали ссылку на статью по сборке в 2010 Студии.

Так я ей и воспользовался. Кстати, exctra_lib днём с огнём разыскивать пришлось. В extra_lib только include, а в download.tsi.telecom-paristech.fr/gpac/gpac_extra_... где они говорят нужно брать библиотеки - ошибка 404. Но я справился. К вопросу о кривых руках. (Рассказ Булычёва вспомнил, где мальчик лодочный мотор собирал)

[DWZ]

Армянское Радио, Ещё раз. По пунктам:
1. Это
2. не
3. мой
4. код

[Армянское Радио]

DWZ, причину неправильные настройки вы не увидели.

[DWZ]

Армянское Радио, Возможно. Но настройки тоже не мои. Говорят - бери 2010-й VC++, взял 2010-й, правда Express. Ничего нигде не менял - и на вот тебе.

Answer 1

[Adamos]

Возможность определять переменные в середине блока появилась в С стандарта 99.
Visual Studio 2010 поддерживает только С 98. Независимо от редакции.

Answer 2

[Армянское Радио]

Похоже на поведение старой-старой версии языка C. Однозначно могу сказать, что 2010 студия с переменными посреди блока работает нормально.

Comments

[DWZ]

Да у меня 2008 была, тоже Express, кстати, всегда так делал и не морочился. А тут прямо озадачился. Кстати, у Вас 2010 Express или какая? А то, может, попытаетесь откомпилировать mp4box из https://github.com/gpac/gpac/ ?

Интересно, будет ли ругаться у Вас?

[Евгений Шатунов]

DWZ, ты лучше покажи точное место, где у тебя сейчас студия ругается.
Неизвестный непонятный код ни кто собирать на своей машине не станет.

[DWZ]

Евгений Шатунов,

gpac-master\applications\mp4box\fileimport.c(797): error C2275: u32: недопустимое использование этого типа в качестве выражения

Неизвестный непонятный код ни кто собирать на своей машине не станет.

Запускать - да. И то существуют всякие виртуалки и Acronis на чёрный день. А от того, что кто-то C в .obj скомпилирует, ничего страшного на его машине не произойдёт, я думаю.

[Евгений Шатунов]

DWZ, ну вот всё и просто.
fileimport.c собирается как C Source Code, стандарт языка C у тебя, кажется, никак не определен и берется по умолчанию - C99. Однако, т.к. у тебя MSVS2010, по умолчанию у тебя код собирается в стандарте C98. 2010 студия не поддерживает C99.
Тег C++ в вопросе стоит неправильно.

ничего страшного на его машине не произойдёт

Это ты или надеешься на некомпетентность людей, или просто не знаешь о том, как вредоносное ПО встраивается в сценарии сборки. В обоих случаях настаивать не стоит.

[DWZ]

При переключении в режим "Компилировать как С++ (/TP)" куча ошибок по приведению типов

>c:\distrib\gpac-master\applications\mp4box\fileimport.c(86): error C2440: =: невозможно преобразовать "void *" в "char *"
1>          Для преобразования "void*" к указателю на тип, не являющемуся "void", требуется явное приведение
1>c:\distrib\gpac-master\applications\mp4box\fileimport.c(142): error C2440: =: невозможно преобразовать "int" в "Bool"
1>          Для преобразования в перечисляемый тип требуется явное приведение (static_cast, приведение в стиле С или приведение в стиле функции)

К тому же 32-битная версия отказалась обрабатывать файл > 4 Гб.

Вывод: Express 2010 - не катит.

[Армянское Радио]

DWZ, к нему можно поставить PlatformSDK 7.1, в котором есть 64 битный компилятор. Но лучше просто перейти на современный инструмент.

[DWZ]

Армянское Радио,

к нему можно поставить PlatformSDK 7.1, в котором есть 64 битный компилятор.

Спасибо, посмотрю.

Но лучше просто перейти на современный инструмент.

Можно-то оно можно, но у меня вопрос - как сами GPAC'овцы это под VC 2010 скомпилировали? Или они выложили это для отвода глаз?

[DWZ]

Армянское Радио, Спасибо, платформа "x64" появилась. Правда, ошибка с описанием переменной посреди блока так и осталась. Хорошо, что уже есть исправленные исходники.

[DWZ]

Армянское Радио, что-то линкер в "x64" на unresolved external symbol стал ругаться и на свои и на системные и все с префиксом __imp_. Для платформы win32 всё нормально.

1>main.obj : error LNK2019: unresolved external symbol __imp_SetConsoleOutputCP referenced in function wmain
1>compositor_2d.obj : error LNK2019: unresolved external symbol __imp_glEnd referenced in function c2d_gl_fill_no_alpha
1>visual_manager_3d_gl.obj : error LNK2001: unresolved external symbol __imp_glEnd
1>compositor_2d.obj : error LNK2019: unresolved external symbol __imp_glVertex2i referenced in function c2d_gl_fill_no_alpha
1>compositor_2d.obj : error LNK2019: unresolved external symbol __imp_glColor3ub referenced in function c2d_gl_fill_no_alpha
1>compositor_2d.obj : error LNK2019: unresolved external symbol __imp_glBegin referenced in function c2d_gl_fill_no_alpha

Как избавиться от этого "__imp_"?

[DWZ]

DWZ, Перешёл на VS 2015. Эта проблема исчезла.

[DWZ]

DWZ,

К тому же 32-битная версия отказалась обрабатывать файл > 4 Гб.

Была ошибка в исходнике. Теперь файл > 4 Гб можно обрабатывать и 32-битной версией

[Армянское Радио]

DWZ, здорово! а как вы ее нашли?

[DWZ]

Армянское Радио, Параллельным запуском под отладчиками двух исходников. Даже вопрос хотел написать про автоматизацию этого дела. Думал, что макросы какие-то не так раскрылись, а оказалось, что

for (i=0; i<import.nb_tracks; i++) {
        import.trackID = import.tk_info[i].track_num;

какой-то диверсант заменил на

nb_tracks = gf_isom_get_track_count(dest);
    for (i=0; i<nb_tracks; i++) {
        import.trackID = import.tk_info[i].track_num;

а gf_isom_get_track_count() возвращает 0, в результате в цикл не заходит и почти ничего в выходной файл не пишется. Сейчас уже откатили.

Answer 3

[res2001]

Посмотрите в настройках проекта какая версия языка используется.
Не скажу на вскидку какие версии поддерживает cl от VS2010, в VS2017 за это отвечает опция /std. В VC2017 эта опция находится в Configuration Properties -> C/C++ -> Language -> C++ Language Standard
Поведение, которое вы описываете, характерно для лохматых версий Си.
Микрософт официально не поддерживает стандарты языка Си в своем компиляторе (хотя стандарты частично реализуются), поэтому в опциях компилятора нельзя это задать.
Если задать опцию /TP, то все файлы исходного кода будут компилироваться как С++ код, не зависимо от расширения файла. По умолчанию, если расширение .c - компилируется как Си код. Возможно эта опция поможет избавиться от ошибки, но есть шанс появления других ошибок. В VC2017 эта опция находится в Configuration Properties -> C/C++ -> Advanced -> Compile As

PS: номер и текст ошибки то же было бы не плохо сюда запостить.

Comments

[DWZ]

Посмотрите в настройках проекта какая версия языка используется

А не подскажете, где именно?

В VC2017 эта опция находится в Configuration Properties -> C/C++ -> Language -> C++ Language Standard

У меня только

PS: номер и текст ошибки то же было бы не плохо сюда запостить.

// очень много всего
if (do_all)
        import.flags |= GF_IMPORT_KEEP_REFS;
    u32 nb_tracks = gf_isom_get_track_count(dest);

ругается на строчку с u32

c:\users\user\documents\visual studio 2010\projects\gpac-master\applications\mp4box\fileimport.c(797): error C2275: u32: недопустимое использование этого типа в качестве выражения

убираем "u32", оставляем только

nb_tracks = gf_isom_get_track_count(dest);

идём наверх, где описываются переменные и ставим там
u32 nb_tracks;

главное поставить до первого оператора присваивания - и ошибка исчезает.

Пример ошибки со встроенным типом сразу не найду, там всё аналогично, только ругается на ";"

[res2001]

DWZ,

А не подскажете, где именно?

Не подскажу. Возможно в VS2010 еще не было этой опции.
Попробуйте вариант с /TP. Другой вариант используйте хотя бы VS2013, а лучше актуальную версию.

[DWZ]

res2001, Самое смешное, что сами авторы, если верить Dependence Walker, использовали VC 2010. Получается, что у них в Гитхабе либо не тот проект, либо не те исходники.

[res2001]

DWZ, Либо они сами собирают не в студии, а из командной строки, используя другой набор параметров компилятора, а не тот что задан в предоставляемом проекте. Выглядит, конечно, довольно странно.