Week 2

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’s hashmap
  • Added improved pause support
  • Added saving system (currently without metadata)

The most interesting was fixing the bug in Hashmap. At first,  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):

carbon (1).png

And this part of HashMap constructor(Line 1):

carbon (2).png

Simple mistake, which was easily fixed

Advertisements

Leave a Reply

Fill in your details below or click an icon to log in:

WordPress.com Logo

You are commenting using your WordPress.com account. Log Out /  Change )

Google+ photo

You are commenting using your Google+ account. Log Out /  Change )

Twitter picture

You are commenting using your Twitter account. Log Out /  Change )

Facebook photo

You are commenting using your Facebook account. Log Out /  Change )

Connecting to %s