Можно ли подавить проверку на GOTO через объявление переменной в g++?

[Sashok]

Мне необходимо перетащить код использующий goto под g++. А g++ жалуется на переходы через объявление переменной.
Кто-нибудь знает, можно ли эту проверку подавить? Или хотя бы превратить в warning'и?

[RXL]

Sashok, что такое "переходы через объявление переменной"?

Покажи пример кода. На использование goto g++ с параметром -Wall не говорит ни слова.


А переписать данный фрагмент никак нельзя?

[Sashok]

RXL, в том-то и беда, что переписывать нельзя. Начальство получило этот код из исследовательского отдела как "работающий". Он действительно без проблем компилируется под Visual Studio. А нам надо под g++. Но начальник говорит, что, если я код перепишу, то поддреживать надо будет нам, а оно(начальство) этого не желает. Вот я и долбаю всех вопросами то по поводу strncpy_s и _countof, то по поводу переходов.  
Фрагмент кода сейчас переслать не могу, проблема с VPN. В понедельник с работы вышлю. Но идея следующая:

Код:

...
if(ret != 0) goto CLEANUP;
...
int var;
...
CLEANUP: 
...

Вызывает даже с флагом -Wall такую диагностику:

BSpline.cpp: 305: error: jump to label 'cleanup
BSpline.cpp: 280: error: from here
BSpline.cpp: 291: error: crosses initialization of `int var'

PS. Кстати, если есть какая-нибудь информация про библиотеки, поддерживающие Микрософтовские расширения С++ вроде тех, что я упоминал, тоже буду благодарен.

[Finch]

Sashok, Нельзя напрямую переносить код с студии в GСС. Тем более код, который писался без учета кроссплатформенности. В студии слишком много введений MS, которые не стандартны А в данном случае, может стоит поднять все объявления переменных наверх?.

[Finch]

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

g++ --version
g++ (GCC) 4.4.1 20090725 (Red Hat 4.4.1-2)

[Sashok]

Finch, я готов подписаться под каждым твоим словом.
Только реальность заставляет чесать левой рукой правое ухо.
Я же говорю, есть исследовательский отдел, который ни за что не отвечает, а нам надо давать готовый продукт. Но большое начальство, подчеркиваю, большое, считает, что, если исследовательский отдел написал код, на его подгонку ни средств, ни времени отводить не надо.
Если бы не эти коллизии, я бы просто переписал все, что не идет, и дело с концом!

К твоему последнему сообщению: у меня g++ 3.4.? (? - не помню)

Да, еще дополнение: у меня не Линукс, у меня Sun.

[Serg79]

Sashok, у тебя может 'goto BLABLABLA;' находиться в одной функции, а 'BALBLABLA' в другой?
Или попробуй переставить 'int var' в начало функции.

[RXL]

Sashok, думаю, можно попробовать такое решение: вы минимальными действиями подгоняете код и отдаете его назад - чтобы не говорили, что они не в курсе. За одно прокомментируйте его так, чтобы им стало стыдно.

[Вад]

А если код не работает на g++ - его нельзя вернуть в этот исследовательский отдел как "нерабочий"? Раз они отвечают за код, пусть и с совместимостью сами воюют.

По крайней мере, если исследовательский отдел написал нерабочий (или не всюду рабочий) код - как можно его в проект вставлять? А если оно скомпилируется, но будет работать некорректно?

[Finch]

Вад, Скорее всего в тех задании ни слова не было про кроссплатформенность. Вот они и написали для Студии.

[Вад]

Ну, я мыслю так: мне дают код, я пытаюсь компилировать на g++, не компилируется. Заношу баг: "не могу скомпилировать на g++". Конечно, это работает только там, где подобные вещи общеприняты - я ж не знаю, как у Sashok на работе процесс поставлен.

Это хорошо, что код дали. А дали бы бинарные библиотеки? Динамические? А потом как хочешь их из соляриса линкуй

[RXL]

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

Лично я в C++ никогда не ставлю объявления посреди кода. Только в начале блока, либо вообще стараюсь делать по правилам Си - объявлять все переменные в начале функции.

[Вад]

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

[Sashok]

По поводу кода и другого отдела: может быть если долго и всерьез жаловаться, они со временем  откроют финансирование и доработают его до кроссплатформенного состояния, но это никак не поможет нам сдать наш код в первом полугодии.
Они действительно писали, как им удобнее. Мало того, часть написана вообще на С#. То, что мы переделываем с С# называется "рефакторизацией" (хотя там полное переписывание с добавлением новых функций). На это время отвели, хотя и мало.

RXL, Я тоже так делаю. Я, кстати, вообще прекрасно обхожусь без goto. У меня еще много всяких претензий к тому, что мне дали, но этого уже не изменишь.
По поводу твоего предложения, да, скорее всего так и получится: минимально перепишу и прокомментирую. Но скорее всего придется это делать в свободное от работы время.

Резюмируя, если мне удастся решить эту проблему и проблему из соседней темы (strncpy_s...), не беспокоя высокое начальство, мое местное начальство будет считать меня героем и спасителем. Что отнюдь не лишнее при интересной(!) работе и высокой безработице. А если не удастся... ну, до конца проекта точно не уволят, но спасителем не буду.

[RXL]

Sashok, честно говоря я не понимаю - кто принял этот код? С какого перепугу он стал "работающим"? Неужели его просто "спускают" и типа дальше сами...


Интересная работа - это хорошо. Тонус мозга надо держать точно так же, как и тонус мышц. Но постоянное напряжение вредно - как своему мозгу, так и производственному процессу: кто-то, кто дает задания и перемещает результаты между отделами, должен узнать о происходящем, понять свои ошибки и сделать выводы.

[Sashok]

RXL, код приняли до того, как я начал здесь работать. Да, подход у большого начальства такой: "Кстати, вот у исследовательского отдела есть работающий код, используйте его. Время и деньги, соответсвтенно, на эту часть не нужны". Кстати, как мне сказал начальник, большое начальство с трудом убедили, что C# нельзя перевести в C++ для Unix с помощью выбора соответствующего пункта меню в Visual Studio.
Исследовательский отдел проверяет свой код на мелких примерах под Visual Studio, и там он действительно работает.
Я работаю на позиции контрактора, так что "беги туда, тащи это". Все равно интересно! Но явно не та позиция, чтобы давать указания высокому начальнику.

[resource]

Лично я в C++ никогда не ставлю объявления посреди кода. Только в начале блока, либо вообще стараюсь делать по правилам Си - объявлять все переменные в начале функции

Это отличная привычка. Мне кажется, только новички разбрасывают объявления переменных по телу функции. Я вот вообще пишу на Си, поэтому даже соблазна не испытываю объявлять переменные где попало. Бывает правда одно исключение из этого правила - когда используешь условные директивы типа #if #ifdef и т.д.. Но тут оправдано, т.к. если кусок кода не будет скомпилирован, то  не нужна и переменная, которая используется исключительно в этой части кода.

[RXL]

resource, у меня вообще преобладает стиль Си, т.к. он мне больше нравится, хоть и в C++ легче работать со строками и всякими STL. Си - это просто и надежно. Считаю, что не зная основ учиться высоким материям нельзя - знание будет неполным.

Sashok, я почему-то не завидую тебе. Извиняюсь, что я тут развел офтоп, но сколько платят за такую работу? И не могу понять твою текущую географию: твой IP принадлежит AT&T, без детальных уточнений.

[resource]

Си - это просто и надежно

Точнее выразиться, просто невозможно

[Sashok]

resource, у меня вообще преобладает стиль Си, т.к. он мне больше нравится, хоть и в C++ легче работать со строками и всякими STL. Си - это просто и надежно. Считаю, что не зная основ учиться высоким материям нельзя - знание будет неполным.

Вот! И я из той же категории программистов.

Sashok, я почему-то не завидую тебе. Извиняюсь, что я тут развел офтоп, но сколько платят за такую работу? И не могу понять твою текущую географию: твой IP принадлежит AT&T, без детальных уточнений.

Насчет географии все просто, я никогда и не скрывал. Я же здесь на форуме лет 5 назад очень часто появлялся, даже о своих горных походах постоянно сообщал. Короче, Америка. А провайдер у меня действительно AT&T.
Сколько платят? Трудно сравнивать в разных странах... Так что могу сказать, что, если не учитывать инфляции, на 30% больше, чем мне когда-либо платили раньше.
Но главное даже не это. Мне реально интересно. Это не банк какой-нибудь, где деньги со счета на счет переводить надо, и не кредитная карточка, где, если у тебя рейтинг такой-то надо предложить тебе кредит под 2%, а если такой-то - под 3%. Здесь есть математика, причем в разных группах разная, и не только у исследователей, но и у нас тоже. Я вот сейчас в свободное время, пока в электричке еду, пишу для своей группы популярную статью о сплайнах и кривых Безье. Мне так легче самому разбираться. 
Если предложат постоянную работу, пойду не задумываясь, хотя там, конечно, деньги совершенно не те. И уж тогда буду стараться исподволь подвести начальство к мысли, что ему выгоднее, если мы это все будем делать сами, не пользуясь "трудами" исследовательского отдела.
Фу-у-у, целая исповедь получилась.

[Алексей++]

RXL, в том-то и беда, что переписывать нельзя. Начальство получило этот код из исследовательского отдела как "работающий". Он действительно без проблем компилируется под Visual Studio. А нам надо под g++. Но начальник говорит, что, если я код перепишу, то поддреживать надо будет нам, а оно(начальство) этого не желает. Вот я и долбаю всех вопросами то по поводу strncpy_s и _countof, то по поводу переходов.  
Фрагмент кода сейчас переслать не могу, проблема с VPN. В понедельник с работы вышлю. Но идея следующая:

Код:

...
if(ret != 0) goto CLEANUP;
...
int var;
...
CLEANUP: 
...

Вызывает даже с флагом -Wall такую диагностику:

нет условий для вчитывания в тему, но если ответ ещё интересует, то надо делать так

Код:

...
if(ret != 0) goto CLEANUP;
...


{
     int var;
}
...
CLEANUP: 
...

то есть ставим блок {} , а где будет использована переменная - это уже по смыслу. Обойти блок {} goto всегда может (то же касается switch)

[Sashok]

Алексей1153++, да, это я знаю.
Мы как-то незаметно отклонились от темы. Вопрос-то был не в том, как переписать код. Вопрос был в том, можно ли подавить диагностику таких вещей в g++.
Если уж вопрос встанет о переписывании кода, я вообще уберу goto - не люблю я их...

[RXL]

Думаю, что никак.
Параметр -fpermissive может сделать зависимую ошибку предупреждением, но подавить саму ошибку кросс-инициализации не выходит.

[Алексей++]

Вопрос был в том, можно ли подавить диагностику таких вещей в g++.

а нельзя подавлять - ведь это, получается, ты перескакиваешь вызов конструкторов. Нехорошо Вот компилятор и в шоке - он привык всегда вызывать конструкторы.

Если уж вопрос встанет о переписывании кода, я вообще уберу goto - не люблю я их...

это да )) Но лучше не трогай - потом глюки править себе дороже. Вот новый код надо писать уже с учётом

[Антон (LogRus)]

Sashok, не слушай его, смело ломай код я всегда так делаю, за некоторое количество лет, что я программирую пришел к выводу, что с кодом надо быть жестче иначе, старый код никогда не подвергнется хоть минимальному рефакторингу, но это ИМХО а не руководство к действию

[resource]

Абсолютно согласен. Тут или ломать, или вообще забыть про него. Я бы именно так поставил вопрос перед начальством. Потому что непонятно как-то, чего им надо. Хотят и рыбку съесть и........ на gcc пересесть. Чтоб код компилился gcc, когда его под студию писали? Попроще бы нашли себе занятие

[Алексей++]

LogRus, Sashok, Слушай, слушай:) Если есть время +  ещё мало пользователей программы, то да - надо править. Иначе - не надо А то поддерживать глюки замучаешься


Совсем другое дело - оставить интерфейс старый, а начинку написать свою заново

[Sashok]

LogRus, resource, Алексей1153++, спасибо.
Я в общем тоже не первый год работаю (желающие могут посмотреть мои статьи здесь на форуме лет 5-7 назад  ) и в данном случае соглашусь с LogRus и resource.

Код для внутреннего пользования, продуктом является то, что код делает, а не сам код. Так что либо я его не буду трогать вообще (если найду как обмануть компилятор), либо уж буду править так, чтобы самому за него отвечать со спокойной совестью. Кстати, феномен того, что код с заплатами начинает выдвигать свои требования и что с ним надо пожестче я тоже наблюдал не раз.

Алексей1153++, насчет сути (перескакиваю конструкторы) я согласен. Беда в том, что Мелкомягкие такое допускают и не жалуются, а получившееся дерьмо расхлебывать мне... 

resource, а начальство - оно всегда хочет все сразу и забесплатно...

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