search

JeffGarland / liaw2019-process

Repository for initial drafting of boost.process standards paper
MIT License
5 stars 3 forks source link

Mostly editorial changes #11

Closed eliaskosunen closed 5 years ago

eliaskosunen commented 5 years ago

Tons of typo fixes Added some notes with prefix EK: (my initials)

Code style changed to be similar to the standard:

template<class T, class... Ts>
void func(const T& a, Ts&&... b) {
  // foo
}
eliaskosunen commented 5 years ago

The diff is humongous, but there are no substantial changes; just typo fixes and whitespace changes, alongside some comments and notes

JeffGarland commented 5 years ago

Ok. I've looked thru the first 1/2 and looks good. Will get back to it in a bit....

eliaskosunen commented 5 years ago

I'll extract the notes/comments to a separate issue, they're just there so I wouldn't forget them

JeffGarland commented 5 years ago

that would be good - probably time to clean up the todos and notes

On Jun 5, 2019, at 5:23 PM, Elias Kosunen notifications@github.com wrote:

I'll extract the notes/comments to a separate issue, they're just there so I wouldn't forget them

— You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.

eliaskosunen commented 5 years ago

PR rebased against current master

Once this gets merged, I'll list the notes and comments to an issue

JeffGarland commented 5 years ago

Thanks for rebasing this!