diff options
author | Mark Salyzyn <salyzyn@google.com> | 2016-02-16 10:35:06 -0800 |
---|---|---|
committer | Mark Salyzyn <salyzyn@google.com> | 2016-03-10 14:44:22 -0800 |
commit | 029c7373801b9e1791df334efa36588c216b4ab2 (patch) | |
tree | 4910afc300e3855f015db1afe72ef7464e5a2a25 /liblog/fake_log_device.c | |
parent | 8edbbe1dc148006e4cd588f42f580bfdaa7ae585 (diff) | |
download | core-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.c | 11 |
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. */ |