From 80d4cb1aff77499aabc2c8b150182a7727f972ce Mon Sep 17 00:00:00 2001
From: "John Ankarstr\\xf6m" <john@ankarstrom.se>
Date: Sat, 5 Jun 2021 01:53:33 +0200
Subject: xbattext.c: Die on ioctl error

---
 xbattext.c | 15 +++++----------
 1 file changed, 5 insertions(+), 10 deletions(-)

diff --git a/xbattext.c b/xbattext.c
index aa13929..33e8877 100644
--- a/xbattext.c
+++ b/xbattext.c
@@ -44,8 +44,6 @@
 /* low battery level */
 #define ALERT 30
 
-void update(XtPointer, XtIntervalId *);
-
 /* resources */
 struct res {
 	XmFontList font_list;
@@ -82,12 +80,14 @@ enum {
 /* application state */
 Arg wargs[10];
 char *s;
-int apmfd, alerting, charging;
+int alerting, apmfd, charging, i;
 struct apm_power_info info;
 Widget toplevel, label;
 XmString xms;
 XtAppContext app_context;
 
+void update(XtPointer, XtIntervalId *);
+
 /* program start */
 int
 main(int argc, char* argv[])
@@ -127,19 +127,14 @@ main(int argc, char* argv[])
 void
 update(XtPointer client_data, XtIntervalId *t)
 {
-	int i;
-
 	/* reset temporary variables */
 	i = 0;
 	change = SET_NONE;
 
 	/* get battery info */
 	memset(&info, 0, sizeof(info));
-	if (ioctl(apmfd, APM_IOC_GETPOWER, &info) == -1) {
-		fprintf(stderr, "ioctl APM_IOC_GETPOWER failed\n");
-		sprintf(s, "?");
-		goto end;
-	}
+	if (ioctl(apmfd, APM_IOC_GETPOWER, &info) == -1)
+		err(1, "ioctl");
 
 	/* put battery status into label */
 	sprintf(s, "%d%%", info.battery_life);
-- 
cgit v1.2.3