mpo/VVVVVV
1
0
Fork 0
mirror of https://github.com/TerryCavanagh/VVVVVV.git synced 2024-12-23 01:59:43 +01:00
Code Issues Releases Wiki Activity

Custom levels: only count inbounds trinkets/crew

Browse source 
In the past, people have reported having glitched levels where they
can't get the trinket star or can't complete the level because the
number of trinkets or crewmates is one higher than what can be obtained
in the level.

How did this happen? Well, it turns out that if you place an entity, and
then resize the level to be smaller, that entity still exists. This is
inconsequential for most entities, but if the entity is a trinket or
crewmate, that entity is still counted towards the number of trinkets or
crewmates in the level.

One fix would be to just remove entities whenever the level is
downsized, but then if someone accidentally downsizes the level and
wants to go back, that entity will be gone. Plus, it would be
inconsistent with tiles, because tiles don't get removed when you
downsize the level. Also, it wouldn't fix existing levels where people
have managed to place trinkets or crewmates out of bounds.

So instead, ed.numtrinkets() and ed.numcrewmates() should simply ignore
trinkets and crewmates that are outside the playable area. That way,
levels with glitched trinkets and crewmates can still be completed, and
can still be completed with the trinket star.
This commit is contained in:
Misa 2021-06-19 10:19:07 -07:00 committed by Ethan Lee
parent 571f6a7098
commit 88d31ab3b6
1 changed files with 11 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
desktop_version/src/editor.cpp
View file

@ -5895,12 +5895,21 @@ Uint32 editorclass::getonewaycol(void)
return graphics.getRGB(255, 255, 255); return graphics.getRGB(255, 255, 255);
} }
static bool inbounds(const edentities* entity)
{
extern editorclass ed;
return entity->x >= 0
&& entity->y >= 0
&& entity->x < ed.mapwidth * 40
&& entity->y < ed.mapheight * 30;
}
int editorclass::numtrinkets(void) int editorclass::numtrinkets(void)
{ {
int temp = 0; int temp = 0;
for (size_t i = 0; i < edentity.size(); i++) for (size_t i = 0; i < edentity.size(); i++)
{ {
if (edentity[i].t == 9) if (edentity[i].t == 9 && inbounds(&edentity[i]))
{ {
temp++; temp++;
} }
@ -5913,7 +5922,7 @@ int editorclass::numcrewmates(void)
int temp = 0; int temp = 0;
for (size_t i = 0; i < edentity.size(); i++) for (size_t i = 0; i < edentity.size(); i++)
{ {
if (edentity[i].t == 15) if (edentity[i].t == 15 && inbounds(&edentity[i]))
{ {
temp++; temp++;
} }