From: Alex Henrie Subject: Re: [PATCH] winepulse.drv: Avoid unnecessary use of ceil function Message-Id: Date: Wed, 5 Dec 2018 10:36:33 -0700 In-Reply-To: <20181205133445.3ftfo7bt2es3l7w2@foghorn.codeweavers.com> References: <20181202230934.12751-1-alexhenrie24@gmail.com> <20181205133445.3ftfo7bt2es3l7w2@foghorn.codeweavers.com> On Wed, Dec 5, 2018 at 6:34 AM Andrew Eikum wrote: > > I guess I'm not opposed to this, but my feeling is "if it's not > broken, don't fix it". Maybe there's something I don't know. Can you > explain what advantage this has? To me it seems cleaner to just use integer operations instead of casting to floating-point and back again. -Alex