in reply to Re^7: odd line in windows
in thread odd line in windows

Update: This is wrong. should have looked at what newSVrv did. 

    SV*      obj_ref = newSViv(0);
    SV*      obj = newSVrv(obj_ref, class);

[download]
should be 
    SV*      obj = newSViv(0);
    SV*      obj_ref = newSVrv(obj, class);

[download]

Replies are listed 'Best First'.
Re^9: odd line in windows
by BrowserUk (Patriarch) on Sep 08, 2011 at 20:12 UTC

    That's essentially the same thing tye said above.

    But, that still leaves the questions

    • How come it works as erroneously shown in the documentation?
    • What happens to all the code that has been written based upon that bad documentation, now that the IV and RV fields have been combined?
    Examine what is said, not who speaks -- Silence betokens consent -- Love the truth but pardon error.
    "Science is about questioning the status quo. Questioning authority".
    In the absence of evidence, opinion is indistinguishable from prejudice.
[reply]

      I stand corrected. The code in the recipe is actually correct. 

      SV* obj_ref = newSViv(0);
SV* obj = newSVrv(obj_ref, class);

      [download]

      newSVrv doesn't create a reference as a I had guessed, it returns a new SV and makes obj_ref a reference to it. WTF? How odd is that!!!

      So obj_ref holds a reference to the blessed scalar in obj, which in turn contains an integer which corresponds to the memory address of the C object.

      set_iv is not being used on the reference, so there are no issues wrt to recent changes in scalar types.

[reply]
[d/l]
[select]
        newSVrv doesn't create a reference as a I had guessed, it returns a new SV and makes obj_ref a reference to it. WTF? How odd is that!!!

        Very odd!

        There's nothing like consistency:

        1. newSViv: Creates a new SV and copies an integer into it
        2. newSVnv: Creates a new SV and copies a floating point value into it.
        3. newSVpv: Creates a new SV and copies a string into it.
        4. newSVrv: Creates a new SV for the RV, rv, to point to.

        And that's nothing like consistent! newRVsv() would make more sense (IMO).

        Examine what is said, not who speaks -- Silence betokens consent -- Love the truth but pardon error.
        "Science is about questioning the status quo. Questioning authority".
        In the absence of evidence, opinion is indistinguishable from prejudice.
[reply]
[d/l]

      Update: This is wrong. should have looked at what newSVrv did.

      Read more... (1503 Bytes)
[reply]
[d/l]
[select]

In Section Seekers of Perl Wisdom