summaryrefslogtreecommitdiffstats
path: root/liblog/fake_log_device.c
diff options
context:
space:
mode:
authorMark Salyzyn <salyzyn@google.com>2016-02-16 10:35:06 -0800
committerMark Salyzyn <salyzyn@google.com>2016-03-10 14:44:22 -0800
commit029c7373801b9e1791df334efa36588c216b4ab2 (patch)
tree4910afc300e3855f015db1afe72ef7464e5a2a25 /liblog/fake_log_device.c
parent8edbbe1dc148006e4cd588f42f580bfdaa7ae585 (diff)
downloadcore-029c7373801b9e1791df334efa36588c216b4ab2.tar.gz
core-029c7373801b9e1791df334efa36588c216b4ab2.tar.bz2
core-029c7373801b9e1791df334efa36588c216b4ab2.zip
liblog: document fakeLogClose never to be called
(cherry pick from commit bc81b17e06cb23501eeca03a530b3a90858ac7c3) Bug: 27107691 Change-Id: Ia699646ec2e83ca46f8b62b5d4f144e533b36074
Diffstat (limited to 'liblog/fake_log_device.c')
-rw-r--r--liblog/fake_log_device.c11
1 files changed, 11 insertions, 0 deletions
diff --git a/liblog/fake_log_device.c b/liblog/fake_log_device.c
index 20e409861..aa88bedf8 100644
--- a/liblog/fake_log_device.c
+++ b/liblog/fake_log_device.c
@@ -691,6 +691,17 @@ int fakeLogOpen(const char *pathName, int flags)
return redirectOpen(pathName, flags);
}
+/*
+ * The logger API has no means or need to 'stop' or 'close' using the logs,
+ * and as such, there is no way for that 'stop' or 'close' to translate into
+ * a close operation to the fake log handler. fakeLogClose is provided for
+ * completeness only.
+ *
+ * We have no intention of adding a log close operation as it would complicate
+ * every user of the logging API with no gain since the only valid place to
+ * call is in the exit handler. Logging can continue in the exit handler to
+ * help debug HOST tools ...
+ */
int fakeLogClose(int fd)
{
/* Assume that open() was called first. */