remove mask const (keep-starknet-strange#110)

* remove mask const * fix ci condition * remove extra try, change defer to errdefer --------- Co-authored-by: d-roak <[email protected]>
lana-shanghai · Nov 7, 2023 · 033ddd7 · 033ddd7
1 parent d2a2037
commit 033ddd7
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/.github/workflows/docs.yml b/.github/workflows/docs.yml
@@ -20,7 +20,7 @@ jobs:
         run: zig test -femit-docs=./docsite src/lib.zig
 
       - name: Deploy
-        if: ${{github.event_name == 'push'}} && ${{ github.ref == 'refs/heads/main' }} && steps.check_changes.outcome == 'success'
+        if: ${{ github.event_name == 'push' && github.ref == 'refs/heads/main' && steps.check_changes.outcome == 'success' }}
         uses: peaceiris/actions-gh-pages@v3
         with:
           github_token: ${{ secrets.GITHUB_TOKEN }}

diff --git a/src/vm/builtins/builtin_runner/range_check.zig b/src/vm/builtins/builtin_runner/range_check.zig
@@ -19,7 +19,6 @@ const RunnerError = Error.RunnerError;
 
 const N_PARTS: u64 = 8;
 const INNER_RC_BOUND_SHIFT: u64 = 16;
-const INNER_RC_BOUND_MASK: u64 = @as(u64, @intCast(u16.MAX));
 
 /// Range check built-in runner
 pub const RangeCheckBuiltinRunner = struct {
@@ -248,6 +247,7 @@ pub const RangeCheckBuiltinRunner = struct {
     /// verification fails.
     pub fn rangeCheckValidationRule(memory: *Memory, address: Relocatable, allocator: Allocator) !std.ArrayList(Relocatable) {
         var result = ArrayList(Relocatable).init(allocator);
+        errdefer result.deinit();
         const num = (memory.get(address) catch {
             return null;
         }).tryIntoFelt() catch {