Archived
This forum has been archived. Please start a new discussion on GitHub.
fds[1]=0
in Bug Reports
Hi,
this is not a bug report but a suggestion to make a small change in
Ice::Service::runDaemon()
The function contains the following lines:
close(fds[1]);
fds[1] = 0;
The second line is to mark the file descriptor as closed and later this value is tested.
Although it works, it is misleading because 0 is usually associated with stdin. Maybe better to use -1:
fds[1] = -1;
Cheers, Nikolai
this is not a bug report but a suggestion to make a small change in
Ice::Service::runDaemon()
The function contains the following lines:
close(fds[1]);
fds[1] = 0;
The second line is to mark the file descriptor as closed and later this value is tested.
Although it works, it is misleading because 0 is usually associated with stdin. Maybe better to use -1:
fds[1] = -1;
Cheers, Nikolai
0
Comments
-
Hi Nikolai,
I agree, it could be misleading. We'll change this.
Thanks,
- Mark0