{"id":35,"date":"2018-05-27T18:57:10","date_gmt":"2018-05-27T18:57:10","guid":{"rendered":"https:\/\/blogs.scummvm.org\/pantherengine\/?p=35"},"modified":"2022-05-23T19:00:43","modified_gmt":"2022-05-23T19:00:43","slug":"week-2","status":"publish","type":"post","link":"https:\/\/blogs.scummvm.org\/pantherengine\/2018\/05\/27\/week-2\/","title":{"rendered":"Week 2"},"content":{"rendered":"<p>At the second week I have done:<span id=\"more-31\"><\/span><\/p>\n<ul>\n<li>Fixed formatting<\/li>\n<li>Removed C++11 features(lambdas, auto, extended initializer lists)<\/li>\n<li>Fixed unsigned vs signed comparison<\/li>\n<li>Found and fixed bug in ScummVm\u2019s hashmap<\/li>\n<li>Added improved pause support<\/li>\n<li>Added saving system (currently without metadata)<\/li>\n<\/ul>\n<p>The most interesting was fixing the bug in Hashmap. At first,\u00a0 I thought that this was problem with my code. Experimented with code and noticed that calling clear with true parameter causes this bug. So I looked at clear method implementation(Line 4):<\/p>\n<p>&nbsp;<\/p>\n<p><img decoding=\"async\" class=\"aligncenter\" src=\"https:\/\/blogs.scummvm.org\/pantherengine\/wp-content\/uploads\/sites\/31\/2022\/05\/carbon-11.png\" \/>And this part of HashMap constructor(Line 1):<\/p>\n<p><img decoding=\"async\" class=\"aligncenter\" src=\"https:\/\/blogs.scummvm.org\/pantherengine\/wp-content\/uploads\/sites\/31\/2022\/05\/carbon-2.png\" \/><\/p>\n<p>Simple mistake, which was easily\u00a0<a href=\"https:\/\/github.com\/scummvm\/scummvm\/pull\/1202\">fixed<\/a><\/p>\n","protected":false},"excerpt":{"rendered":"<p>At the second week I have done: Fixed formatting Removed C++11 features(lambdas, auto, extended initializer lists) Fixed unsigned vs signed comparison Found and fixed bug in ScummVm\u2019s hashmap Added improved pause support Added saving system (currently without metadata) The most interesting was fixing the bug in Hashmap. At first,\u00a0 I thought that this was problem [&hellip;]<\/p>\n","protected":false},"author":1,"featured_media":0,"comment_status":"closed","ping_status":"closed","sticky":false,"template":"","format":"standard","meta":{"footnotes":""},"categories":[1],"tags":[],"class_list":["post-35","post","type-post","status-publish","format-standard","hentry","category-uncategorized"],"_links":{"self":[{"href":"https:\/\/blogs.scummvm.org\/pantherengine\/wp-json\/wp\/v2\/posts\/35","targetHints":{"allow":["GET"]}}],"collection":[{"href":"https:\/\/blogs.scummvm.org\/pantherengine\/wp-json\/wp\/v2\/posts"}],"about":[{"href":"https:\/\/blogs.scummvm.org\/pantherengine\/wp-json\/wp\/v2\/types\/post"}],"author":[{"embeddable":true,"href":"https:\/\/blogs.scummvm.org\/pantherengine\/wp-json\/wp\/v2\/users\/1"}],"replies":[{"embeddable":true,"href":"https:\/\/blogs.scummvm.org\/pantherengine\/wp-json\/wp\/v2\/comments?post=35"}],"version-history":[{"count":2,"href":"https:\/\/blogs.scummvm.org\/pantherengine\/wp-json\/wp\/v2\/posts\/35\/revisions"}],"predecessor-version":[{"id":37,"href":"https:\/\/blogs.scummvm.org\/pantherengine\/wp-json\/wp\/v2\/posts\/35\/revisions\/37"}],"wp:attachment":[{"href":"https:\/\/blogs.scummvm.org\/pantherengine\/wp-json\/wp\/v2\/media?parent=35"}],"wp:term":[{"taxonomy":"category","embeddable":true,"href":"https:\/\/blogs.scummvm.org\/pantherengine\/wp-json\/wp\/v2\/categories?post=35"},{"taxonomy":"post_tag","embeddable":true,"href":"https:\/\/blogs.scummvm.org\/pantherengine\/wp-json\/wp\/v2\/tags?post=35"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}}