Skip Menu |

This queue is for tickets about the Linux-MemInfo CPAN distribution.

Report information
The Basics
Id: 93079
Status: new
Priority: 0/
Queue: Linux-MemInfo

People
Owner: Nobody in particular
Requestors: SREZIC [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Wishlist
Broken in: 0.03
Fixed in: (no value)



Subject: Better diagnostics if /proc/meminfo could not be opened
Currently the following line is used to open /proc/meminfo: open(INFIL,"/proc/meminfo") || die("Unable To Open /proc/meminfo\n"); Two problems here: - The reason for the problem is missing --- the die() message should probably add also $! - Because of the trailing "\n" it's not visible where the problem happens. It's probably better to remove the "\n" here. Regards, Slaven
On 2014-02-17 04:17:58, SREZIC wrote: Show quoted text
> Currently the following line is used to open /proc/meminfo: > > open(INFIL,"/proc/meminfo") || die("Unable To Open /proc/meminfo\n"); > > Two problems here: > - The reason for the problem is missing --- the die() message should > probably add also $! > - Because of the trailing "\n" it's not visible where the problem > happens. It's probably better to remove the "\n" here. > > Regards, > Slaven
Possible patch: http://cpan.cpantesters.org/authors/id/S/SR/SREZIC/patches/Linux-MemInfo-0.03-RT93079.patch