John writes:

> Not sure of this is fixed in the latest beta release, but line 779 of
> BackupPC_dump looks wierd. It appears to have unbalanced " marks.
> It reads:
> 
>       $XferLOG->write(\"$logMsg\n");
> 
> but I claim it should read:
> 
>      $XferLOG->write("$logMsg\n");

The original version is correct.  The backslash is a reference
operator: the code passes a reference to the string, rather than
the string itself.  $XferLOG->write expects a string reference.
This saves copying potentially large string arguments.

> Also when running with 'perl -w', BackupPC_dump throws a number of
> warnings including:
> 
>   "my" variable $str masks earlier declaration in same scope at
>   /tools/BackupPC-3.1.0beta0/lib/BackupPC/Xfer/BackupPCd.pm line 200.

This is benign.

Craig

-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >> http://get.splunk.com/
_______________________________________________
BackupPC-users mailing list
[email protected]
List:    https://lists.sourceforge.net/lists/listinfo/backuppc-users
Wiki:    http://backuppc.wiki.sourceforge.net
Project: http://backuppc.sourceforge.net/

Reply via email to