Looks like for 5.x versions, file_get_contents() doesn't support the "flags" argument anyway, and that we should simply drop the 2nd (FILE_BINARY) argument from the call.

i.e. change:
$contents = file_get_contents($file_name, FILE_BINARY); 
to 
$contents = file_get_contents($file_name);

Can you try that out to see if it fixes your issue? 

When we get a chance, we'll try it out as well, and put in a fix.

> Can you try that out to see if it fixes your issue? 

Tried it - solves the problem.

Looking closer at the purpose of FILE_BINARY, I believe that relates to purely PHP6 features anyway - it tells PHP6 not to try to be smart about character set handling / conversion when reading a file.

Harry,

Thanks for reporting the issue and trying out the fix.

--

To summarize/clarify: This issue shouldn't cause any functional incorrectness. It'll cause notice messages to be spewed when browsing your report. We'll put in the fix.

Also note: the issue exists in XHProfRuns_Default() which is just meant to be a sample implementation of the iXHProfRuns interface. 

We anticipate users will provide their own implementation of iXHProfRuns (e.g., to save the results to a central DB, or to compress the data before saving, etc.).

For example, in the save_run() step you can do a:
   gzcompress(serialize($xhprof_data)

and in the get_run() you can return:
    unserialize(gzuncompress($contents))

This bug has been fixed in CVS.

In case this was a documentation problem, the fix will show up at the
end of next Sunday (CET) on pecl.php.net.

In case this was a pecl.php.net website problem, the change will show
up on the website in short time.
 
Thank you for the report, and for helping us make PECL better.

cvs head has the fix now. 

The next release (0.9.2) should have it as well.