<feed xmlns='http://www.w3.org/2005/Atom'>
<title>system_core/liblog, branch replicant-10</title>
<subtitle>Fork of system/core
</subtitle>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant-next/system_core/'/>
<entry>
<title>Merge "Version liblog LL-NDK library"</title>
<updated>2019-04-02T23:16:22+00:00</updated>
<author>
<name>Treehugger Robot</name>
<email>treehugger-gerrit@google.com</email>
</author>
<published>2019-04-02T23:16:22+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant-next/system_core/commit/?id=92c9e8ab7f5c54c2b616b49a038b2a113b7bd017'/>
<id>92c9e8ab7f5c54c2b616b49a038b2a113b7bd017</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>Version liblog LL-NDK library</title>
<updated>2019-04-02T19:22:50+00:00</updated>
<author>
<name>Vic Yang</name>
<email>victoryang@google.com</email>
</author>
<published>2019-04-02T19:22:50+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant-next/system_core/commit/?id=270f65ffdb4e789407bc18b50c1e05d15b430183'/>
<id>270f65ffdb4e789407bc18b50c1e05d15b430183</id>
<content type='text'>
The system version of liblog has been versioned.  Do the same for
LL-NDK verison as well.

Test: Build and examine the dynamic symbols.
Change-Id: Ie63cc845c891d9372fb1990e713b3fe4f0f2f723
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The system version of liblog has been versioned.  Do the same for
LL-NDK verison as well.

Test: Build and examine the dynamic symbols.
Change-Id: Ie63cc845c891d9372fb1990e713b3fe4f0f2f723
</pre>
</div>
</content>
</entry>
<entry>
<title>liblog: don't return 0xFFFFFFFF as an invalid log id.</title>
<updated>2019-03-25T23:41:39+00:00</updated>
<author>
<name>Josh Gao</name>
<email>jmgao@google.com</email>
</author>
<published>2019-03-25T21:55:39+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant-next/system_core/commit/?id=c6ad69d1d467580455aacc2399e155610f76def6'/>
<id>c6ad69d1d467580455aacc2399e155610f76def6</id>
<content type='text'>
There are a bunch of branches that check "id &gt;= LOG_ID_MAX", but because
C++ hates you, this does a promotion to signed int despite the
fact that both sides of the comparison are the same enum with an
underlying type of unsigned int. (C++17 §7.6.3)

Return LOG_ID_MAX instead of a value that gets promoted to signed -1, to
avoid this.

Bug: http://b/129272512
Test: /data/nativetest64/logcat-unit-tests/logcat-unit-tests
Change-Id: I4b3ee662d76d5cc80d9a9625d17f7e5b5980de41
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
There are a bunch of branches that check "id &gt;= LOG_ID_MAX", but because
C++ hates you, this does a promotion to signed int despite the
fact that both sides of the comparison are the same enum with an
underlying type of unsigned int. (C++17 §7.6.3)

Return LOG_ID_MAX instead of a value that gets promoted to signed -1, to
avoid this.

Bug: http://b/129272512
Test: /data/nativetest64/logcat-unit-tests/logcat-unit-tests
Change-Id: I4b3ee662d76d5cc80d9a9625d17f7e5b5980de41
</pre>
</div>
</content>
</entry>
<entry>
<title>Suppress -Wstring-plus-int warning</title>
<updated>2019-03-19T05:16:14+00:00</updated>
<author>
<name>Yi Kong</name>
<email>yikong@google.com</email>
</author>
<published>2019-03-19T05:16:14+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant-next/system_core/commit/?id=dd2ea5fd682b797d60ef138a9b81330830185fb7'/>
<id>dd2ea5fd682b797d60ef138a9b81330830185fb7</id>
<content type='text'>
The code section is already marked as to be removed, simply suppress
the -Wstring-plus-int warning.

Test: m checkbuild
Bug: 128878287
Change-Id: I95a03aca90dbb5d27db49f5baf534cbaacf9b830
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The code section is already marked as to be removed, simply suppress
the -Wstring-plus-int warning.

Test: m checkbuild
Bug: 128878287
Change-Id: I95a03aca90dbb5d27db49f5baf534cbaacf9b830
</pre>
</div>
</content>
</entry>
<entry>
<title>__android_log_is_debuggable() shouldn't be an apex symbol</title>
<updated>2019-02-20T20:52:02+00:00</updated>
<author>
<name>Tom Cherry</name>
<email>tomcherry@google.com</email>
</author>
<published>2019-02-20T20:51:33+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant-next/system_core/commit/?id=15800946e09f7272fe487cbebb4c9e009cb59811'/>
<id>15800946e09f7272fe487cbebb4c9e009cb59811</id>
<content type='text'>
Remove a superfluous check in fs_mgr, since those return values are
already impossible for a non-debuggable build.  Replace a one time
call to __android_log_is_debuggable(), since there's no reason to
cache the value or use this symbol.

