Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Feature Request:(?) overload writer::send_chunk(void) #123

Closed
Anticom opened this issue Mar 22, 2017 · 1 comment
Closed

Feature Request:(?) overload writer::send_chunk(void) #123

Anticom opened this issue Mar 22, 2017 · 1 comment

Comments

@Anticom
Copy link

Anticom commented Mar 22, 2017

Is there any reason why pion::http::writer::send_chunk(void) is not overloaded?
I'm trying to figure out, what the SendHandler i have to provide to send_chunk is actually supposed to do.

Background info: I'm trying to get Server Sent Events to work with pion.
@Anticom
Copy link
Author

Anticom commented Mar 22, 2017

This FR is kind of obsolete, I've found a solution.
However for the sake of consistent APIs my question is still valid.

firedaemon-cto pushed a commit to FireDaemon/pion that referenced this issue Aug 27, 2020
@JeffGarland
initial set of core changes for constexpr support (issue splunk#123) (s…
4e1b7cd 
…plunk#126)

* initial set of core changes for constexpr support (issue splunk#123)

* fixes for constexpr core support based on ci failures (issue splunk#123)

* drive by fix for gcc9.2 warning on polymorphic exception

* fixes for constexpr core support based on for cpp11 and cpp14 build variants (issue splunk#123)

* more fixes for constexpr core support  (issue splunk#123) - also fix for issue splunk#124

* next round of changes for constexpr core support (issue splunk#123)

* fix last msvc regression for constexpr core

* driveby fix to remove detail include

* 3rd round of changes for constexpr

* yet another msvc fix for cpp14

* minor doc fix for issue splunk#127 - tm_isddst -> tm_isdst

* minor doc fix for issue splunk#127 - tm_isddst -> tm_isdst

* push constexpr deep into the library including date, ptime (issue splunk#123)

* push constexpr deep into the library including date, ptime (issue splunk#123)

* fix latests regressions from constexpr changes in older gcc
firedaemon-cto pushed a commit to FireDaemon/pion that referenced this issue Aug 27, 2020
@JeffGarland
pull request to make date-time all inline (#135)
d8c7a59 
* remove legacy to_simple_string and prefer streaming in test code

* driveby fix to clean up self-assign compiler warnings in test code

* driveby fix to clean up unused variable compiler warnings in test code

* another update for issue splunk#123 (constexpr) to allow time_duration constexpr to work

* final updates to make date_time free of library - issue #134
firedaemon-cto pushed a commit to FireDaemon/pion that referenced this issue Aug 27, 2020
@JeffGarland
add stub library for backward compatibility (#137)
bbac217 
* remove legacy to_simple_string and prefer streaming in test code

* driveby fix to clean up self-assign compiler warnings in test code

* driveby fix to clean up unused variable compiler warnings in test code

* another update for issue splunk#123 (constexpr) to allow time_duration constexpr to work

* final updates to make date_time free of library - issue #134

* make a stub library for backward compatibility of libs that link date_time library
firedaemon-cto pushed a commit to FireDaemon/pion that referenced this issue Aug 27, 2020
@JeffGarland @joaquintides
make date-time all inline, additional constexpr fixes, clang 2a fix f…
cb3bd50 
…or boost 1.73 (#142)

* pull request to make date-time all inline (#135)

* remove legacy to_simple_string and prefer streaming in test code

* driveby fix to clean up self-assign compiler warnings in test code

* driveby fix to clean up unused variable compiler warnings in test code

* another update for issue splunk#123 (constexpr) to allow time_duration constexpr to work

* final updates to make date_time free of library - issue #134

* add stub library for backward compatibility (#137)

* remove legacy to_simple_string and prefer streaming in test code

* driveby fix to clean up self-assign compiler warnings in test code

* driveby fix to clean up unused variable compiler warnings in test code

* another update for issue splunk#123 (constexpr) to allow time_duration constexpr to work

* final updates to make date_time free of library - issue #134

* make a stub library for backward compatibility of libs that link date_time library

* suppressed const qualifier on function return type (#138)

* suppressed const qualifier on function return type

As warned in https://www.boost.org/development/tests/develop/output/teeks99-dkr-dg4-6-warn-date_time-gcc-4-6~c++0x~warn-warnings.html#hours_special_value
```../boost/date_time/gregorian/greg_weekday.hpp:52:41: warning: type qualifiers ignored on function return type [-Wignored-qualifiers]
../boost/date_time/gregorian/greg_weekday.hpp:61:40: warning: type qualifiers ignored on function return type [-Wignored-qualifiers]```

* trivial commit to trigger CI

* fix c++2a ambiguity on clang10 with operator== issue #132 (#140)

Co-authored-by: joaquintides <joaquin.lopezmunoz@gmail.com>
@Anticom Anticom closed this as completed Jan 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Anticom