[pull] master from nu-book:master #33
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
![pull[bot]](https://avatars.githubusercontent.com/in/12910?s=60&v=4){kind=small}
publish-android.yml: fix typo of ci name
This should fix the root cause of the iOS crash discussed in the #717.
…tring This adds a layer of security to prevent nil crashes in case there are more cases with invalid UTF8 error messages. This supersedes #717.
As @markusfisch was so kind to point out: actually assigning the new NSString to somewhere might be of use.
This fixes a build issue because the latest clap crate requires a more modern rustc than Ubuntu currently provides.
Apart from the prefix, the rest of the naming is strictly following the C++ symbol names using _ to replace wither :: or .. So in that sense the prefix should be `ZXing_` to replace the `ZXing` c++ namespace. `ZXing::ReaderOptions::tryHarder` becomes `ZXing_ReaderOptions_tryHarder`. For more info see also #720
The ARM_NEON=ON setting does not seem to have an effect.
Since it requires specifying all parameters and that isn't usually very useful. It's better to always use `init` and the corresponding `setter` functions.
…727) Turns out `__lzcnt()` can function as a random number generator on old Windows boxes. Nice. This fixes #727 and supersedes #728, as it takes care to prevent even more wrong results by calling it for non 32 bit values. A C++20 build would have fixed this as well. Thanks to @Abooow786 for the effort of debugging this.
Use the new option to read the image files without forcing stb_image to return rgb data also in the black box testing related ImageLoader.
* There was an extra byte allocated but not set to 0. * Don't crash if len pointer argument is NULL. * Print `bytesECI()` in testing code. * Add comment about ZXing_LastErrorMsg() return values.
ios: remove initWithFormats initializer
Replace `ReaderOptions` + `read_barcodes()` with `BarcodeReader` and `BarcodeReader.read()`. For the motivation behind this change, see #696 (comment) Also replace `barcode_formats_from_string` with more idiomatic `BarcodeFormats::from_str`.
This adds a Kotlin/Native wrapper based on the C-API.
Without this, the usage of the __FILE__ macro leaves the build directory in the binary. When building the Python extension with build isolation enabled this would lead to random paths in the binary.
cmake: Make build reproducible across different build directories
Also add this to dotnet and rust wrapper.
Apparently Microsoft dropped support of ARM32 with latest SDK. See e.g. ncroxon/gnu-efi#54.
This showed up in the kotlin/Native CI build.
Use a trick from the C# playbook: return a static empty vector that does not need to be allocated and deleted on every call.
This supersedes #878.
The maven central based releases (android and k/n) need some extra care to make the SNAPSHOT stuff keep working.
HRI: update AIs to latest gs1-syntax-dictionary.txt
I thought to use the latest release 2.13.0 instead but that lacks support for in memory file generation which is crucial to our use case.
no comment... :-/
This fixes #884.
Thanks to Antonio Rojas for pointing it out to me.
With the current version 19.42.34435.0, it seems to work again. This 'fixes' #735.
Change `thread_local` to `ZX_THREAD_LOCAL` so it can be disabled centrally (as it used to).
This is considered a fix for #888.
Add `cmake` PyPI dependency only if the program is not found, and use the system version instead. This avoids unnecessary dependencies on third-party binary packages, and improves portability by using downstream-patched CMake version.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by
pull[bot]
Can you help keep this open source service alive? 💖 Please sponsor : )