Github user arpadboda commented on a diff in the pull request:

    https://github.com/apache/nifi-minifi-cpp/pull/429#discussion_r229108728
  
    --- Diff: libminifi/src/capi/api.cpp ---
    @@ -400,11 +395,19 @@ processor *add_processor_with_linkage(flow *flow, 
const char *processor_name) {
       return nullptr;
     }
     
    -int add_failure_callback(flow *flow, void (*onerror_callback)(const 
flow_file_record*)) {
    +int add_failure_callback(flow *flow, void 
(*onerror_callback)(flow_file_record*)) {
       ExecutionPlan *plan = static_cast<ExecutionPlan*>(flow->plan);
       return plan->setFailureCallback(onerror_callback) ? 0 : 1;
     }
     
    +int set_failure_strategy(flow *flow, int strategy) {
    --- End diff --
    
    The reason is pretty simple: I didn't have enough coffee when I wrote this, 
so forgot that C also supports enums. :) Will replace. 


---

Reply via email to