remove unnecessary builtin name (keep-starknet-strange#384)

src/hint_processor/math_hints.zig

@@ -94,9 +94,9 @@ pub fn verifyEcdsaSignature(
 
     const ecdsa_ptr = try hint_utils.getPtrFromVarName("ecdsa_ptr", vm, ids_data, ap_tracking);
 
-    const builtin_runner = try vm.getBuiltinRunner(.Signature);
+    const builtin_runner = try vm.getSignatureBuiltin();
 
-    try builtin_runner.Signature.addSignature(ecdsa_ptr, .{
+    try builtin_runner.addSignature(ecdsa_ptr, .{
         r,
         s,
     });

src/vm/builtins/builtin_runner/builtin_runner.zig

@@ -47,29 +47,8 @@ pub const POSEIDON_BUILTIN_NAME = "poseidon_builtin";
 /// The name of the segment arena builtin.
 pub const SEGMENT_ARENA_BUILTIN_NAME = "segment_arena_builtin";
 
-pub const BuiltinName = enum {
-    /// Bitwise built-in runner for bitwise operations.
-    Bitwise,
-    /// EC Operation built-in runner for elliptic curve operations.
-    EcOp,
-    /// Hash built-in runner for hash operations.
-    Hash,
-    /// Output built-in runner for output operations.
-    Output,
-    /// Range Check built-in runner for range check operations.
-    RangeCheck,
-    /// Keccak built-in runner for Keccak operations.
-    Keccak,
-    /// Signature built-in runner for signature operations.
-    Signature,
-    /// Poseidon built-in runner for Poseidon operations.
-    Poseidon,
-    /// Segment Arena built-in runner for segment arena operations.
-    SegmentArena,
-};
-
 /// Built-in runner
-pub const BuiltinRunner = union(BuiltinName) {
+pub const BuiltinRunner = union(enum) {
     const Self = @This();
 
     /// Bitwise built-in runner for bitwise operations.

src/vm/core.zig

@@ -196,16 +196,14 @@ pub const CairoVM = struct {
         return &self.builtin_runners;
     }
 
-    /// Gets builtin runner by name
-    /// if not exist return not found builtin error
-    pub fn getBuiltinRunner(self: *Self, name: BuiltinName) !*BuiltinRunner {
-        for (self.builtin_runners.items) |*runner| {
-            if (@intFromEnum(runner.*) == @intFromEnum(name)) {
-                return runner;
-            }
-        }
+    pub fn getSignatureBuiltin(self: *const Self) !*builtin_runner.SignatureBuiltinRunner {
+        for (self.builtin_runners.items) |*runner|
+            switch (runner.*) {
+                .Signature => |*signature_builtin| return signature_builtin,
+                else => {},
+            };
 
-        return CairoVMError.NotFoundBuiltin;
+        return CairoVMError.NoSignatureBuiltin;
     }
 
     pub fn insertInMemory(

src/vm/error.zig

@@ -63,6 +63,8 @@ pub const CairoVMError = error{
     Unexpected,
     /// Expected range_check builtin to be present
     NoRangeCheckBuiltin,
+    /// Expected signature builtin to be present
+    NoSignatureBuiltin,
     /// getBuiltin by name, if not exist error
     NotFoundBuiltin,
     ReferenceNotFound,

src/vm/types/layout.zig

@@ -382,10 +382,7 @@ test "CairoLayout: allCairoInstance" {
     try expectEqual(
         @as(
             ?PedersenInstanceDef,
-            .{
-                .ratio = 256,
-				.repetitions = 1
-            },
+            .{ .ratio = 256, .repetitions = 1 },
         ),
         actual.builtins.pedersen,
     );