Skip to content

Commit 800da1e

Browse files
authored
Merge pull request #7 from dimacurrentai/overflow
Integer overflow tests in release mode.
2 parents 28dc7d1 + b51b0c9 commit 800da1e

File tree

3 files changed

+9
-4
lines changed

3 files changed

+9
-4
lines changed

step01_cmdline/code/Cargo.toml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,5 +2,8 @@
22
name = "add"
33
edition = "2021"
44

5+
[profile.release]
6+
overflow-checks = true
7+
58
[dependencies]
69
clap = { version = "4.0", features = ["derive"] }

step01_cmdline/code/src/main.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,9 +3,9 @@ use clap::Parser;
33
#[derive(Parser)]
44
struct Args {
55
#[arg(long)]
6-
a: u32,
6+
a: i8,
77
#[arg(long)]
8-
b: u32,
8+
b: i8,
99
}
1010

1111
fn main() {

step01_cmdline/run.sh

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,5 +3,7 @@
33
set -e
44

55
docker build -f ../Dockerfile.template . -t demo
6-
docker run --rm -t demo --a 1 --b 2
7-
docker run --rm -t demo --a 3 --b 4
6+
docker run --rm -t demo --a 1 --b 2 && echo OK
7+
docker run --rm -t demo --a 3 --b 4 && echo OK
8+
docker run --rm -t demo --a 100 --b 100 && (echo "Error, should overflow."; exit 1)
9+
echo "Oveflow handled correctly."

0 commit comments

Comments
 (0)