User Tag List

Results 1 to 10 of 10

Thread: Subscreen crashes ZC in weird instances

  1. #1
    Gel Just registered Lunaria's Avatar
    Join Date
    Jun 2017
    Posts
    8
    Mentioned
    3 Post(s)
    Tagged
    0 Thread(s)
    vBActivity - Stats
    Points
    45
    Level
    2
    vBActivity - Bars
    Lv. Percent
    41.67%
    Daily Activity
    0%
    Weekly Activity
    1.13%
    Monthly Activity
    2.77%

    Verified 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.
    If you want to reach me I recommend elsewhere, I'm not really active here.

  2. #2
    Is this the end?
    ZC Developer
    Saffith's Avatar
    Join Date
    Jan 2001
    Age
    34
    Posts
    3,355
    Mentioned
    164 Post(s)
    Tagged
    6 Thread(s)
    vBActivity - Stats
    Points
    6,031
    Level
    24
    vBActivity - Bars
    Lv. Percent
    3.06%
    Daily Activity
    0%
    Weekly Activity
    1.13%
    Monthly Activity
    5.4%
    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.
    [téknolŕiz]

    ffcscript.zh 1.1.1 - Updated 2014-08-19
    ghost.zh 2.8.2 - Updated 2017-09-10
    tango.zh 1.3.0 - Updated 2017-09-10

  3. #3
    ZC Contirbutor ZoriaRPG's Avatar
    Join Date
    Oct 2006
    Location
    Prydon Academy
    Posts
    1,029
    Mentioned
    95 Post(s)
    Tagged
    1 Thread(s)
    vBActivity - Stats
    Points
    3,585
    Level
    19
    vBActivity - Bars
    Lv. Percent
    17.45%
    Daily Activity
    197.63%
    Weekly Activity
    29.38%
    Monthly Activity
    9.36%
    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.


    Featured Scripts & Headers: RPG.zh ( v. a0.97.1 ) ( RPG.zh Thread ) | Zelda 3 Thief's Town Treasure Chest Minigame (ffc) | Bobomb (enemy)
    ZScript & ZC-Related Pastebin | Quests and ZScript Repository | ZC Dev & Betas | Latest ZC 2.54 Beta | YouTube Channel
    All of the code that I create and publish here is free for use, modification and distribution under the GPL v2.0, or v3.0 where applicable..

  4. #4
    Administrator DarkDragon's Avatar
    Join Date
    Oct 2001
    Posts
    6,236
    Mentioned
    68 Post(s)
    Tagged
    0 Thread(s)
    vBActivity - Stats
    Points
    10,793
    Level
    30
    vBActivity - Bars
    Lv. Percent
    80.21%
    Daily Activity
    0%
    Weekly Activity
    1.13%
    Monthly Activity
    2.77%
    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
    ZC Contirbutor ZoriaRPG's Avatar
    Join Date
    Oct 2006
    Location
    Prydon Academy
    Posts
    1,029
    Mentioned
    95 Post(s)
    Tagged
    1 Thread(s)
    vBActivity - Stats
    Points
    3,585
    Level
    19
    vBActivity - Bars
    Lv. Percent
    17.45%
    Daily Activity
    197.63%
    Weekly Activity
    29.38%
    Monthly Activity
    9.36%
    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.


    Featured Scripts & Headers: RPG.zh ( v. a0.97.1 ) ( RPG.zh Thread ) | Zelda 3 Thief's Town Treasure Chest Minigame (ffc) | Bobomb (enemy)
    ZScript & ZC-Related Pastebin | Quests and ZScript Repository | ZC Dev & Betas | Latest ZC 2.54 Beta | YouTube Channel
    All of the code that I create and publish here is free for use, modification and distribution under the GPL v2.0, or v3.0 where applicable..

  6. #6
    Administrator DarkDragon's Avatar
    Join Date
    Oct 2001
    Posts
    6,236
    Mentioned
    68 Post(s)
    Tagged
    0 Thread(s)
    vBActivity - Stats
    Points
    10,793
    Level
    30
    vBActivity - Bars
    Lv. Percent
    80.21%
    Daily Activity
    0%
    Weekly Activity
    1.13%
    Monthly Activity
    2.77%
    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
    Developer ZC Developer
    Gleeok's Avatar
    Join Date
    Apr 2007
    Posts
    4,704
    Mentioned
    241 Post(s)
    Tagged
    10 Thread(s)
    vBActivity - Stats
    Points
    12,385
    Level
    32
    vBActivity - Bars
    Lv. Percent
    63.59%
    Daily Activity
    0%
    Weekly Activity
    18.08%
    Monthly Activity
    6.72%
    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,236
    Mentioned
    68 Post(s)
    Tagged
    0 Thread(s)
    vBActivity - Stats
    Points
    10,793
    Level
    30
    vBActivity - Bars
    Lv. Percent
    80.21%
    Daily Activity
    0%
    Weekly Activity
    1.13%
    Monthly Activity
    2.77%
    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
    ZC Contirbutor ZoriaRPG's Avatar
    Join Date
    Oct 2006
    Location
    Prydon Academy
    Posts
    1,029
    Mentioned
    95 Post(s)
    Tagged
    1 Thread(s)
    vBActivity - Stats
    Points
    3,585
    Level
    19
    vBActivity - Bars
    Lv. Percent
    17.45%
    Daily Activity
    197.63%
    Weekly Activity
    29.38%
    Monthly Activity
    9.36%
    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.


    Featured Scripts & Headers: RPG.zh ( v. a0.97.1 ) ( RPG.zh Thread ) | Zelda 3 Thief's Town Treasure Chest Minigame (ffc) | Bobomb (enemy)
    ZScript & ZC-Related Pastebin | Quests and ZScript Repository | ZC Dev & Betas | Latest ZC 2.54 Beta | YouTube Channel
    All of the code that I create and publish here is free for use, modification and distribution under the GPL v2.0, or v3.0 where applicable..

  10. #10
    Administrator DarkDragon's Avatar
    Join Date
    Oct 2001
    Posts
    6,236
    Mentioned
    68 Post(s)
    Tagged
    0 Thread(s)
    vBActivity - Stats
    Points
    10,793
    Level
    30
    vBActivity - Bars
    Lv. Percent
    80.21%
    Daily Activity
    0%
    Weekly Activity
    1.13%
    Monthly Activity
    2.77%
    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