<feed xmlns='http://www.w3.org/2005/Atom'>
<title>viewer.git/indra/llcommon/tests/llleap_test.cpp, branch cef_147</title>
<subtitle>Megapahit's fork of the Second Life viewer.
</subtitle>
<id>https://megapahit.org/viewer.git/atom?h=cef_147</id>
<link rel='self' href='https://megapahit.org/viewer.git/atom?h=cef_147'/>
<link rel='alternate' type='text/html' href='https://megapahit.org/viewer.git/'/>
<updated>2024-05-16T17:30:14Z</updated>
<entry>
<title>On Windows, defend test.cpp against structured exceptions too.</title>
<updated>2024-05-16T17:30:14Z</updated>
<author>
<name>Nat Goodspeed</name>
<email>nat@lindenlab.com</email>
</author>
<published>2024-05-16T17:30:14Z</published>
<link rel='alternate' type='text/html' href='https://megapahit.org/viewer.git/commit/?id=1b6e2ef62cec9608d160ea25d99080f0e2964ee5'/>
<id>urn:sha1:1b6e2ef62cec9608d160ea25d99080f0e2964ee5</id>
<content type='text'>
Since August 2023, we've seen occasional GitHub Windows build test runs
terminate with 0xC00000FD: stack overflow. We've usually responded by bumping
up the default coroutine stack size.

On closer examination, it's always llleap_test.cpp that blows up that way --
and llleap_test.cpp doesn't appear to use coroutines at all. So apparently
we've been consuming more address space for ALL viewer coroutines without
actually addressing the problem.

Reset the default coroutine stack size to where it was before we started
bumping it up in response to these llleap_test.cpp stack overflow failures.
Note that LLCoros already catches and reports Windows structured exceptions,
underscoring that the observed stack overflow is not from within a coroutine.

While at it, restore the Windows llleap_test.cpp data volume to match Posix.
We think the problem that led to reducing that data volume was an APR bug,
which we hope has been fixed.

Equip test.cpp, the test driver program for all our TUT unit and integration
tests, with a Windows structured exception handler. Try to treat a Windows
structured exception as a test failure -- instead of silently terminating with
0xC00000FD. Moreover, when a structured exception occurs, output a stack trace
so we can try to track it down.
</content>
</entry>
<entry>
<title>#824 Process source files in bulk: replace tabs with spaces, convert CRLF to LF, and trim trailing whitespaces as needed</title>
<updated>2024-04-29T04:56:09Z</updated>
<author>
<name>Andrey Lihatskiy</name>
<email>alihatskiy@productengine.com</email>
</author>
<published>2024-04-29T04:43:28Z</published>
<link rel='alternate' type='text/html' href='https://megapahit.org/viewer.git/commit/?id=1b68f71348ecf3983b76b40d7940da8377f049b7'/>
<id>urn:sha1:1b68f71348ecf3983b76b40d7940da8377f049b7</id>
<content type='text'>
</content>
</entry>
<entry>
<title>SL-18837: Merge branch 'main' into actions</title>
<updated>2023-09-07T15:15:56Z</updated>
<author>
<name>Nat Goodspeed</name>
<email>nat@lindenlab.com</email>
</author>
<published>2023-09-07T15:15:56Z</published>
<link rel='alternate' type='text/html' href='https://megapahit.org/viewer.git/commit/?id=b8e67774397f2878aafc5906845f5f738e00c408'/>
<id>urn:sha1:b8e67774397f2878aafc5906845f5f738e00c408</id>
<content type='text'>
</content>
</entry>
<entry>
<title>SL-18837: Revert "Force llprocess_test and llleap_test to use just 'python'."</title>
<updated>2023-07-10T18:35:41Z</updated>
<author>
<name>Nat Goodspeed</name>
<email>nat@lindenlab.com</email>
</author>
<published>2023-07-10T18:35:41Z</published>
<link rel='alternate' type='text/html' href='https://megapahit.org/viewer.git/commit/?id=31ccef8a666da54312a55663a7ac03061c4903be'/>
<id>urn:sha1:31ccef8a666da54312a55663a7ac03061c4903be</id>
<content type='text'>
Turns out that the pathname of the Python executable wasn't the issue.

This reverts commit 7dc6211ad5ea83685a35c6fff740278343aa8b9d.
</content>
</entry>
<entry>
<title>SL-18837: Force llprocess_test and llleap_test to use just 'python'.</title>
<updated>2023-07-08T18:08:16Z</updated>
<author>
<name>Nat Goodspeed</name>
<email>nat@lindenlab.com</email>
</author>
<published>2023-07-08T18:08:16Z</published>
<link rel='alternate' type='text/html' href='https://megapahit.org/viewer.git/commit/?id=7dc6211ad5ea83685a35c6fff740278343aa8b9d'/>
<id>urn:sha1:7dc6211ad5ea83685a35c6fff740278343aa8b9d</id>
<content type='text'>
On GitHub Windows runners, trying to make build.yaml set PYTHON=python in the
environment doesn't work: integration tests still fail with "Access is denied"
because they're still trying to execute the interpreter's full pathname.

