fic a workflow

Author: zfogg

.github/workflows/benchmark.yml

@@ -29,8 +29,7 @@ jobs:
     - name: Build optimized version
       run: |
         # Build library and test binary with release optimization
-        make release
-        make tests
+        make tests-release
 
     - name: Run performance tests
       run: |
@@ -39,11 +38,11 @@ jobs:
 
         # Time the test execution
         echo "## Test Suite Performance"
-        /usr/bin/time -v ./bin/bin_tests 2>&1 | tee benchmark_results.txt
+        /usr/bin/time -v ./bin/bin_tests_release 2>&1 | tee benchmark_results.txt
 
         echo "## Memory Usage Analysis"
         echo "Binary sizes:"
-        ls -lh build/libbin.so bin/bin_tests
+        ls -lh build/libbin_release.so bin/bin_tests_release
 
         echo "## Library Metrics"
         echo "Source lines of code:"

.github/workflows/ci.yml

@@ -133,8 +133,12 @@ jobs:
         echo "Checking for bitwise operators (forbidden by educational constraints):"
 
         # Check for actual bitwise operators (the ones we want to avoid)
-        # Use sed to strip all comments before checking
-        BITWISE_VIOLATIONS=$(find src/ -name "*.c" -o -name "*.h" | xargs sed -e 's|//.*||g' -e 's|/\*.*\*/||g' | grep -n -E '(\&[^&]|\|[^|]|\^|<<|>>|~)' | grep -v '&&' | grep -v '||' || true)
+        # Use a more robust approach to strip comments
+        BITWISE_VIOLATIONS=$(find src/ -name "*.c" -o -name "*.h" | while read file; do
+          # Remove all types of C comments, then check for violations
+          # This handles both // and /* */ comments properly
+          perl -pe 's|//.*||g; s|/\*.*?\*/||gs' "$file" | grep -n -E '(\&[^&]|\|[^|]|\^|<<|>>|~)' | grep -v '&&' | grep -v '||' | sed "s|^|$file:|" || true
+        done)
         
         BITWISE_COUNT=$(echo "$BITWISE_VIOLATIONS" | grep -c . 2>/dev/null || true)
         if [ -z "$BITWISE_COUNT" ] || [ "$BITWISE_COUNT" = "" ]; then

.github/workflows/security.yml

@@ -118,7 +118,11 @@ jobs:
 
         # Check for bitwise operators (should be forbidden by project rules)
         echo "### Bitwise operators:"
-        BITWISE_VIOLATIONS=$(find src/ -name "*.c" -o -name "*.h" | xargs sed -e 's|//.*||g' -e 's|/\*.*\*/||g' | grep -n -E '(\&[^&]|\|[^|]|\^|<<|>>|~)' | grep -v '&&' | grep -v '||' || true)
+        BITWISE_VIOLATIONS=$(find src/ -name "*.c" -o -name "*.h" | while read file; do
+          # Remove all types of C comments, then check for violations
+          # This handles both // and /* */ comments properly
+          perl -pe 's|//.*||g; s|/\*.*?\*/||gs' "$file" | grep -n -E '(\&[^&]|\|[^|]|\^|<<|>>|~)' | grep -v '&&' | grep -v '||' | sed "s|^|$file:|" || true
+        done)
         
         BITWISE_COUNT=$(echo "$BITWISE_VIOLATIONS" | grep -c . 2>/dev/null || true)
         if [ -z "$BITWISE_COUNT" ] || [ "$BITWISE_COUNT" = "" ]; then