Forum :: EOSERV :: lewis999's EOserv rev 189

Change SAKE_SEEK to SEEK_END. That works for me. In your quest.cpp file.

---
"Pineapples and shit."

Thanks but now I get this :

C:\Users\LewisH\Downloads\eoserv-trunk-r189\eoserv-trunk-r189\trunk\src\world.hpp||In constructor 'Quest::Quest(short int, World*)':|

C:\Users\LewisH\Downloads\eoserv-trunk-r189\eoserv-trunk-r189\trunk\src\world.hpp|270|error: '2' cannot be used as a function|

C:\Users\LewisH\Downloads\eoserv-trunk-r189\eoserv-trunk-r189\trunk\src\world.hpp|272|error: '2' cannot be used as a function|

C:\Users\LewisH\Downloads\eoserv-trunk-r189\eoserv-trunk-r189\trunk\src\world.hpp|275|warning: comparison between signed and unsigned integer expressions|

||=== Build finished: 2 errors, 23 warnings ===|

Can you pastebin your quest.cpp and post it. I could get a clear view of what you're using and how to fix this.

---
"Pineapples and shit."

Here it is:

http://pastebin.com/fvHkiKGQ

Try and switch your Quest::Quest function with this one;

Quest::Quest(short id, World *world)
{
    char namebuf[4];

    if (id < 0)
    {
        return;
    }
    this->id = id;
    this->exists = true;

    std::string filename = "";
    filename = world->config["QuestDir"].GetString();
    std::sprintf(namebuf, "%05i", id);
    filename.append(namebuf);
    filename.append(".eqf");
    std::FILE *fh = std::fopen(filename.c_str(), "rt");
    std::string buf;

    if (!fh)
    {
        Console::Err("Could not load file: %s", filename.c_str());
        this->exists = false;
        return;
    }

    std::fseek(fh, 0, SEEK_END);
    long size = ftell(fh);
    std::rewind(fh);

    buf.resize(size);
    std::fread(&buf[0], 1, size, fh);
    std::fclose(fh);

    Load(buf);
}

---
"Pineapples and shit."

Thank you for all you help, its finaly working :D

Nice! :] If you need help on errors and such you could always ask me in this topic.

---
"Pineapples and shit."

I gave up on trying to get Addison's thing working because the problems it has are conveniently never explained in the thread about it and when I tried to ask about it I just got told to hold off until Apollo's bulk rev comes out.

hey lewis i think you forgot to add quest.cpp again in v0.3

---
Live and let live. Hurt but do not kill. Finish what you start. NEVER GIVE UP!!!

SAKE_SEEK is what Sausage uses to load map files safely. I decided to add into my source to be cool. Here's what you need to add at the top of quest.cpp:

static const char *safe_fail_filename;

static void safe_fail(int line)
{
    Console::Err("Invalid file / failed read/seek: %s -- %i", safe_fail_filename, line);
    std::exit(1);
}

#define SAFE_SEEK(fh, offset, from) if (std::fseek(fh, offset, from) != 0) { std::fclose(fh); safe_fail(__LINE__); }
#define SAFE_READ(buf, size, count, fh) if (std::fread(buf, size, count, fh) != static_cast<int>(count)) {  std::fclose(fh); safe_fail(__LINE__);}

---
http://www.addipop.com

Don't you mean SAFE_SEEK Instead of SAKE_SEEK?

---
"Pineapples and shit."

He's right, the 4 Quest.cpp/hpp files aren't added into this project, I had to manually add them myself.

There was also a weird issue with recompiling eoserv.  I readded the server-specific changes I have normally (std::abs map fix, disabling of #class, and pet items), but my project refused to acknowledge that there had been a change, so I had to rebuild instead of build.

Shit was kinda weird.

Ananas, we all make mistakes. I've heard about that mistake a few times. Eventually, I'll get off my lazy ass and finish my quest system and upload all the fixes.

---
http://www.addipop.com

Addison posted: (30th Aug 2010 12:13 pm)

Ananas, we all make mistakes. I've heard about that mistake a few times. Eventually, I'll get off my lazy ass and finish my quest system and upload all the fixes.

---
"Pineapples and shit."

With this new quest thing, I am unable to make the Wise Man thing happen more than once.  After I finish changing my class, he is unclickable again.  I don't know if this is a problem in Lewis' implementation or if it was always a problem in the original quest code itself,butregardless it makes for a depressingly lackluster Wise Man, unless you actually want people to be able to change their class only once.

edit: See next post, iFail@life