<feed xmlns='http://www.w3.org/2005/Atom'>
<title>platform_libnativehelper/tests, branch pie-qpr3-release</title>
<subtitle>Unnamed repository; edit this file 'description' to name the repository.
</subtitle>
<link rel='alternate' type='text/html' href='https://git.replicant.us/mirrors/AOSP/platform_libnativehelper/'/>
<entry>
<title>macros: Add compile-time checking/inferencing for native method registration</title>
<updated>2018-02-22T23:33:24+00:00</updated>
<author>
<name>Igor Murashkin</name>
<email>iam@google.com</email>
</author>
<published>2018-02-16T21:30:57+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/mirrors/AOSP/platform_libnativehelper/commit/?id=a1969c41f6eb46cc5156399489024d1afe5b0904'/>
<id>a1969c41f6eb46cc5156399489024d1afe5b0904</id>
<content type='text'>
Convert existing NATIVE_METHOD/FAST_NATIVE_METHOD macros into
also checking that the JNI signature descriptor matches the C++ function
type.

For example it matches "(J)V" with "void funcname(JNIEnv*, jclass, jlong)"
with regular native methods.

Also add several new macros:
  MAKE_JNI_[FAST_|CRITICAL_]NATIVE_METHOD - Non-convention-requiring macro.
  MAKE_JNI_[FAST_|CRITICAL_]NATIVE_METHOD_AUTOSIG - Infer signature for above.
  [FAST_|CRITICAL_]NATIVE_METHOD_AUTOSIG - Convention-requiring signature inferencing.

All of the checking/inferencing is done purely at compile time and has
zero runtime cost. If any of the checks fail there will be a compilation
error with a stack trace pointing to the reason.

See jni_macros.h for exact details on check/inference rules.

Bug: 35325126
Change-Id: I98797727ca80caf7bc462582f582b23701a1b661
Test: make -j32 JniSafeRegisterNativeMethods_test \
      &amp;&amp; out/host/linux-x86/nativetest64/JniSafeRegisterNativeMethods_test/JniSafeRegisterNativeMethods_test
Test: make -j32  # build all of android, then see if it boots.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Convert existing NATIVE_METHOD/FAST_NATIVE_METHOD macros into
also checking that the JNI signature descriptor matches the C++ function
type.

For example it matches "(J)V" with "void funcname(JNIEnv*, jclass, jlong)"
with regular native methods.

Also add several new macros:
  MAKE_JNI_[FAST_|CRITICAL_]NATIVE_METHOD - Non-convention-requiring macro.
  MAKE_JNI_[FAST_|CRITICAL_]NATIVE_METHOD_AUTOSIG - Infer signature for above.
  [FAST_|CRITICAL_]NATIVE_METHOD_AUTOSIG - Convention-requiring signature inferencing.

All of the checking/inferencing is done purely at compile time and has
zero runtime cost. If any of the checks fail there will be a compilation
error with a stack trace pointing to the reason.

See jni_macros.h for exact details on check/inference rules.

Bug: 35325126
Change-Id: I98797727ca80caf7bc462582f582b23701a1b661
Test: make -j32 JniSafeRegisterNativeMethods_test \
      &amp;&amp; out/host/linux-x86/nativetest64/JniSafeRegisterNativeMethods_test/JniSafeRegisterNativeMethods_test
Test: make -j32  # build all of android, then see if it boots.
</pre>
</div>
</content>
</entry>
<entry>
<title>Do not reinitialize system properties to fake their values</title>
<updated>2017-12-15T01:06:22+00:00</updated>
<author>
<name>Tom Cherry</name>
<email>tomcherry@google.com</email>
</author>
<published>2017-12-14T08:00:15+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/mirrors/AOSP/platform_libnativehelper/commit/?id=3c46171d3fd507906834858b8b6fdf9802109051'/>
<id>3c46171d3fd507906834858b8b6fdf9802109051</id>
<content type='text'>
Reinitializing system properties can result in crashes later in the
program, and is generally not recommended or even supported.  Instead,
this change uses a very rudamentary approach to mocking out the system
property functions.

Bug: 62197783
Test: unit tests
Change-Id: I99c57e1d87490926e4839ee154c862ee8a199e26
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Reinitializing system properties can result in crashes later in the
program, and is generally not recommended or even supported.  Instead,
this change uses a very rudamentary approach to mocking out the system
property functions.

