Boost logo

Boost :

From: Johan Nilsson (r.johan.nilsson_at_[hidden])
Date: 2006-11-15 02:59:29

Christopher Kohlhoff wrote:
> Johan Nilsson <r.johan.nilsson_at_[hidden]> wrote:
>> Imagine that you're implementing a "middle-layer" library,
>> calling down to "lower-layer" methods that do the actual O/S
>> specific calls. The lower-layer methods might be C methods, or
>> a third-party library.
>> Examples for comparison:
> [...]
> In my experience the lower layer tends to look more like:
> void foo()
> {
> error_code ec;
> foo(ec); // Called foo_impl() in your examples
> if (ec) throw system_error(ec);
> }
> error_code foo(error_code& ec)
> {
> ... windows implementation ...
> ec = error_code(::GetLastError(), native_ecat);
> #else
> ... posix implementation ...
> ec = error_code(errno, native_ecat);
> #endif
> return ec;
> }

There's still code duplication here, which might or might not be possible to
get rid of depending on what actually could be hiding in the "... <os>
implementation ..." parts.

> At least, this is my understanding of how error_code and
> system_error should be used.

"Should" is a strong word, IMHO. I find the usage of error_code as a
parameter to methods pollutes the domain interfaces and make them more
complex. I'd probably prefer a TSS-based solution.

> IMHO if the lowest level wrappers
> use the error_code class in this way, there's no need for a
> publicly available wrapper for ::GetLastError()/errno.

That's a big if. If we had this facility, I could use it and you could still
not use it. You, as a user, don't need to pay for what you don't need.
(Also, you're missing out the modifying wrappers for SetLastError, errno=).

I feel this is similar to saying that there's no need for the std facility
"foo", as you can use the native "foo" functionality at the lowest level.
Isn't this all about making usage of common functionality more

> Having said that, I do have some "portable" helper functions
> inside of asio to do effectively what you're proposing,

There it is. That makes us at least two people who have written their own
similar functionality ;-)

> However
> they're only truly useful on a subset of the functions and are
> mostly a hangover from before I switched asio to use
> error_code/system_error.

Many subsets in many libraries end up with a pretty big "user" base. The
general usefulness would also depend on the interface of the helpers.

I respect your opinions, but I don't agree with you.

Regards / Johan

Boost list run by bdawes at, gregod at, cpdaniel at, john at