Instead, make llprocess_test and llleap_test detect the case of GitHub Windows
and override the environment variable PYTHON with a baked-in string constant
"python".
</content>
</entry>
<entry>
<title>SL-18837: Partially revert e933ace, keeping useful tweaks.</title>
<updated>2023-07-07T20:57:20Z</updated>
<author>
<name>Nat Goodspeed</name>
<email>nat@lindenlab.com</email>
</author>
<published>2023-07-07T20:57:20Z</published>
<link rel='alternate' type='text/html' href='https://megapahit.org/viewer.git/commit/?id=c4b5d089dad5680a0dd12b2d386b692318eb5c58'/>
<id>urn:sha1:c4b5d089dad5680a0dd12b2d386b692318eb5c58</id>
<content type='text'>
Introducing indirection via test_python_script.py did NOT address the "Access
is denied" errors on GitHub Windows runners.
</content>
</entry>
<entry>
<title>SL-18837: Ditch Boost.Phoenix implicit lambda syntax.</title>
<updated>2023-06-06T14:04:26Z</updated>
<author>
<name>Nat Goodspeed</name>
<email>nat@lindenlab.com</email>
</author>
<published>2023-06-06T14:04:26Z</published>
<link rel='alternate' type='text/html' href='https://megapahit.org/viewer.git/commit/?id=c4366378b61cacb9d87eb2917302fa17e9207491'/>
<id>urn:sha1:c4366378b61cacb9d87eb2917302fa17e9207491</id>
<content type='text'>
It's cool to be able to write 'arg1 &lt;&lt; "stuff" &lt;&lt; var ...;' for a lambda
accepting a std::ostream reference, but cascading compile errors mean it's no
longer worth trying to make that work -- given actual C++ lambdas.

Also clean up a lingering BOOST_FOREACH() and a boost::bind() while at it.
</content>
</entry>
<entry>
<title>SL-18837: NamedTempFile back to std::function, use boost::phoenix &lt;&lt;</title>
<updated>2023-06-06T11:44:42Z</updated>
<author>
<name>Nat Goodspeed</name>
<email>nat@lindenlab.com</email>
</author>
<published>2023-06-06T11:44:42Z</published>
<link rel='alternate' type='text/html' href='https://megapahit.org/viewer.git/commit/?id=6516c9d07db42beba5ba9c0c41a33925794a249c'/>
<id>urn:sha1:6516c9d07db42beba5ba9c0c41a33925794a249c</id>
<content type='text'>
It seems the problem addressed by aab769e wasn't some synergy between
Boost.Phoenix and Boost.Function, but rather the lack of a Phoenix header file
introducing operator&lt;&lt;().
</content>
</entry>
<entry>
<title>SL-18837: Try giving temp Python scripts a .py extension.</title>
<updated>2023-06-06T02:08:26Z</updated>
<author>
<name>Nat Goodspeed</name>
<email>nat@lindenlab.com</email>
</author>
<published>2023-06-06T02:08:26Z</published>
<link rel='alternate' type='text/html' href='https://megapahit.org/viewer.git/commit/?id=ca510f6c299335c8db27c65c10a8553801c06023'/>
<id>urn:sha1:ca510f6c299335c8db27c65c10a8553801c06023</id>
<content type='text'>
On GitHub Windows Actions runners, we're getting permissions errors trying to
tell the Python interpreter to run a NamedTempFile script. Try using
NamedExtTempFile to give each such script a .py extension.
</content>
</entry>
<entry>
<title>Merge branch 'main' into marchcat/main-contrib-merge</title>
<updated>2023-04-06T21:20:59Z</updated>
<author>
<name>Andrey Lihatskiy</name>
<email>alihatskiy@productengine.com</email>
</author>
<published>2023-04-06T21:20:59Z</published>
<link rel='alternate' type='text/html' href='https://megapahit.org/viewer.git/commit/?id=f1095d78315818642e62b7f4af9984557e176b5d'/>
<id>urn:sha1:f1095d78315818642e62b7f4af9984557e176b5d</id>
<content type='text'>
# Conflicts:
#	indra/cmake/CMakeLists.txt
#	indra/llcommon/llsdserialize.cpp
#	indra/llcommon/llsdserialize.h
#	indra/llcommon/tests/llleap_test.cpp
#	indra/newview/llfilepicker.h
#	indra/newview/llfilepicker_mac.h
#	indra/newview/llfilepicker_mac.mm
#	indra/newview/skins/default/xui/en/strings.xml
</content>
</entry>
</feed>
