Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: pnappa/subprocesscpp
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: master
Choose a base ref
...
head repository: pnappa/subprocesscpp
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: new_api
Choose a head ref
  • 18 commits
  • 15 files changed
  • 1 contributor

Commits on Nov 20, 2018

  1. Configuration menu
    Copy the full SHA
    4bed072 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    12b05ec View commit details
    Browse the repository at this point in the history

Commits on Nov 21, 2018

  1. Configuration menu
    Copy the full SHA
    52ad3c2 View commit details
    Browse the repository at this point in the history
  2. merged master into this

    pnappa committed Nov 21, 2018
    Configuration menu
    Copy the full SHA
    91c24a8 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    1d5f1cc View commit details
    Browse the repository at this point in the history

Commits on Nov 22, 2018

  1. Configuration menu
    Copy the full SHA
    52aef26 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    20047cd View commit details
    Browse the repository at this point in the history

Commits on Nov 23, 2018

  1. Configuration menu
    Copy the full SHA
    f686ec3 View commit details
    Browse the repository at this point in the history

Commits on Dec 5, 2018

  1. barebones Process added

    It can pipe to new processes, but not sure if that's working...
    pnappa committed Dec 5, 2018
    Configuration menu
    Copy the full SHA
    9235201 View commit details
    Browse the repository at this point in the history

Commits on Dec 7, 2018

  1. replaced piping to file with outputting to string - complicated lifet…

    …ime problems otherwise
    pnappa committed Dec 7, 2018
    Configuration menu
    Copy the full SHA
    7422cdd View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    d16a229 View commit details
    Browse the repository at this point in the history
  3. fixed borked output bug

    the process args were populated via temporary stack variables
    (intialiser lists), so having two processes created caused process
    arguments to overwrite each other.
    pnappa committed Dec 7, 2018
    Configuration menu
    Copy the full SHA
    2e5a030 View commit details
    Browse the repository at this point in the history

Commits on Dec 8, 2018

  1. Configuration menu
    Copy the full SHA
    339243d View commit details
    Browse the repository at this point in the history
  2. Wrote some tests for Process/minor internal mods

    One currently doesn't pass - it doesn't fail, mind you, but the test
    never terminates. A read that is happening in one of the tests is
    blocking, but it is reliant on the input of the previous process.
    
    Perhaps, I modify it that if there isn't a line immediately, it
    recursively requests a line from the parent process; this isn't neat and
    is prone to lockup either way.
    
    I also added a buffer to the stdout too, as some might build up in a
    process that is piped to, and not read frequently. Unfortunately, with
    that addition comes another problem - if the line isn't read explicitly
    (currently only via >>), the functors DO get called, but the buffer is
    never empties and is increasingly filled up.
    
    I also wrote a new testing program - it simply out puts a lot of data,
    so that we can test the pipe congestion better. I just wrote this one
    because we're gonna start writing the processes that are invoked from
    the testing program from scratch - enabling the tests to be run
    cross-platform.
    pnappa committed Dec 8, 2018
    Configuration menu
    Copy the full SHA
    5998eed View commit details
    Browse the repository at this point in the history
  3. Procesess with functors can no longer be read from

    Using operator>> or .read is prohibited for processes that have
    processes/files to output to, or if they have a functor to execute.
    
    The reason is to avoid massive memory increases for those who don't need
    the output, and only wish to use a functor.
    
    I need to fix the tests so they compile however. Tomorrow's job!
    pnappa committed Dec 8, 2018
    Configuration menu
    Copy the full SHA
    8b9068c View commit details
    Browse the repository at this point in the history

Commits on Dec 14, 2018

  1. Configuration menu
    Copy the full SHA
    9673a6c View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    abe7eea View commit details
    Browse the repository at this point in the history
  3. network topology seems to work

    pnappa committed Dec 14, 2018
    Configuration menu
    Copy the full SHA
    58e1dcc View commit details
    Browse the repository at this point in the history
Loading