Skip to content
This repository was archived by the owner on Jun 11, 2025. It is now read-only.

Rudy/feat/handle bytes32 #465

Merged
merged 5 commits into from
Apr 8, 2025
Merged

Rudy/feat/handle bytes32 #465

merged 5 commits into from
Apr 8, 2025

Conversation

rudy-6-4
Copy link
Contributor

@rudy-6-4 rudy-6-4 commented Apr 7, 2025

No description provided.

@cla-bot cla-bot bot added the cla-signed label Apr 7, 2025
@rudy-6-4 rudy-6-4 changed the base branch from main to new-types April 7, 2025 11:38
@rudy-6-4 rudy-6-4 marked this pull request as ready for review April 7, 2025 11:40
@rudy-6-4 rudy-6-4 requested a review from a team as a code owner April 7, 2025 11:41
@rudy-6-4 rudy-6-4 force-pushed the rudy/feat/handle-bytes32 branch from dfaa143 to b20564d Compare April 7, 2025 11:42
@rudy-6-4 rudy-6-4 requested a review from dartdart26 April 7, 2025 11:45
@@ -1,168 +1,255 @@
// SPDX-License-Identifier: BSD-3-Clause-Clear
pragma solidity ^0.8.24;

enum FheType {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See if it could be imported by tweaking build.rs

event FheMax(address indexed caller, uint256 lhs, uint256 rhs, bytes1 scalarByte, uint256 result);
event FheNeg(address indexed caller, uint256 ct, uint256 result);
event FheNot(address indexed caller, uint256 ct, uint256 result);
event FheAdd(address indexed caller, bytes32 lhs, bytes32 rhs, bytes1 scalarByte, bytes32 result);
Copy link
Contributor Author

@rudy-6-4 rudy-6-4 Apr 7, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same. Isolate events in real contract and import

@rudy-6-4 rudy-6-4 force-pushed the rudy/feat/handle-bytes32 branch from b20564d to 91ac1bb Compare April 7, 2025 12:45
@rudy-6-4 rudy-6-4 force-pushed the rudy/feat/handle-bytes32 branch from 91ac1bb to 89998fa Compare April 7, 2025 13:18
@rudy-6-4
Copy link
Contributor Author

rudy-6-4 commented Apr 7, 2025

I have removed "disabling build.rs in release mode" since the test are run in release mode.

@dartdart26
Copy link
Collaborator

@rudy-6-4 rudy-6-4 force-pushed the rudy/feat/handle-bytes32 branch from 622db5d to 35f4fd4 Compare April 8, 2025 13:44
@rudy-6-4 rudy-6-4 merged commit 6420178 into new-types Apr 8, 2025
12 checks passed
@rudy-6-4 rudy-6-4 deleted the rudy/feat/handle-bytes32 branch April 8, 2025 14:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants