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

    https://github.com/apache/nifi-minifi-cpp/pull/420#discussion_r226382717
  
    --- Diff: libminifi/src/capi/api.cpp ---
    @@ -57,38 +57,28 @@ class DirectoryConfiguration {
       }
     };
     
    -nifi_port *create_port(const char *port) {
    -  if (nullptr == port)
    -    return nullptr;
    -  nifi_port *p = new nifi_port();
    -  p->port_id = new char[strlen(port) + 1];
    -  memset(p->port_id, 0x00, strlen(port) + 1);
    -  strncpy(p->port_id, port, strlen(port));
    -  return p;
    -}
    -
    -int free_port(nifi_port *port) {
    -  if (port == nullptr)
    -    return -1;
    -  delete[] port->port_id;
    -  delete port;
    -  return 0;
    -}
    -
     /**
      * Creates a NiFi Instance from the url and output port.
      * @param url http URL for NiFi instance
      * @param port Remote output port.
    + * @Deprecated for 0.6.0 in favor of
    --- End diff --
    
    Sorry to nitpick, but shall we refer to minifi releases or API version, 
could see this getting a little confusing at some point?  How could we best 
document these version relationships?


---

Reply via email to