remove PC_PYTHNET flag

Author: cctdaniel

program/c/makefile

@@ -11,14 +11,7 @@ else
   include $(SOLANA)/sdk/sbf/c/sbf.mk
 endif
 
-# Propagate the PC_PYTHNET feature by conditionally defining it in a
-# features.h header. The makefile included from Solana SDK does not
-# have an easy way to pass extra C flags which motivates this approach.
-ifdef PC_PYTHNET
-  FEATURES_H_BODY:="\#pragma once\n\#define PC_PYTHNET 1"
-else
-  FEATURES_H_BODY:="\#pragma once"
-endif
+FEATURES_H_BODY:="\#pragma once"
 
 
 .PHONY: features.h # Putting this in .PHONY makes sure the header is always regenerated

program/c/src/oracle/native/upd_aggregate.c

@@ -9,11 +9,7 @@ char heap_start[8192];
 #include "../upd_aggregate.h"
 #include "../features.h"
 
-#ifdef PC_PYTHNET
 extern bool c_upd_aggregate_pythnet( pc_price_t *ptr, uint64_t slot, int64_t timestamp ){
-#else
-extern bool c_upd_aggregate_solana( pc_price_t *ptr, uint64_t slot, int64_t timestamp ){
-#endif
   return upd_aggregate(ptr, slot, timestamp );
 }
 

program/c/src/oracle/oracle.h

@@ -27,12 +27,9 @@ extern "C" {
 #define PC_NUM_COMP_PYTHNET     128
 
 // PC_NUM_COMP - number of price components in use
-#ifdef PC_PYTHNET
 // Not whole PC_NUM_COMP_PYTHNET because of stack issues appearing in upd_aggregate()
 #define PC_NUM_COMP 64
-#else
-#define PC_NUM_COMP PC_NUM_COMP_SOLANA
-#endif
+
 
 #define PC_PROD_ACC_SIZE    512
 #define PC_EXP_DECAY         -9
@@ -209,7 +206,6 @@ typedef struct pc_price
   pc_price_comp_t comp_[PC_NUM_COMP];// component prices
 } pc_price_t;
 
-#ifdef PC_PYTHNET
 
 // Replace Solana component size's contribution with Pythnet's
 #define PC_EXPECTED_PRICE_T_SIZE_PYTHNET (3312 \
@@ -220,9 +216,6 @@ typedef struct pc_price
 static_assert( sizeof( pc_price_t ) == PC_EXPECTED_PRICE_T_SIZE_PYTHNET, "" );
 #undef PC_EXPECTED_PRICE_SIZE_PYTHNET
 
-#else
-static_assert( sizeof( pc_price_t ) == 3312, "" );
-#endif
 
 // This constant needs to be an upper bound of the price account size, it is used within pythd for ztsd.
 // It is set tighly to the current price account + 96 component prices + 48 bytes for cumulative sums

program/c/src/oracle/upd_aggregate.c

@@ -6,12 +6,8 @@
 #include "upd_aggregate.h"
 #include "features.h"
 
-// Dynamically deciding the name prevents linking the wrong C binary flavor
-#ifdef PC_PYTHNET
+
 extern bool c_upd_aggregate_pythnet( pc_price_t *ptr, uint64_t slot, int64_t timestamp ){
-#else
-extern bool c_upd_aggregate_solana( pc_price_t *ptr, uint64_t slot, int64_t timestamp ){
-#endif
   return upd_aggregate(ptr, slot, timestamp );
 }
 

program/rust/build.rs

@@ -24,15 +24,6 @@ fn main() {
     eprintln!("OUT_DIR is {}", out_dir);
     let out_dir = PathBuf::from(out_dir);
 
-    let mut make_extra_flags = vec![];
-    let mut clang_extra_flags = vec![];
-
-    // Define PC_PYTHNET for the C binary build
-    make_extra_flags.push("PC_PYTHNET=1");
-
-    // Define PC_PYTHNET for the bindings build
-    clang_extra_flags.push("-DPC_PYTHNET=1");
-
     let mut make_targets = vec![];
     if target_arch == "bpf" {
         make_targets.push("cpyth-bpf");
@@ -48,7 +39,7 @@ fn main() {
     if has_feat_check {
         eprintln!("WARNING: `check` feature active, make build is skipped");
     } else {
-        do_make_build(make_extra_flags, make_targets, &out_dir);
+        do_make_build(make_targets, &out_dir);
 
         // Link against the right library for the architecture
         if target_arch == "bpf" {
@@ -71,7 +62,6 @@ fn main() {
     // Generate and write bindings
     let bindings = Builder::default()
         .clang_arg(format!("-I{:}", get_solana_inc_path().display()))
-        .clang_args(clang_extra_flags)
         .header("./src/bindings.h")
         .rustfmt_bindings(true)
         .generate()
@@ -85,14 +75,13 @@ fn main() {
     println!("cargo:rerun-if-changed=../");
 }
 
-fn do_make_build(extra_flags: Vec<&str>, targets: Vec<&str>, out_dir: &Path) {
+fn do_make_build(targets: Vec<&str>, out_dir: &Path) {
     // We must forward OUT_DIR as an env variable to the make script otherwise it will output
     // its artifacts to the wrong place.
     let make_output = std::process::Command::new("make")
         .env("VERBOSE", "1")
         .env("OUT_DIR", out_dir.display().to_string())
         .current_dir("../c")
-        .args(extra_flags)
         .args(targets)
         .output()
         .expect("Failed to run make for C oracle program");