|
Ф/\Ей|\/| Ну сами понимаете... должно быть место, которое не модерируется... Но не злоупотребляйте... |
![]() |
|
Опции темы | Опции просмотра |
![]() |
#1 |
Занятой мадер
![]() Регистрация: 24.09.2007
Адрес: Харьков
Сообщений: 149
|
![]()
Буду краток.
Делая свой мегамад на основе кода Былин, я нашел серьезную ошибку в этом коде. В файле item.creation.cpp нарушен баланс фигурных скобок, причем серьезно, я так и не понял, почему этот файл, а точнее команда создания и редактирования рецептов mredit всё таки как-то работает. Ошибка заключается в том, что при вводе ошибочной команды (например вместо выхода "q" я нажал не в том регистре букву "й") не выдается "ошибочный ввод", а мад тупо вываливается в корку. Анализ исходника и показал мне нарушенный баланс скобок, точнее скобки не на том месте (синтаксически верно, компилятор ничего такого не замечает, а выполняется неверно). Анализировался файл item.creation.cpp версии Date: 2009/11/22 08:30:50 Revision: 1.29, полученный сегодня при помощи cvs с сайта Былин, то есть свежий. Вот исправленный файл http://prool.kharkov.org/mud/bugfixe...m.creation.cpp Я там в нужном месте пометил комментарием // prool, но лучше сделайте diff с оригинальным файлом и всё будет видно |
![]() |
![]() |
![]() |
#2 |
Гость
Сообщений: n/a
|
![]()
Ну командой очевидно никто не пользовался, не факт, что она вообще доделанная изначально. За скобки эти канеш спасибо, но перед использованием я бы плотно ознакомился че она там вообще редактирует и как. ж)
Дифф на 2000 строк из-за переформатированных под k&r скобок доставил. |
![]() |
![]() |
#3 | |
Занятой мадер
![]() Регистрация: 24.09.2007
Адрес: Харьков
Сообщений: 149
|
![]() Цитата:
Что же касается диффа, я ничего не пойму, лично я форматирование не менял. Вот мой дифф: 161a162 > #if 1 // prool 235a237 > } // prool 391d392 < } 394a396,397 > #endif > На появление строк #if 1 #endif не смотри, я из использовал для экспериментов и отладки. Если число строк не поменялось, мой дифф даст нужную информацию |
|
![]() |
![]() |
![]() |
#4 |
Занятой мадер
![]() Регистрация: 24.09.2007
Адрес: Харьков
Сообщений: 149
|
![]()
Или вот так: расскажу устно:
В функции mredit_parse() в альтернативе case MREDIT_MAIN_MENU: должен быть вот такой фрагмент: if (sagr == "q") { // Проверяем не производилось ли изменение if (OLC_VAL(d)) { send_to_char("Вы желаете сохранить изменения в рецепте ?(y/n) : ", d ->character); OLC_MODE(d) = MREDIT_CONFIRM_SAVE; return; } else { // Загружаем рецепты из файла // Это восстановит текущее состояние дел. make_recepts.load(); // Очищаем структуры OLC выходим в нормальный режим работы cleanup_olc(d, CLEANUP_ALL); return; } } // prool send_to_char("Неверный ввод.\r\n", d->character); mredit_disp_menu(d); break; Помеченную комментарием // prool фигурную скобку я добавил, чтобы правильно закрыть оператор if. А для сохранения баланса скобок надо убрать неправильную закрывающуюся скобку в конце функции. |
![]() |
![]() |
![]() |
#5 |
Гость
Сообщений: n/a
|
![]()
Исправил я эту скобку еще перед тем как ответить, шо ж ты так плохо обо мне думаешь. ж)
На счет дифа - нет в коде давно такого форматирования: if () { ... } else { ... } Может у тя редактор автоматом меняет и дифф с какими-то умными опциями, я хз. |
![]() |
![]() |
#6 | ||
Занятой мадер
![]() Регистрация: 24.09.2007
Адрес: Харьков
Сообщений: 149
|
![]() Цитата:
В файле, вытащенном вчера по cvs с Былин эта ошибка еще была Цитата:
|
||
![]() |
![]() |
![]() |
#7 |
Новичок
Регистрация: 03.04.2005
Сообщений: 26
|
![]()
Настоящие мадеры настолько суровы, что не пользуются другими текстовыми редакторами, кроме vi
![]() Простите за флейм, не удержался ![]() |
![]() |
![]() |
![]() |
#8 |
Занятой мадер
![]() Регистрация: 24.09.2007
Адрес: Харьков
Сообщений: 149
|
![]()
Нет, vi для мадера слишком мощный и слишком визуальный редактор. Во встроенном в мад редакторе объектов, комнат и мобов olc есть встроенный миниредактор текстовых описаний, это даже не vi, это нечто похожее на построчный редактор ed, который был предшественником vi
|
![]() |
![]() |
![]() |
#9 |
Отличник форума
![]() Регистрация: 06.03.2004
Адрес: Деревня РСП
Сообщений: 1,018
|
![]()
Святое не трожьте!
__________________
Прыг секунда, скок столетие... |
![]() |
![]() |
![]() |
#10 |
Занятой мадер
![]() Регистрация: 01.05.2006
Сообщений: 125
|
![]()
__________________
Хто до нас з пиво прийде, той за горілкою і побіжить! |
![]() |
![]() |
![]() |
#11 |
Админ форума
![]() Регистрация: 05.03.2004
Сообщений: 635
|
![]()
ты ж уже "прошел эту игру"
![]() ![]()
__________________
©º°¨¨°º© МПМ "Былины" ©º°¨¨°º© Мир преданий и сказаний Древней Руси |
![]() |
![]() |