MirrorHub/construct
0
0
Fork 0
mirror of https://github.com/matrix-construct/construct synced 2024-09-26 18:38:52 +02:00
Code Issues Releases Wiki Activity

ircd::ctx: Eliminate the std future_status and simplify our real-use interfaces.

Browse source
This commit is contained in:
Jason Volk 2018-04-05 21:19:16 -07:00
parent 022bf95f7f
commit 8d0681e7b1
7 changed files with 42 additions and 71 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

96
include/ircd/ctx/future.h
View file

@ -18,16 +18,8 @@ namespace ircd::ctx
template<class T = void> class future;
template<> class future<void>;
template<class... T> struct scoped_future;
enum class future_status;
}
enum class ircd::ctx::future_status
{
ready,
timeout,
deferred,
};
template<class T>
struct ircd::ctx::future
:private shared_state<T>
@ -43,12 +35,12 @@ struct ircd::ctx::future
bool operator!() const { return !valid(); }
operator bool() const { return valid(); }
template<class U, class time_point> friend future_status wait_until(const future<U> &, const time_point &, std::nothrow_t);
template<class U, class time_point> friend future_status wait_until(const future<U> &, const time_point &);
template<class time_point> future_status wait_until(const time_point &, std::nothrow_t) const;
template<class time_point> future_status wait_until(const time_point &) const;
template<class duration> future_status wait(const duration &d, std::nothrow_t) const;
template<class duration> future_status wait(const duration &d) const;
template<class U, class time_point> friend bool wait_until(const future<U> &, const time_point &, std::nothrow_t);
template<class U, class time_point> friend void wait_until(const future<U> &, const time_point &);
template<class time_point> bool wait_until(const time_point &, std::nothrow_t) const;
template<class time_point> void wait_until(const time_point &) const;
template<class duration> bool wait(const duration &d, std::nothrow_t) const;
template<class duration> void wait(const duration &d) const;
void wait() const;
T get();
@ -76,12 +68,12 @@ struct ircd::ctx::future<void>
bool operator!() const { return !valid(); }
operator bool() const { return valid(); }
template<class U, class time_point> friend future_status wait_until(const future<U> &, const time_point &, std::nothrow_t);
template<class U, class time_point> friend future_status wait_until(const future<U> &, const time_point &);
template<class time_point> future_status wait_until(const time_point &, std::nothrow_t) const;
template<class time_point> future_status wait_until(const time_point &) const;
template<class duration> future_status wait(const duration &d, std::nothrow_t) const;
template<class duration> future_status wait(const duration &d) const;
template<class U, class time_point> friend bool wait_until(const future<U> &, const time_point &, std::nothrow_t);
template<class U, class time_point> friend void wait_until(const future<U> &, const time_point &);
template<class time_point> bool wait_until(const time_point &, std::nothrow_t) const;
template<class time_point> void wait_until(const time_point &) const;
template<class duration> bool wait(const duration &d, std::nothrow_t) const;
template<class duration> void wait(const duration &d) const;
void wait() const;
IRCD_OVERLOAD(already)
@ -100,7 +92,7 @@ namespace ircd::ctx
{
template<class T,
class time_point>
future_status wait_until(const future<T> &, const time_point &, std::nothrow_t);
bool wait_until(const future<T> &, const time_point &, std::nothrow_t);
}
template<class... T>
@ -240,24 +232,24 @@ const
template<class T>
template<class duration>
ircd::ctx::future_status
void
ircd::ctx::future<T>::wait(const duration &d)
const
{
return this->wait_until(steady_clock::now() + d);
this->wait_until(steady_clock::now() + d);
}
template<class duration>
ircd::ctx::future_status
void
ircd::ctx::future<void>::wait(const duration &d)
const
{
return this->wait_until(steady_clock::now() + d);
this->wait_until(steady_clock::now() + d);
}
template<class T>
template<class duration>
ircd::ctx::future_status
bool
ircd::ctx::future<T>::wait(const duration &d,
std::nothrow_t)
const
@ -266,7 +258,7 @@ const
}
template<class duration>
ircd::ctx::future_status
bool
ircd::ctx::future<void>::wait(const duration &d,
std::nothrow_t)
const
@ -276,32 +268,25 @@ const
template<class T>
template<class time_point>
ircd::ctx::future_status
void
ircd::ctx::future<T>::wait_until(const time_point &tp)
const
{
return ircd::ctx::wait_until(*this, tp);
ircd::ctx::wait_until(*this, tp);
}
template<class time_point>
ircd::ctx::future_status
void
ircd::ctx::future<void>::wait_until(const time_point &tp)
const
{
const auto status
{
this->wait_until(tp, std::nothrow)
};
if(status == future_status::timeout)
if(!this->wait_until(tp, std::nothrow))
throw timeout{};
return status;
}
template<class T>
template<class time_point>
ircd::ctx::future_status
bool
ircd::ctx::future<T>::wait_until(const time_point &tp,
std::nothrow_t)
const
@ -310,17 +295,12 @@ const
}
template<class time_point>
ircd::ctx::future_status
bool
ircd::ctx::future<void>::wait_until(const time_point &tp,
std::nothrow_t)
const
{
const auto status
{
ircd::ctx::wait_until(*this, tp, std::nothrow)
};
if(status == future_status::ready)
if(ircd::ctx::wait_until(*this, tp, std::nothrow))
{
auto &state
{
@ -330,31 +310,25 @@ const
set(state, future_state::RETRIEVED);
if(bool(state.eptr))
std::rethrow_exception(state.eptr);
}
return status;
return true;
}
else return false;
}
template<class T,
class time_point>
ircd::ctx::future_status
void
ircd::ctx::wait_until(const future<T> &f,
const time_point &tp)
{
const auto ret
{
wait_until(f, tp, std::nothrow)
};
if(ret == future_status::timeout)
if(!wait_until(f, tp, std::nothrow))
throw timeout{};
return ret;
}
template<class T,
class time_point>
ircd::ctx::future_status
bool
ircd::ctx::wait_until(const future<T> &f,
const time_point &tp,
std::nothrow_t)
@ -373,9 +347,7 @@ ircd::ctx::wait_until(const future<T> &f,
throw no_state{};
if(unlikely(!state.cond.wait_until(tp, wfun)))
return future_status::timeout;
return false;
return likely(wfun())?
future_status::ready:
future_status::deferred;
return wfun();
}

