Search Unity

  1. Unity Asset Manager is now available in public beta. Try it out now and join the conversation here in the forums.
    Dismiss Notice

Resolved InitializeHideInCallstackMethodsCache() throwing

Discussion in '2022.2 Beta' started by SugoiDev, May 19, 2022.

  1. SugoiDev

    SugoiDev

    Joined:
    Mar 27, 2013
    Posts:
    395
    Just notice this method throwing while testing a13.

    The problem is in this line (github permalink). It seems it is possible to create identical HideInCallstackGenericMethodKey structs, so that line will throw when the key is already there.
    The immediate fix is to use TryAdd(method) and bail if returns false.

    I will see if I can submit a proper bug report, but I'm a bit pressed for time right now, so I decided to at least get this threat going in the meantime.


    Thanks for reading!
     
    SugoiDev, May 19, 2022
    #1

  2. claudiaantoun

    claudiaantoun

    Unity Technologies

    Joined:
    May 14, 2021
    Posts:
    9
    Thanks for looking into this! I'll fix it as soon as possible.
     
    claudiaantoun, Jun 2, 2022
    #2
    JonSWEDEN and SugoiDev like this.

  3. SugoiDev

    SugoiDev

    Joined:
    Mar 27, 2013
    Posts:
    395
    I did a bit of digging today and might have found the cause.
    It seems it doesn't support overloads.
    Using TryAdd() instead of Add() is a possible fix.

    Case #IN-8307
     
    SugoiDev, Jun 25, 2022
    #3

  4. claudiaantoun

    claudiaantoun

    Unity Technologies

    Joined:
    May 14, 2021
    Posts:
    9
    Hey @SugoiDev,

    Thank you for looking into this, and apologies for the delay.
    The bug has been fixed, but hasn't been added to the 2022.2 version.
    I will let you know as soon as it has!
     
    claudiaantoun, Jul 11, 2022
    #4
    SugoiDev likes this.

  5. claudiaantoun

    claudiaantoun

    Unity Technologies

    Joined:
    May 14, 2021
    Posts:
    9
    Hey @SugoiDev,
    The fix should have landed in the latest version of 2022.2.
     
    claudiaantoun, Aug 12, 2022
    #5
    SugoiDev likes this.