Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Developer Tools (fixed, improved, expanded) #601

Open
wants to merge 37 commits into
base: master
Choose a base branch
from

Conversation

AquariusPower
Copy link
Contributor

@AquariusPower AquariusPower commented May 8, 2020

Developer Tools:

  • Moved wizard auto play code to it's own files, and improved it a bit.
  • Created cursed developer game mode, where player won't die (neither score), to skip/avoid any conditions governed by wizard mode.
  • Expanded commands list on developer console and fixed it (thx @fejoa!).

copy from #587, merged with meld on linux
I will check if there are other things that can be split from that PR w/o getting overly complicated.
Some splits required new code dependencies that are now in 3+ branches.
May be that PR becomes something like "Some Fixes" just as initially intended.

AquariusPower and others added 30 commits May 24, 2018 16:22
(ready) bug fix for show items under (Attnam#369)
(ready) Fantasy name generator (Attnam#363)
Moved wizard auto play code to it's own files, and improved it a bit.
Created cursed developer game mode, where player won't die (neither score).
Expanded commands list on developer console and fixed it (was broken).
- now commands can be many separated by ';' on the same line.
- added a command DbgSetVar to let debug variables be easily set/changed while the game is running.
@ryfactor
Copy link
Member

@AquariusPower Are you able to update this PR? I get a error: 'GetUserDataDir' is not a member of 'game' in char.cpp when I try to compile.

if(curseddeveloper::IsCursedDeveloper())
return;

highscore HScore(game::GetUserDataDir() + HIGH_SCORE_FILENAME);
Copy link
Contributor Author

@AquariusPower AquariusPower Jan 23, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I saw the error here:
https://github.com/Attnam/ivan/runs/4004758260?check_suite_focus=true
I think just removing game:: will solve it like:
highscore HScore(GetUserDataDir() + HIGH_SCORE_FILENAME);
I downloaded my DevTools branch and found no other matches for game::GetUserDataDir

fix build error about `game::GetUserDataDir()`
@AquariusPower
Copy link
Contributor Author

AquariusPower commented Jan 23, 2022

dungeon.cpp

/home/runner/work/ivan/ivan/Main/Source/dungeon.cpp:130:57: error: ‘GetAutoPlayMode’ is not a member of ‘game’
  130 |                              Displacement, WHITE, game::GetAutoPlayMode()<2,

Some things from wizautoplay.h(.cpp) were merged into Attnam:master in the past?
like GetAutoPlayMode() that was copied into game.h(.cpp)?
But now I think I may have created wizautoplay.h after and moved from game.h to it.
Anyway there are some inconsistencies like dungeon.cpp expects GetAutoPlayMode() to be at game.h(.cpp), but DevTools have these only at wizautoplay.h(.cpp).

I made this backup https://github.com/AquariusPower/ivan/tree/DevTools.TempBackup of this branch.
I will see what I can do.

@AquariusPower
Copy link
Contributor Author

AquariusPower commented Jan 28, 2022

I still havent found a way to install libsdl2-dev on ubuntu20.04 (there is some dep problems that makes no sense to me and would mess everything, it just wont install anyway).
If someone knows how to setup SDL2 development environment on ubuntu20.04 I would like to know :) (I thought on downloading and extracting all packages to some user folder, and just configure the terminal environment to look there, I guess it could work)
So, this means I fixed online only and could not test on my machine. It will compile but I could not test the way I like: run it to see if it is working.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants