Re: LoA from XS

The 'for' loop is not quite correct.
This rendition of it works for me:

        for( j = 0; j <= av_len( av ); ++j ) {

            SV **sv2 =  av_fetch( av, j, 0  );
            char * str = SvPVX(*sv2);

            if(sv2 != NULL) printf( "[%d][%d] %x %d '%s'\n",
                i, j, *sv2, SvTYPE( *sv2 ), str );
        }
[download]

Cheers,
Rob

Comment on Re: LoA from XS Download Code

Replies are listed 'Best First'.
Re^2: LoA from XS by BrowserUk (Patriarch) on Apr 21, 2009 at 01:06 UTC
Yep! That works here also. Many thanks! Now I just need to undestand the changes. Update: Got it. Casting when I should have dereferenced. 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. "Too many [] have been sedated by an oppressive environment of political correctness and risk aversion."	[reply]
Re^3: LoA from XS by ikegami (Patriarch) on Apr 21, 2009 at 01:20 UTC
Now I just need to undestand the changes. Two problems were fixed. `av_fetch` returns a pointer to a SV* (which may be NULL). It needs to be dereferenced into a `SV`, not casted to `SV`. Each element of the array is a string, not a reference to the string. The `SvRV` was extraneous.	[reply] [d/l] [select]