User Tag List

Results 1 to 10 of 10

Thread: Subscreen crashes ZC in weird instances

  1. #1
    Banned
    Join Date
    Jun 2017
    Posts
    17
    Mentioned
    4 Post(s)
    Tagged
    0 Thread(s)
    vBActivity - Stats
    Points
    144
    Level
    4
    vBActivity - Bars
    Lv. Percent
    39.67%

    Dead Bug Subscreen crashes ZC in weird instances

    I'll provide an example quest containing the problematic subscreen, since I have not been able to replicated it, and I'm unsure what even causes it: Download for the quest file

    The specific steps taken to cause the crash:
    1. Have an arrow item in your inventory but not any other equip-able items.
    2. Press start to open the active subscreen.

    I found this bug in 2.52 (?) (v2.50 Build 29 is listed in about), one of the more recent stable releases. I sent the quest file to Avataro for a second opinion and it crashes for him to in 2.53 beta 8.

    Something is obvious wrong with the subscreen, but neither he nor I could seem to replicate it. Usually when something is a problem with the subscreen it just stops working though, rather than straight up causing ZC to crash.

  2. #2
    Is this the end?
    ZC Developer
    Saffith's Avatar
    Join Date
    Jan 2001
    Age
    41
    Posts
    3,389
    Mentioned
    178 Post(s)
    Tagged
    6 Thread(s)
    vBActivity - Stats
    Points
    6,430
    Level
    24
    vBActivity - Bars
    Lv. Percent
    69.57%
    It's crashing trying to write the name. subscr.cpp, line 3636:
    Code:
                        if(Bitem->dummy_bool[0]==true)  //if we also have a bow
    Bitem is null. A simple Bitem && seems to be sufficient.

  3. #3
    The Timelord
    QDB Manager
    ZC Developer

    Join Date
    Oct 2006
    Location
    Prydon Academy
    Posts
    1,396
    Mentioned
    112 Post(s)
    Tagged
    1 Thread(s)
    vBActivity - Stats
    Points
    4,760
    Level
    21
    vBActivity - Bars
    Lv. Percent
    68.7%
    Quote Originally Posted by Saffith View Post
    It's crashing trying to write the name. subscr.cpp, line 3636:
    Code:
                        if(Bitem->dummy_bool[0]==true)  //if we also have a bow
    Bitem is null. A simple Bitem && seems to be sufficient.
    I put that in a pull request for 2.50.x for the present. Thank you @Saffith for the quick fix, and thank you @Lunaria for the report and the example files.

    https://github.com/ArmageddonGames/Z...assic/pull/129

    Needs to be applied to master too, but a lot of this is different, there. I will be doing a re-merge of my local and fork assets in a few days to synchronise.

  4. #4
    Administrator DarkDragon's Avatar
    Join Date
    Oct 2001
    Posts
    6,228
    Mentioned
    70 Post(s)
    Tagged
    0 Thread(s)
    vBActivity - Stats
    Points
    11,025
    Level
    31
    vBActivity - Bars
    Lv. Percent
    8.16%
    I'm not sure what you mean by a "re-merge of local and fork assets" but the command to pull changes from the remote repository is simply

    git pull

    In any case I've applied Saffith's fix to both 2.50.x and master (the fix is identical).

  5. #5
    The Timelord
    QDB Manager
    ZC Developer

    Join Date
    Oct 2006
    Location
    Prydon Academy
    Posts
    1,396
    Mentioned
    112 Post(s)
    Tagged
    1 Thread(s)
    vBActivity - Stats
    Points
    4,760
    Level
    21
    vBActivity - Bars
    Lv. Percent
    68.7%
    Quote Originally Posted by DarkDragon View Post
    I'm not sure what you mean by a "re-merge of local and fork assets" but the command to pull changes from the remote repository is simply

    git pull

    In any case I've applied Saffith's fix to both 2.50.x and master (the fix is identical).
    I know. :)
    I had local files that were WIP that were not oin the repo, filesthat were only local, and files in the rep
    o that were different and needed git-merge.

    I fixed all of it, stashed the uncommitted files, backed up my local files, did a merge from ANG/master to my remote, and fixed all of the locals.

    This happened because we were both doing different things with files such as zdefs, and I helted all of my commits until you finished, but the merging process was a pain. I still have changes that I need to reapply and push, but my priority is to commit the 2.53 changes to master, before I do anything else.

    That is why I asked if someone would handle the commits to master while I bugfixed 2.53. I will be committing the latest stuff soon, to master, but I wanted to perfect the fixes in 2.53 first, as it is easier to deduce if bugs are stemming from specific factors in that (the most stable) branch.

    Your changes to water need redaction. They introduced bugs that are worse than what was reported in the logs from Lut.

  6. #6
    Administrator DarkDragon's Avatar
    Join Date
    Oct 2001
    Posts
    6,228
    Mentioned
    70 Post(s)
    Tagged
    0 Thread(s)
    vBActivity - Stats
    Points
    11,025
    Level
    31
    vBActivity - Bars
    Lv. Percent
    8.16%
    What water bugs? Where is the report thread?

    I saw something on Pure about Link getting knocked into water by damage, and haven't looked into it yet, but this should be independent from any of the changes to Link walking between land and water across screens...

  7. #7
    The Time-Loop Continues ZC Developer
    Gleeok's Avatar
    Join Date
    Apr 2007
    Posts
    4,815
    Mentioned
    259 Post(s)
    Tagged
    10 Thread(s)
    vBActivity - Stats
    Points
    12,933
    Level
    33
    vBActivity - Bars
    Lv. Percent
    23.44%
    We tried to put a single thread at purezc for bug reports for the peeps that only go there, but it somehow began replicating itself and spreading past the quarantine zone. Kind of like Zombies. In fact it's exactly like Zombies.
    This post contains the official Gleeok seal of approval. Look for these and other posts in an area near you.

  8. #8
    Administrator DarkDragon's Avatar
    Join Date
    Oct 2001
    Posts
    6,228
    Mentioned
    70 Post(s)
    Tagged
    0 Thread(s)
    vBActivity - Stats
    Points
    11,025
    Level
    31
    vBActivity - Bars
    Lv. Percent
    8.16%
    I've fixed the one issue I found on Pure (Link pops out of water when hit by his own fire/boms due to some buggy logic in the collision code). I couldn't find any posts about problems with the fix to land/water transitions across mapscreens, but I'll take a look if you point them out to me.

  9. #9
    The Timelord
    QDB Manager
    ZC Developer

    Join Date
    Oct 2006
    Location
    Prydon Academy
    Posts
    1,396
    Mentioned
    112 Post(s)
    Tagged
    1 Thread(s)
    vBActivity - Stats
    Points
    4,760
    Level
    21
    vBActivity - Bars
    Lv. Percent
    68.7%
    Quote Originally Posted by DarkDragon View Post
    I've fixed the one issue I found on Pure (Link pops out of water when hit by his own fire/boms due to some buggy logic in the collision code). I couldn't find any posts about problems with the fix to land/water transitions across mapscreens, but I'll take a look if you point them out to me.


    Good, good: I fixed the NES doors (diagonal movement) and pushed it to 2.50.x. I'll start integrating the changes to master, after I sleep.

    I asked in the pull if I should make this a QR. Probably not, but the sensitivity could be in quest data in the future, or in FFScript:oorTimer.

  10. #10
    Administrator DarkDragon's Avatar
    Join Date
    Oct 2001
    Posts
    6,228
    Mentioned
    70 Post(s)
    Tagged
    0 Thread(s)
    vBActivity - Stats
    Points
    11,025
    Level
    31
    vBActivity - Bars
    Lv. Percent
    8.16%
    I've posted some comments on the pull request on GitHub.

    A quest rule is not out of the question (if you add one, please make sure it's set up consistently across 2.50.x and master, including its position in the bit field!!) but I don't think it's absolutely required here, unless you can think of a reason the gameplay of an old quest might be affected.

Thread Information

Users Browsing this Thread

There are currently 1 users browsing this thread. (0 members and 1 guests)

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •  
About us
Armageddon Games is a game development group founded in 1997. We are extremely passionate about our work and our inspirations are mostly drawn from games of the 8-bit and 16-bit era.
Social