Кодерский уголок

Хм, ну да, тогда надо.

 

Дальше идет еще большая дичь:

extrasidbank.h

Код:

class ExtraSidBank : public Bank
{
private:
    typedef std::vector<c64sid*> sids_t;

    class resetSID
    {
    public:
        void operator() (sids_t::value_type &e) { e->reset(0xf); }
    };

private:
    /**
     * Size of mapping table. Each 32 bytes another SID chip base address
     * can be assigned to.
     */
    static const int MAPPER_SIZE = 8;

private:
    /**
     * SID mapping table.
     * Maps a SID chip base address to a SID
     * or to the underlying bank.
     */
    Bank *mapper[MAPPER_SIZE];

    sids_t sids;

private:
    static unsigned int mapperIndex(int address) { return address >> 5 & (MAPPER_SIZE - 1); }

public:
    virtual ~ExtraSidBank() {}

    void reset()
    {
        std::for_each(sids.begin(), sids.end(), resetSID());
    }

    void resetSIDMapper(Bank *bank)
    {
        for (int i = 0; i < MAPPER_SIZE; i++)
            mapper[i] = bank;
    }

    uint8_t peek(uint_least16_t addr)
    {
        return mapper[mapperIndex(addr)]->peek(addr);
    }

    void poke(uint_least16_t addr, uint8_t data)
    {
        mapper[mapperIndex(addr)]->poke(addr, data);
    }

    /**
     * Set SID emulation.
     *
     * @param s the emulation
     * @param address the address where to put the chip
     */
    void addSID(c64sid *s, int address)
    {
        sids.push_back(s);
        mapper[mapperIndex(address)] = s;
    }
};

c64.h

Код:

class c64: private c64env
{
    ...
private:
    typedef std::map<int, ExtraSidBank*> sidBankMap_t;
    ...

    /// Extra SIDs
    sidBankMap_t extraSidBanks;

    ...
};

c64.cpp

Код:

       ExtraSidBank *extraSidBank = extraSidBanks.insert(it, sidBankMap_t::value_type(idx, new ExtraSidBank()))->second;

Вот эта последняя строчка и выдает: sidplayfp\c64\c64.cpp(157): Error! E041:col(58) expression for '->' must be pointer to class, struct or union

 

Geryon, #include "extrasidbank.h" имеетя в наличии в c64.cpp? Хотя не то.
Вообще insert в данном случае возвращает итератор на пару std:air<int, ExtraSidBank*>, где .first - это int, а .second - это ExtraSidBank*. В каком именно виде итератор - сильно зависит от реализации, там может быть и не указатель.
Попробуй так - ExtraSidBank *extraSidBank = extraSidBanks.insert(it, sidBankMap_t::value_type(idx, new ExtraSidBank())).second;

 

Нет, но имеется в c64.h, который в свою очередь имеется в c64.cpp, так что дело, видимо, не в этом.

Кажется, я это победил таким путем:

было:

Код:

ExtraSidBank *extraSidBank = extraSidBanks.insert(it, sidBankMap_t::value_type(idx, new ExtraSidBank()))->second;

стало:

Код:

ExtraSidBank *extraSidBank = extraSidBanks.insert(it, sidBankMap_t::value_type(idx, new ExtraSidBank())).first->second;

Экспериментальным путем я выяснил, что тип возвращаемого .insert() объекта следующий:

Код:

   std::pair<std::map<int, ExtraSidBank*>::iterator,bool>

А автор кода ожидал не bool значение, а ExtraSidBank* - вот я и заменил ->second на .first->second. Остается лишь гадать, то ли в коде ошибка, то я чего-то не понимаю.

 

first - это ключ, а он у нас int, у инта точно нет second. Не знаю, что ты победил. =)

 

ss_temp += sprintf( ss_temp, ........ );

 

Я тоже совсем не уверен в своих решениях насчет этой мути, которую вижу впервые, но теперь стало компилироваться. Разве это было бы возможно, если я попытался применить ->second к чему-то, имеющему тип int?

 

Ассоциативность у эл-тов структур - слева направо. Т.е. эквивалент:

Код:

[COLOR="DarkRed"][B]([/B][/COLOR] extraSidBanks.insert(it, sidBankMap_t::value_type(.....)).first [B][COLOR="DarkRed"])[/COLOR][/B]->second;

---------- Сообщение добавлено в 21:16 ---------- Предыдущее сообщение размещено в 21:14 ----------

А вообще - интересно, это ж сколько в этой коротенькой строчке будет выполнено new'ов и delete'ов? :-(((((

 

Geryon, это называется оверинжинирнг. И это ещё далеко не самый худший образец. Разбей на отдельные операции, чтобы понять как работает, кто что возвращает.

 

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

Код:

    std::pair<std::map<int, ExtraSidBank*>::iterator,bool> ret;

    ...

    ret = extraSidBanks.insert(it, sidBankMap_t::value_type(idx, new ExtraSidBank()));
    ExtraSidBank *extraSidBank = ret.first->second;    

И вот новая напасть:

romCheck.h

Код:

class romCheck
{
private:
    typedef std::map<std::string, const char*> md5map;

private:
    md5map m_checksums;

    ...

private:
    std::string checksum() const
    {
        sidmd5 md5;
        md5.append (m_rom, m_size);
        md5.finish();

        return md5.getDigest();
    }

    ...

public:
    const char* info() const
    {
        md5map::const_iterator res = m_checksums.find(checksum()); // sidplayfp\romCheck.h(95): Error! E419:col(38) cannot call non-const function for a constant object
        return (res!=m_checksums.end())?res->second:"Unknown Rom";
    }
};

 

Может в старом стандарте было что-то типа find_const? Есди нет, то сделай const char* info() const

 

Я заглянул во включаемый файл map на предмет чего-то типа find_const, а там:

Код:

    iterator find(const key_type& k);
    const_terator find(const key_type& x) const;

Исправил const_terator на const_iterator... но ошибка, как ни странно, никуда не исчезла.

А, ну понятно - там этот код был внутри комментария для справки, а на самом деле никакой const версии нет

Это помогло.

 

player.h

Код:

class Player
#ifdef PC64_TESTSUITE
  : public testEnv
#endif
{
    ...
public:
    ...
    uint_least32_t time() const { return (uint_least32_t)(m_c64.getEventScheduler().getTime(EVENT_CLOCK_PHI1) / cpuFreq()); }
    ...
};

player.cpp

Код:

const char TXT_NA[]             = "NA";


Player::Player () :
    /* Set default settings for system */
    m_tune(0),
    m_errorString(TXT_NA),
    m_isPlaying(false),
    m_rand((unsigned int)::time(0)) // sidplayfp\player.cpp(44): Error! E927: col(28) syntax error: 'time' has not been declared
{
    ...
}

Что вообще означает эта запись: Player::Player() : и тут какие-то члены класса пошли со скобками?

Мало того, что компилятору не нравится запись ::time, так еще и определение Player::time не содержит аргументов, а вызывается с аргументом: time(0)

При этом у автора кода все отлично компилируется gcc.

Как такое возможно?

 

Та ж конструктор же ж.

Инициализация членов класса

Это не ф-ция-член класса, это ::time, т.е. вызов функции вне класса. А m_rand инициализируется значением, возвращённым этой функцией.

Отсутствие пробелов?

 

Вот оно что! Действительно, включение <time.h> решило проблему (до этого был включен только <ctime>).

Какое отсутствие пробелов? Я теперь полагаю, что просто у gcc в <ctime> есть функция time, а у OW нет.

Заглянул в \WATCOM\h\ctime:

Код:

namespace std {
...
_WCRTLINK extern time_t       time( time_t *__timer );
...
}

Это что значит? Что есть std::time, а просто time нету?

 

Geryon, всё, что в неймспейсе std, относится к стандартной библиотеке C++. Сделано для разделения от С.

 

Хм. Почему этот код может не видеть неймспейса std?

 

Ну, например:

Код:

int jx=0,jy=0,jz=jx+++jy;

- чему будет равен jz?
Не, ну я не настаиваю. Но вот такая запись

Код:

m_rand( (unsigned int) ::time( NULL ) )

читается не в пример понятнее.

 

Понятия не имею! Трактовать эти +++ можно минимум двумя способами.

Эта запись на мой взгляд, в отличие от +++, не является многозначной. На всякий случай я расставил пробелы, но ничего не изменилось: пока не включишь <time.h> - не компилируется.

 

(поднимает коготь) Изменилось восприятие "::time()" - теперь видно, что это не член класса, а отдельная функция. Соответственно, разбор ошибок будет идти быстрее.

А Х его З. В те времена, когда я изучал C++, использование namespace'ов то ли не было обязательным, то ли даже не поощрялось. А было это в аккурат на закате эпохи MS-DOS, чуть меньше 20 лет назад.