Re: Release of xPL-Perl v0.02

[Mark Hindess <xpl@xxxxxxxxxxxxxxxxx>]

On 13 December 2005 at 22:19, "Ian Lowe" <ianlowe@xxxxxxx>
wrote:
>
> That runs much easier Mark - got the hub and logger running without
> difficulty.
>
> Next problem? It doesn't see more than half of the xpl messages flying
> across the network!

Maybe it is missing messages with a 'target' of some other device - i.e.
not '*' and not the logger itself.  I made some changes to this code for
this release but forgot to update the logger.  I've just checked in this
change:

svn diff http://www.xpl-perl.org.uk/svn/xpl-perl/trunk/bin/xpl-logger
 http://www.xpl-perl.org.uk/svn/xpl-perl/branches/r0.02/bin/xpl-logger
Index: xpl-logger
===================================================================
--- xpl-logger  (.../trunk/bin/xpl-logger)      (revision 55)
+++ xpl-logger  (.../branches/r0.02/bin/xpl-logger)     (revision 55)
@@ -63,9 +63,7 @@
my $xpl = xPL::Client->new(%args) or die "Failed to create
xPL::Client\n";

# Add a callback to receive all incoming xPL messages
-$xpl->add_xpl_callback(id => "logger",
-                       self_skip => 0, targetted => 0,
-                       callback => \&log);
+$xpl->add_xpl_callback(id => "logger", self_skip => 0,
callback => \&log);

# Run the main loop
$xpl->main_loop();

which disables the target checking code on the logger by adding the
"targetted => 0" to the xPL message callback initialisation.

> I'll have a rummage around and see if I can see a pattern, just
wondering if
> the receive is being a little too strict on case? (several apps have
not
> been converted across to use lower case only in structure elements).

I don't think so because the code should just drop to lower case for
everything but it might be.  Having said that, I don't think I have any
test cases that explicitly cover this so I might add some.

Thanks for the very useful feedback.

Regards,
Mark.