Bug: 62197783
Test: unit tests
Change-Id: I99c57e1d87490926e4839ee154c862ee8a199e26
</pre>
</div>
</content>
</entry>
<entry>
<title>Revert "Rewrite scoped arrays with template and introduced nullable one."</title>
<updated>2017-12-09T12:48:21+00:00</updated>
<author>
<name>Nicolas Geoffray</name>
<email>ngeoffray@google.com</email>
</author>
<published>2017-12-09T12:48:21+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/mirrors/AOSP/platform_libnativehelper/commit/?id=396b56feb637973bca1fea08347411aef952f579'/>
<id>396b56feb637973bca1fea08347411aef952f579</id>
<content type='text'>
Tests are still broken.


This reverts commit 8d0e6a25a34ed2e4b89630cdab4eb5e0f4298f31.

Change-Id: If046a1b0bc47fa73750a5296118ab879678f41b8
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Tests are still broken.


This reverts commit 8d0e6a25a34ed2e4b89630cdab4eb5e0f4298f31.

Change-Id: If046a1b0bc47fa73750a5296118ab879678f41b8
</pre>
</div>
</content>
</entry>
<entry>
<title>Rewrite scoped arrays with template and introduced nullable one.</title>
<updated>2017-12-08T01:52:35+00:00</updated>
<author>
<name>Seigo Nonaka</name>
<email>nona@google.com</email>
</author>
<published>2017-11-29T22:58:56+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/mirrors/AOSP/platform_libnativehelper/commit/?id=8d0e6a25a34ed2e4b89630cdab4eb5e0f4298f31'/>
<id>8d0e6a25a34ed2e4b89630cdab4eb5e0f4298f31</id>
<content type='text'>
Single arugment constructor and reset method is not well supportd.
Drop them from Nullable and RW scoped array.

Test: m checkbuild &amp;&amp; emulator
Test: m JniInvocation_test &amp;&amp;
      adb push ${OUT}/testcases/JniInvocation_test/x86/JniInvocation_test \
      /data/local/tmp/JniInvocation_test &amp;&amp;
      adb shell /data/local/tmp/JniInvocation_test
Bug: N/A

Change-Id: I3c7c07fb840e8130be3c015d9d5dd82167f5911f
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Single arugment constructor and reset method is not well supportd.
Drop them from Nullable and RW scoped array.

Test: m checkbuild &amp;&amp; emulator
Test: m JniInvocation_test &amp;&amp;
      adb push ${OUT}/testcases/JniInvocation_test/x86/JniInvocation_test \
      /data/local/tmp/JniInvocation_test &amp;&amp;
      adb shell /data/local/tmp/JniInvocation_test
Bug: N/A

Change-Id: I3c7c07fb840e8130be3c015d9d5dd82167f5911f
</pre>
</div>
</content>
</entry>
<entry>
<title>Use -Werror in libnativehelper/tests</title>
<updated>2017-10-03T17:39:23+00:00</updated>
<author>
<name>Chih-Hung Hsieh</name>
<email>chh@google.com</email>
</author>
<published>2017-10-03T17:39:11+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/mirrors/AOSP/platform_libnativehelper/commit/?id=bc2a237dd4ee9ebef022ef9f9c130314e9f6dce4'/>
<id>bc2a237dd4ee9ebef022ef9f9c130314e9f6dce4</id>
<content type='text'>
Bug: 66996870
Test: build with WITH_TIDY=1
Change-Id: Ica0cea75412945868c995ed94c194e3bebf2bd40
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Bug: 66996870
Test: build with WITH_TIDY=1
Change-Id: Ica0cea75412945868c995ed94c194e3bebf2bd40
</pre>
</div>
</content>
</entry>
<entry>
<title>Remove clang: true</title>
<updated>2017-07-25T19:52:29+00:00</updated>
<author>
<name>Lennart Wieboldt</name>
<email>lennart.1997@gmx.de</email>
</author>
<published>2017-07-25T19:52:29+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/mirrors/AOSP/platform_libnativehelper/commit/?id=4cad9577aeed389fa480968c6a149095547811b9'/>
<id>4cad9577aeed389fa480968c6a149095547811b9</id>
<content type='text'>
clang is the default compiler since Android nougat

Change-Id: I43e6298e645337f02fc2dcc9f4b3b3f33188038b
Signed-off-by: Lennart Wieboldt &lt;lennart.1997@gmx.de&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
clang is the default compiler since Android nougat

Change-Id: I43e6298e645337f02fc2dcc9f4b3b3f33188038b
Signed-off-by: Lennart Wieboldt &lt;lennart.1997@gmx.de&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Remove some global headers.</title>
<updated>2017-07-20T21:25:43+00:00</updated>
<author>
<name>Steven Moreland</name>
<email>smoreland@google.com</email>
</author>
<published>2017-07-19T17:26:05+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/mirrors/AOSP/platform_libnativehelper/commit/?id=3544a938142edc0439d120dabd45234c62ace6d7'/>
<id>3544a938142edc0439d120dabd45234c62ace6d7</id>
<content type='text'>
Removing:
- JniInvocation.h
- ScopedLocalFrame.h

