Thanks for sharing this great implementation of Pacman. I have used it to test a port of SDL2 to QNX 7 and it works really well.

I did encounter a problem while building the code, though. Line 621 in highscore.cpp is generating an error:

highscore.cpp: In member function 'void HighscoreList::load()':
highscore.cpp:621:27: error: operands to ?: have different types 'bool' and 'std::__1::basic_istream<char>'
    while (fileIsEncrypted ? readEncryptedLine(f, line) : std::getline(f, line)) {

The error can be fixed with the following patch, which wraps std::getline() with a function that returns bool:

diff --git a/src/highscore.cpp b/src/highscore.cpp
index 47baa78..a3a26da 100644
--- a/src/highscore.cpp
+++ b/src/highscore.cpp
@@ -586,6 +586,11 @@ bool HighscoreList::readEncryptedLine(std::ifstream &f, std::string &line) {
        }
 }
 
+bool HighscoreList::readLine(std::ifstream &f, std::string &line) {
+    std::getline(f, line);
+    return (line.length() > 0);
+}
+
 void HighscoreList::load() {
        if (readonly)
                return;
@@ -618,7 +623,7 @@ void HighscoreList::load() {
                if (f.is_open()) {
                        std::string line;
                        uint8_t linesRead = 0, validLines = 0;
-                       while (fileIsEncrypted ? readEncryptedLine(f, line) : std::getline(f, line)) {
+                       while (fileIsEncrypted ? readEncryptedLine(f, line) : readLine(f, line)) {
                                if (line.length() >= 1) {
                                        ++linesRead;
                                        std::string::size_type pos = line.find('|');
diff --git a/src/highscore.h b/src/highscore.h
index 8b3069f..88c26d4 100644
--- a/src/highscore.h
+++ b/src/highscore.h
@@ -49,6 +49,7 @@ class HighscoreList {
                void draw(bool nameAlterable, bool highlightLast);
                bool eventloop(bool nameAlterable, bool *redrawNeeded);
                bool readEncryptedLine(std::ifstream &f, std::string &line);
+               bool readLine(std::ifstream &f, std::string &line);
                uint8_t maxSize;
                std::vector<HighscoreEntry*> *entries;
                int idxLastInsertedEntry, idxHighlightedEntry;

I also had to remove -lpthread from Makefile.am. This is a Linux-specific library (QNX has pthread support in the C library). It may be better to enhance the configuration script to detect that.

--Elad

use automake generated config.h to get the version number

• code review
• ghost hunting mode after eating super pills
• the bonus fruits
• pacman life counting
• new level after eating all pills
• sounds
• start screen with some configurations

I think I could play this much better if I could use the vi keys that I'm used to, instead of having to use the arrow keypad.

why?
I've just download the repository.

add autogen.sh in root dir:

!/bin/sh

aclocal && automake --add-missing --copy && autoconf

hi, i want to play with gamepad, what should i do

OS: Ubuntu 18.0.4
SDL version: 2.0
gcc version: 7.4.0

The ghosts are rendered as eyes only, no ghost color is shown (see screenshot).

This is because the black background on augen_0.png, augen_1.png, augen_2.png, and augen_3.png are not set to transparent. I think it is a conflict between the transparent index that you have set for the ghost images (255) and the eye images (0).

The fix I employed was:

1. 1. Insert the following at line 195 of src/screen.cpp: SDL_SetSurfaceBlendMode(surface, SDL_BLENDMODE_BLEND);
2. Modify the augen_?.png files so that alpha=0 for the background
3. Change the last argument of lines 65-68 of src/ghost_figur.cpp from 0 to -1.

There is a similar bug affecting gfx/escaping_ghost_?.png.

I have these fixes in a branch. I'll send it to you if you are interested, or push it if you grant access to me.

This is the result of my fix:

My idea is increasing score when user passes a level.I'm not sure it's a good idea.

It would be cool if a simple .desktop file could be included with the distribution. Here is the one I have now:

[Desktop Entry]
Name=Pacman (SDL)
Comment=Yet another pacman clone in C/C++ and SDL
Exec=pacman
Icon=pacman
Type=Application
Categories=Game;ArcadeGame;

And this is the icon I am using:

set initial pill limit to -1 in branch level_difficulty

Hi.
Level 1 or 2 are just the same thing and this makes the game boring.

here is a simple patch for adding per 10000 points life

diff --git a/src/game.cpp b/src/game.cpp
index c8c7567..23d3376 100644
--- a/src/game.cpp
+++ b/src/game.cpp
@@ -325,7 +325,9 @@ bool Game::checkLastPillEaten() {
 void Game::checkScoreForExtraLife() {
 	int lastScore = currentScore;
 	currentScore = Labyrinth::getInstance()->getScore();
-	if ((lastScore<10000 && currentScore>=10000) || (lastScore<30000 && currentScore>=30000)) {
+	int target = ( ( currentScore / 10000 ) * 10000 ) ;
+
+	if (lastScore<target && currentScore>=target){
 		Pacman::getInstance()->addLives(1);
 		Sounds::getInstance()->playSingleSound(Sounds::EXTRA_MAN);
 	}

also, pacman_sdl-ebuc99-git is an AUR package for this repo

I'm using MSYS2, and the build currently fails. The cause might be a misconfiguration of the project, leading to G++ trying to link with WinMain (it is not present in SDL2 and any of it's sub-libraries).

Here are the errors I get when trying to make : https://pastebin.com/pXzf8gnm

Please, can you verify license for your included fonts?
Are these fonts:

• pacman/data/fonts/comic.ttf
• pacman/data/fonts/comicbd.ttf

from MS? If so, they doesn't distributed under GNU GPLv2.

Thanks for your attention!

Hi! Thanks for providing a great Pacman game!

The icon isn't really correctly handled on Linux. The correct way of doing it is putting the icon in to /usr/share/icons and then setting the icon in the .desktop file to be just pacman. This will let icon themes that have a pacman icon use that and if the icon theme doesn't have one, then it will use your icon.

Link for more information:
https://askubuntu.com/questions/435603/desktop-files-how-to-specify-the-icon-path

Example of Qsynth where it puts it's icons:
https://dev.getsol.us/source/qsynth/browse/master/pspec_x86_64.xml