2
modules/client/directory/room.cc
View file

@ -222,7 +222,7 @@ room_alias_fetch(const mutable_buffer &out,
};
//TODO: conf
if(federation_request.wait(seconds(8)) == ctx::future_status::timeout)
if(!federation_request.wait(seconds(8), std::nothrow))
throw http::error
{
http::REQUEST_TIMEOUT

2
modules/client/profile.cc
View file

@ -163,7 +163,7 @@ get__profile_remote(client &client,
};
//TODO: conf
if(federation_request.wait(seconds(8)) == ctx::future_status::timeout)
if(!federation_request.wait(seconds(8), std::nothrow))
throw http::error
{
http::REQUEST_TIMEOUT

5
modules/console.cc
View file

@ -2036,10 +2036,9 @@ console_cmd__fed__groups(opt &out, const string_view &line)
node, vector_view<const m::user::id>(ids, count), buf, std::move(opts)
};
if(request.wait(seconds(10)) == ctx::future_status::timeout)
throw http::error{http::REQUEST_TIMEOUT};
request.wait(seconds(10));
request.get();
const json::object response
{
request.in.content

2
modules/federation/sender.cc
View file

@ -240,7 +240,7 @@ try
ctx::when_any(begin(txns), end(txns))
};
if(next.wait(seconds(2), std::nothrow) == ctx::future_status::timeout) //TODO: conf
if(!next.wait(seconds(2), std::nothrow)) //TODO: conf
return;
const auto it

4
modules/key/keys.cc
View file

@ -77,7 +77,7 @@ get__keys(const string_view &server_name,
};
//TODO: conf
if(tag.wait(seconds(20)) == ctx::future_status::timeout)
if(!tag.wait(seconds(20), std::nothrow))
{
cancel(tag);
throw m::error
@ -170,7 +170,7 @@ try
server_name, { head }, { in }
};
if(tag.wait(seconds(30)) == ctx::future_status::timeout)
if(!tag.wait(seconds(30), std::nothrow))
throw m::error
{
http::REQUEST_TIMEOUT, "M_TIMEOUT",

2
modules/media/thumbnail.cc
View file

@ -161,7 +161,7 @@ get__thumbnail_remote(client &client,
remote, { out_head }, { in_head, in_content }, &opts
};
if(remote_request.wait(seconds(10)) == ctx::future_status::timeout)
if(!remote_request.wait(seconds(10), std::nothrow))
throw http::error
{
http::REQUEST_TIMEOUT