Now that all usage is cleaned up.

libnativehelper now exports "include" as well so that
its headers can be cleaned up in the same way.

Test: none
Bug: 63762847
Change-Id: Iee8f59542dc1c80e2e262d2faf4eea829426a0d8
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Removing:
- JniInvocation.h
- ScopedLocalFrame.h

Now that all usage is cleaned up.

libnativehelper now exports "include" as well so that
its headers can be cleaned up in the same way.

Test: none
Bug: 63762847
Change-Id: Iee8f59542dc1c80e2e262d2faf4eea829426a0d8
</pre>
</div>
</content>
</entry>
<entry>
<title>Add test config to JniInvocation_test</title>
<updated>2017-03-30T06:49:45+00:00</updated>
<author>
<name>Dan Shi</name>
<email>dshi@google.com</email>
</author>
<published>2017-03-30T06:49:45+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/mirrors/AOSP/platform_libnativehelper/commit/?id=8ffbd0d15024dca8631badef53ebd4fc49b8cdc4'/>
<id>8ffbd0d15024dca8631badef53ebd4fc49b8cdc4</id>
<content type='text'>
Design doc:
Generalized Suites &amp; the Unification of APCT &amp; CTS Workflows Design/Roadmap
https://docs.google.com/document/d/1eabK3srlBLouMiBMrNP3xJPiRRdcoCquNxC8gBWPvx8/edit#heading=h.78vup5eivwzo

Details about test configs changes are tracked in doc
https://docs.google.com/document/d/1EWUjJ7fjy8ge_Nk0YQbFdRp8DSHo3z6GU0R8jLgrAcw/edit#

Bug: 35882476
Test: local test
Change-Id: I960ae8d6c1d2b2a489e8d11b534be5127ea8d018
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Design doc:
Generalized Suites &amp; the Unification of APCT &amp; CTS Workflows Design/Roadmap
https://docs.google.com/document/d/1eabK3srlBLouMiBMrNP3xJPiRRdcoCquNxC8gBWPvx8/edit#heading=h.78vup5eivwzo

Details about test configs changes are tracked in doc
https://docs.google.com/document/d/1EWUjJ7fjy8ge_Nk0YQbFdRp8DSHo3z6GU0R8jLgrAcw/edit#

Bug: 35882476
Test: local test
Change-Id: I960ae8d6c1d2b2a489e8d11b534be5127ea8d018
</pre>
</div>
</content>
</entry>
<entry>
<title>Stop using __system_property_area__</title>
<updated>2017-02-28T20:22:14+00:00</updated>
<author>
<name>Victor Khimenko</name>
<email>khim@google.com</email>
</author>
<published>2017-02-28T20:15:42+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/mirrors/AOSP/platform_libnativehelper/commit/?id=5038189fffb867bf977221ffc871dd9db9d2f4f4'/>
<id>5038189fffb867bf977221ffc871dd9db9d2f4f4</id>
<content type='text'>
Apparently that "backdoor" is no longer needed - the proper way is
to reinitialize properties:
  https://android-review.googlesource.com/#/c/181794/24/tests/system_properties_test.cpp

Test: refactoring CL, existsing tests still pass

BUG=21852512

Change-Id: I2fe193d65a0e6804bfc15726d9b973a97b378fef
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Apparently that "backdoor" is no longer needed - the proper way is
to reinitialize properties:
  https://android-review.googlesource.com/#/c/181794/24/tests/system_properties_test.cpp

Test: refactoring CL, existsing tests still pass

BUG=21852512

Change-Id: I2fe193d65a0e6804bfc15726d9b973a97b378fef
</pre>
</div>
</content>
</entry>
<entry>
<title>libnativehelper: convert Android.mk to Android.bp</title>
<updated>2016-07-27T21:19:32+00:00</updated>
<author>
<name>Colin Cross</name>
<email>ccross@android.com</email>
</author>
<published>2016-07-27T17:35:53+00:00</published>
<link rel='alternate' type='text/html' href='https://git.replicant.us/mirrors/AOSP/platform_libnativehelper/commit/?id=cf47d9b7052034b41093f2052d75e68992df487f'/>
<id>cf47d9b7052034b41093f2052d75e68992df487f</id>
<content type='text'>
Change-Id: I6f9fd46b02fd10d8e5e4111623248a7bf2572633
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Change-Id: I6f9fd46b02fd10d8e5e4111623248a7bf2572633
</pre>
</div>
</content>
</entry>
</feed>
