-
Notifications
You must be signed in to change notification settings - Fork 9
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
Duct Tape + Do While + Zero Cost Card #2
Comments
I just tested this with a Do While+ and Claw. With 3 energy:
Claw played 3+1 times. |
Well this might be a confounding bug -- but I just reproduced it (differently) with AstralHammer (not sure if that mattered). I played DoWhile + Claw on the awaken one. After he died, it kept do-whiling indefinitely. I had to save and quit. Here's a screen shot of it in action: https://ibb.co/nopE6p The astralHammer interaction makes sense I believe -- it keeps setting the cost of the card that doWhile tries to increase back to zero. Not sure why you can't reproduce the original bug... I'm pretty sure I didn't have AstralHammer during it. |
I implemented a fix and was going to put out a PR, but when I went to test the fix, i noticed that all the images in your resources/ directory download as 0 byte files which are unopenable and empty. It compiles fine, but fails when trying to load the garbage file. They are corrupt if you download the zip or do a git clone, but not when you manually go and view/saveas them on github.com. It's quite strange haha. If you ever wanted to manually zip the resources folder and give me a link I'd help out with bug fixes and what not (if you'd like). |
They're not corrupt. I'm using Git LFS for all the resource files. |
Ahhhh, I've never heard of that. Thanks for the link! |
Action: Use Duct Tape on Do While + Claw, play duct tape card, assume
n
energy.Expects: Claw to be played
n+1
timesResult: Duct Tape plays infinitely many times (claws to death).
DoWhile loops itself, but since the first instance of claw is always free, it never terminates.
The text was updated successfully, but these errors were encountered: