From c2d8bfdf59ac20795e4d43fdad304911150e25b3 Mon Sep 17 00:00:00 2001 From: Elliott Hughes Date: Tue, 9 Apr 2024 16:16:03 +0000 Subject: [PATCH] Don't define LINE_MAX. It's a POSIX mistake we'd been avoiding, but finally added to bionic. It's unclear to me that this code shouldn't just be using fscanf() and getline() to simplify the code and avoid arbitrary hard-coded limits -- since the code does do some allocation already -- but that's a bigger cleanup for a code owner. Change-Id: Ibab28f082a084c1f933ee6a39aa4ee3fa7868e5a --- include/lmkd.h | 5 ----- 1 file changed, 5 deletions(-) diff --git a/include/lmkd.h b/include/lmkd.h index c57b244..f57548a 100644 --- a/include/lmkd.h +++ b/include/lmkd.h @@ -52,11 +52,6 @@ enum lmk_cmd { */ #define CTRL_PACKET_MAX_SIZE (sizeof(int) * (MAX_TARGETS * 2 + 1)) -/* - * Max number of characters in line. - */ -#define LINE_MAX 128 - /* LMKD packet - first int is lmk_cmd followed by payload */ typedef int LMKD_CTRL_PACKET[CTRL_PACKET_MAX_SIZE / sizeof(int)];