[GitHub] nifi-minifi-cpp pull request #436: MINIFICPP-667: Add structural definitions...

2018-11-15 Thread phrocker
Github user phrocker commented on a diff in the pull request: https://github.com/apache/nifi-minifi-cpp/pull/436#discussion_r234011794 --- Diff: nanofi/include/core/utlist.h --- @@ -0,0 +1,1073 @@ +/* --- End diff -- As per the title this is to work out a plan so th

[GitHub] nifi-minifi-cpp pull request #436: MINIFICPP-667: Add structural definitions...

2018-11-15 Thread arpadboda
Github user arpadboda commented on a diff in the pull request: https://github.com/apache/nifi-minifi-cpp/pull/436#discussion_r233919866 --- Diff: nanofi/include/core/utlist.h --- @@ -0,0 +1,1073 @@ +/* --- End diff -- Okay, fine for me. 👍 Happy to approve as it g

[GitHub] nifi-minifi-cpp pull request #436: MINIFICPP-667: Add structural definitions...

2018-11-15 Thread phrocker
Github user phrocker commented on a diff in the pull request: https://github.com/apache/nifi-minifi-cpp/pull/436#discussion_r233919353 --- Diff: nanofi/include/core/utlist.h --- @@ -0,0 +1,1073 @@ +/* --- End diff -- sorry CMAKE_SOURCE_DIR. It can go there and shoul

[GitHub] nifi-minifi-cpp pull request #436: MINIFICPP-667: Add structural definitions...

2018-11-15 Thread arpadboda
Github user arpadboda commented on a diff in the pull request: https://github.com/apache/nifi-minifi-cpp/pull/436#discussion_r233913935 --- Diff: nanofi/include/core/utlist.h --- @@ -0,0 +1,1073 @@ +/* --- End diff -- Not yet in nanofi :) Guess this library is

[GitHub] nifi-minifi-cpp pull request #436: MINIFICPP-667: Add structural definitions...

2018-11-15 Thread phrocker
Github user phrocker commented on a diff in the pull request: https://github.com/apache/nifi-minifi-cpp/pull/436#discussion_r233913139 --- Diff: nanofi/include/core/utlist.h --- @@ -0,0 +1,1073 @@ +/* --- End diff -- We have a third party dir in the main directory,

[GitHub] nifi-minifi-cpp pull request #436: MINIFICPP-667: Add structural definitions...

2018-11-15 Thread phrocker
Github user phrocker commented on a diff in the pull request: https://github.com/apache/nifi-minifi-cpp/pull/436#discussion_r233913016 --- Diff: nanofi/include/core/utlist.h --- @@ -0,0 +1,1073 @@ +/* --- End diff -- Ah thanks for calling this out. I have a more rec

[GitHub] nifi-minifi-cpp pull request #436: MINIFICPP-667: Add structural definitions...

2018-11-15 Thread phrocker
Github user phrocker commented on a diff in the pull request: https://github.com/apache/nifi-minifi-cpp/pull/436#discussion_r233911949 --- Diff: nanofi/include/core/cstructs.h --- @@ -76,6 +80,12 @@ typedef struct { * ###

[GitHub] nifi-minifi-cpp pull request #436: MINIFICPP-667: Add structural definitions...

2018-11-15 Thread phrocker
Github user phrocker commented on a diff in the pull request: https://github.com/apache/nifi-minifi-cpp/pull/436#discussion_r233911441 --- Diff: nanofi/src/api/nanofi.cpp --- @@ -72,19 +41,17 @@ class DirectoryConfiguration { * nifi_instance *create_instance(nifi_port const *p

[GitHub] nifi-minifi-cpp pull request #436: MINIFICPP-667: Add structural definitions...

2018-11-15 Thread arpadboda
Github user arpadboda commented on a diff in the pull request: https://github.com/apache/nifi-minifi-cpp/pull/436#discussion_r233910986 --- Diff: nanofi/src/api/nanofi.cpp --- @@ -72,19 +41,17 @@ class DirectoryConfiguration { * nifi_instance *create_instance(nifi_port const *

[GitHub] nifi-minifi-cpp pull request #436: MINIFICPP-667: Add structural definitions...

2018-11-15 Thread phrocker
Github user phrocker commented on a diff in the pull request: https://github.com/apache/nifi-minifi-cpp/pull/436#discussion_r233910857 --- Diff: nanofi/include/core/cstructs.h --- @@ -76,6 +80,12 @@ typedef struct { * ###

[GitHub] nifi-minifi-cpp pull request #436: MINIFICPP-667: Add structural definitions...

2018-11-15 Thread arpadboda
Github user arpadboda commented on a diff in the pull request: https://github.com/apache/nifi-minifi-cpp/pull/436#discussion_r233910142 --- Diff: nanofi/include/core/cstructs.h --- @@ -76,6 +80,12 @@ typedef struct { * ##

[GitHub] nifi-minifi-cpp pull request #436: MINIFICPP-667: Add structural definitions...

2018-11-15 Thread arpadboda
Github user arpadboda commented on a diff in the pull request: https://github.com/apache/nifi-minifi-cpp/pull/436#discussion_r233909297 --- Diff: nanofi/include/core/utlist.h --- @@ -0,0 +1,1073 @@ +/* --- End diff -- I would prefer to place this file somewhere else

[GitHub] nifi-minifi-cpp pull request #436: MINIFICPP-667: Add structural definitions...

2018-11-12 Thread phrocker
GitHub user phrocker opened a pull request: https://github.com/apache/nifi-minifi-cpp/pull/436 MINIFICPP-667: Add structural definitions to work out plan for C migr… …ation Thank you for submitting a contribution to Apache NiFi - MiNiFi C++. In order to streamli