From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <kostja@tarantool.org>
Date: Thu, 15 Aug 2019 01:25:10 +0300
From: Konstantin Osipov <kostja@tarantool.org>
Subject: Re: [PATCH v2 6/8] decimal: allow to encode/decode decimals as
 MsgPack
Message-ID: <20190814222510.GG28533@atlas>
References: <cover.1565263272.git.sergepetrenko@tarantool.org>
 <820cab72f09d48cf2d9e9cba5e0bba9620e32d96.1565263272.git.sergepetrenko@tarantool.org>
 <20190812213454.GI32337@atlas>
 <A9106CB2-19A2-4135-80F3-0D7026CF9A0F@tarantool.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <A9106CB2-19A2-4135-80F3-0D7026CF9A0F@tarantool.org>
To: Serge Petrenko <sergepetrenko@tarantool.org>
Cc: Vladimir Davydov <vdavydov.dev@gmail.com>, tarantool-patches@freelists.org
List-ID: <tarantool-patches.dev.tarantool.org>

* Serge Petrenko <sergepetrenko@tarantool.org> [19/08/13 17:05]:
> > This is a dependency hell.
> > 
> > Why did you add a separate header for decimal encoding/encoding if
> > you have to include *everything* in here anyway? Noone will ever
> > be able to understand why you decided to include all of these
> > here.
> 
> Do you want me to add comments describing why we need each header?

My point is, including mp_decimal.h should be enough.

> 

-- 
Konstantin Osipov, Moscow, Russia