Re: [Perl-workflow-devel] RT #5943

[Jonas B. N. <jo...@gm...>]

Hi Grant,

After much thinking and examination I have come to the conclusion that  
XML::SAX::ParserFactory has an issue.

The following:

         if (@{$self->{KnownParsers}}) {
                 return $self->{KnownParsers}[-1]{Name};
         }
         else {
                 return "XML::SAX::PurePerl"; # backup plan!
         }

Yields unpredictable behavior based on installation order of parsers.

I would suggest addressing this and documenting how to control parser  
load.

I would rewrite it to:

         if (@{$self->{KnownParsers}}) {
                 return $self->{KnownParsers}[-1]{Name};
         }
         else {
                 return "XML::SAX::PurePerl"; # backup plan!
         }

But this breaks existing installtions and require people to revisit  
their: XML/SAX/ParserDetails.ini files.

Perhaps the best solution would be to look into the feature system,  
but this will require multiple patches to a lot of distributions in  
order to get properly propagated - and I currently do not have the  
knowledge on XML::SAX/XML::Simple to conclude that it can be  
controlled that way.

Ref: http://use.perl.org/~jonasbn/journal/37344

jonasbn

Ps. I have Cc'ed the Workflow development list to keep my fellow  
developers on workflow aware of my progress, you probably cannot post  
to the list, but I will see to that a possible response from you gets  
relayed.

On 20/08/2008, at 11.59, Grant McLean wrote:

> Hi Jonas
>
> The only thing that's holding this up is lack of time.  I will get  
> to it
> but I have more important problems to deal with first.  If you want to
> get co-maintainership of XML::SAX::RTF and do a release then go right
> ahead.
>
> Cheers
> Grant
>
> On Wed, 2008-08-20 at 09:07 +0200, Jonas Brømsø Nielsen wrote:
>> Hello Grant,
>>
>> I have been chasing this bug:
>>
>> http://www.perlmonks.org/?node_id=512040
>>
>> And I ended up at this ticket:
>>
>> http://rt.cpan.org/Ticket/Display.html?id=5943
>>
>> How are things doing with this, do you need assistance, help, test,
>> patches??
>>
>> Looking forward to hearing from you,
>>
>> jonasbn
>>
>>
>> --
>> pauseid: JONASBN
>> email: jo...@cp...
>> blog: http://use.perl.org/~jonasbn/journal/
>>
>>
>