Test: build
Change-Id: Icd4bef7b616c49d304303747388d7e3018c6fcfc
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Remove a superfluous check in fs_mgr, since those return values are
already impossible for a non-debuggable build.  Replace a one time
call to __android_log_is_debuggable(), since there's no reason to
cache the value or use this symbol.

Test: build
Change-Id: Icd4bef7b616c49d304303747388d7e3018c6fcfc
</pre>
</div>
</content>
</entry>
<entry>
<title>liblog: add stubs variant</title>
<updated>2019-02-20T20:32:12+00:00</updated>
<author>
<name>Tom Cherry</name>
<email>tomcherry@google.com</email>
</author>
<published>2019-02-08T19:55:36+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant-next/system_core/commit/?id=fe1f7017617d1e8b4b68cac15bf659adc88790b2'/>
<id>fe1f7017617d1e8b4b68cac15bf659adc88790b2</id>
<content type='text'>
Ensure that only the symbols in liblog.map.txt can be used by the
platform.

Bug: 123349183
Test: build
Change-Id: I99ae5d0e8ba8f5061ec20701c941d861c9eb615d
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Ensure that only the symbols in liblog.map.txt can be used by the
platform.

Bug: 123349183
Test: build
Change-Id: I99ae5d0e8ba8f5061ec20701c941d861c9eb615d
</pre>
</div>
</content>
</entry>
<entry>
<title>Merge "liblog: make liblog_headers truly stand-alone."</title>
<updated>2019-02-16T00:12:51+00:00</updated>
<author>
<name>Elliott Hughes</name>
<email>enh@google.com</email>
</author>
<published>2019-02-16T00:12:51+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant-next/system_core/commit/?id=c911d5af6f92ac88680c766c9b12869875465fac'/>
<id>c911d5af6f92ac88680c766c9b12869875465fac</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>liblog: make liblog_headers truly stand-alone.</title>
<updated>2019-02-13T20:42:06+00:00</updated>
<author>
<name>Elliott Hughes</name>
<email>enh@google.com</email>
</author>
<published>2019-02-13T20:42:06+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant-next/system_core/commit/?id=3c3bdc128f1a71014db3d601a6efe25cc8d691d7'/>
<id>3c3bdc128f1a71014db3d601a6efe25cc8d691d7</id>
<content type='text'>
We don't want to imply dependencies on libc or libc++.

Bug: https://issuetracker.google.com/issues/119713191
Test: boots
Change-Id: I4abe312e73fad3ca8e96abd328b16387507baed4
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
We don't want to imply dependencies on libc or libc++.

Bug: https://issuetracker.google.com/issues/119713191
Test: boots
Change-Id: I4abe312e73fad3ca8e96abd328b16387507baed4
</pre>
</div>
</content>
</entry>
<entry>
<title>liblog: simplify socket_local_client() and always use CLOEXEC</title>
<updated>2019-02-12T20:53:19+00:00</updated>
<author>
<name>Tom Cherry</name>
<email>tomcherry@google.com</email>
</author>
<published>2019-02-12T19:26:23+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant-next/system_core/commit/?id=05de1ba38eea7675c7af547c6e608417fd3ca1bc'/>
<id>05de1ba38eea7675c7af547c6e608417fd3ca1bc</id>
<content type='text'>
socket_local_client() was copied from libcutils, but we only need a
small subset of its functionality.  We instead create our own version
with just the needed parts.

Importantly, use CLOEXEC in this new function and other places where
it was missing previously.

Test: logging works, liblog-unit-tests
Change-Id: Ifb929227af67bafa13e391eab92358d9f6fe6450
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
socket_local_client() was copied from libcutils, but we only need a
small subset of its functionality.  We instead create our own version
with just the needed parts.

Importantly, use CLOEXEC in this new function and other places where
it was missing previously.

Test: logging works, liblog-unit-tests
Change-Id: Ifb929227af67bafa13e391eab92358d9f6fe6450
</pre>
</div>
</content>
</entry>
<entry>
<title>liblog: remove visibility macros</title>
<updated>2019-02-08T19:48:19+00:00</updated>
<author>
<name>Tom Cherry</name>
<email>tomcherry@google.com</email>
</author>
<published>2019-02-08T19:46:19+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/replicant-next/system_core/commit/?id=2d9779e87b110b33745de9cec006f205396bef5d'/>
<id>2d9779e87b110b33745de9cec006f205396bef5d</id>
<content type='text'>
Visibility is managed through version scripts now.

Bug: 123349183
Test: build
Change-Id: Ia388611a2ec14d0ff9c3896cfe97ccdce17dbb8b
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Visibility is managed through version scripts now.

Bug: 123349183
Test: build
Change-Id: Ia388611a2ec14d0ff9c3896cfe97ccdce17dbb8b
</pre>
</div>
</content>
</entry>
</feed>
