{"id":30,"date":"2022-06-27T12:29:51","date_gmt":"2022-06-27T12:29:51","guid":{"rendered":"https:\/\/blogs.scummvm.org\/grisenti\/?p=30"},"modified":"2022-06-27T12:31:43","modified_gmt":"2022-06-27T12:31:43","slug":"compiling-the-game-and-completing-resource-loading","status":"publish","type":"post","link":"https:\/\/blogs.scummvm.org\/grisenti\/2022\/06\/27\/compiling-the-game-and-completing-resource-loading\/","title":{"rendered":"Compiling the game and completing resource loading"},"content":{"rendered":"

The first thing I worked on this week was finishing up the code that does resource loading which has now been moved to the LowLevelResources<\/code> class inside of the resources folder. After doing that I worked on bringing in the game code. This was easier than expected with only a few problems with conversions from strings of wchar_t<\/code>, used mainly for file code, to char<\/code> strings. I then did some testing and got to load the game.cfg<\/em> file, which is part of the game\u2019s assets. After that the code to instantiate the various systems gets executed and the program crashed by trying to dereference a null pointer.<\/p>\n

Next, I worked on replacing image loading. This was done by the LowLevelResourcesSDL<\/code> class through the SDL image library. The data was then stored in the SDLBitmap2D<\/code> class in the form of SDL_surface<\/code>. Everything has now been moved to the Bitmap2D<\/code> class in the graphics folder which also handles image loading through the Image::*Decoder<\/code> family of classes. One problem I encountered was that after loading an image the relative Graphics::Surface<\/code> cannot be modified, which is required by the fillRect<\/code> and drawToBitmap<\/code> methods. This led to the implementation of a copy-on-write mechanism.<\/p>\n

This week the codebase was also modified to reduce the number of warnings produced by gcc. After bringing in the engine code the total lines of warnings were about 24000. A lot of these where caused by missing virtual destructors in classes that had virtual methods. After adding these, the line count was reduced to 3000. With more changes, it\u2019s now at less than 2000, which is a huge improvement. The credit for this work goes to sev with me doing only very minor changes.<\/p>\n

The last thing I worked on this week was font loading. The game uses a format split into a .fnt xml file and some tga image files which contain a table of characters. This concludes resource loading for now, there\u2019s still the save system but this has been pushed to a later date since it\u2019s not necessary to get the game working.<\/p>\n

Bonus problems<\/h4>\n
    \n
  • After including pixelformat.h<\/em> in some files, the compiler started generating some weird errors which I couldn\u2019t figure out. Turn out the problem was that hpl1\/engine\/graphics\/PixelFormat.h<\/em> and graphics\/pixelformat.h<\/em> were considered the same file. I don\u2019t know what happened with the directories but for the files, this was caused by the windows file system being case insensitive.<\/li>\n
  • In the bitmap class I used Common::ScopedPtr<\/code> for storing the image decoder. Trying to use the move assignment operator though generated compilation errors. The operation is implemented as:\n
    template<\/span><<\/span>class<\/span> T2><\/span>\r\nScopedPtr &<\/span>operator<\/span>=<\/span>(<\/span>ScopedPtr<<\/span>T2><\/span> &<\/span>&<\/span>other)<\/span> {<\/span>\r\n\tPointerType oldPointer =<\/span> _pointer;<\/span>\r\n\t_pointer =<\/span> other.<\/span>_pointer;<\/span>\r\n\tother.<\/span>_pointer =<\/span> nullptr<\/span>;<\/span>\r\n\tDL(<\/span>)<\/span>(<\/span>oldPointer)<\/span>;<\/span>\r\n\treturn<\/span> *<\/span>this<\/span>;<\/span>\r\n}<\/span>\r\n<\/pre>\n
    The problem turned out to be that different istances of the same template class cannot access each other\u2019s private members. The fix was a friend declaration in Common::ScopedPtr<\/code> to all the other instances of itself:<\/p>\n
    template<\/span><<\/span>class<\/span> T2,<\/span> class<\/span> DL2><\/span>\r\nfriend<\/span> ScopedPtr;<\/span> \r\n<\/pre>\n<\/li>\n<\/ul>\n
    Next week<\/h4>\n
    Next week I\u2019ll start working the graphics system.<\/p>\n
    Thanks for reading.<\/p>\n","protected":false},"excerpt":{"rendered":"
    The first thing I worked on this week was finishing up the code that does resource loading which has now been moved to the LowLevelResources class inside of the resources folder. After doing that I worked on bringing in the game code. This was easier than expected with only a few problems with conversions from […]<\/p>\n","protected":false},"author":13,"featured_media":0,"comment_status":"open","ping_status":"open","sticky":false,"template":"","format":"standard","meta":{"footnotes":""},"categories":[1],"tags":[],"class_list":["post-30","post","type-post","status-publish","format-standard","hentry","category-uncategorized"],"_links":{"self":[{"href":"https:\/\/blogs.scummvm.org\/grisenti\/wp-json\/wp\/v2\/posts\/30","targetHints":{"allow":["GET"]}}],"collection":[{"href":"https:\/\/blogs.scummvm.org\/grisenti\/wp-json\/wp\/v2\/posts"}],"about":[{"href":"https:\/\/blogs.scummvm.org\/grisenti\/wp-json\/wp\/v2\/types\/post"}],"author":[{"embeddable":true,"href":"https:\/\/blogs.scummvm.org\/grisenti\/wp-json\/wp\/v2\/users\/13"}],"replies":[{"embeddable":true,"href":"https:\/\/blogs.scummvm.org\/grisenti\/wp-json\/wp\/v2\/comments?post=30"}],"version-history":[{"count":2,"href":"https:\/\/blogs.scummvm.org\/grisenti\/wp-json\/wp\/v2\/posts\/30\/revisions"}],"predecessor-version":[{"id":32,"href":"https:\/\/blogs.scummvm.org\/grisenti\/wp-json\/wp\/v2\/posts\/30\/revisions\/32"}],"wp:attachment":[{"href":"https:\/\/blogs.scummvm.org\/grisenti\/wp-json\/wp\/v2\/media?parent=30"}],"wp:term":[{"taxonomy":"category","embeddable":true,"href":"https:\/\/blogs.scummvm.org\/grisenti\/wp-json\/wp\/v2\/categories?post=30"},{"taxonomy":"post_tag","embeddable":true,"href":"https:\/\/blogs.scummvm.org\/grisenti\/wp-json\/wp\/v2\/tags?post=30"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}}