[sane-devel] Minor issue with SANE standard

[Henning Meier-Geinitz]

On Mon, Dec 19, 2005 at 05:05:45PM -0700, Ron Hunter-Duvar wrote:
> This is for the people involved in developing and documenting the SANE 
> standard. 
> 
> When I was getting familiar with SANE, I read the latest standard (version 
> 1.03). A very well written standard, IMHO. I just had one little issue (yeah, 
> I'm being pedantic, it's a personality flaw 8^), with the formula for c in 
> section 4.3.8 (page 29).

I think you are right in both cases. This formula is in the standard
since version 0.3 (more or less exactly 9 years old now). I never
liked it for some reason :-)

[formula 2:]

The bug was already discovered during discussion of SANE2, but was
never fixed in SANE1.

> 	B * n * ceiling(d / 8)
> 
> or:
> 
> 	B * n * floor ((d + 7) / 8)

Or simply:

B * n * d / 8

(d can only be 8 or 16 according to the standard).

Please everybody check if the following patch if correct. If I don't
hear any objections, I'll commit it to CVS.

Bye,
  Henning

Index: doc/sane.tex
===================================================================
RCS file: /cvsroot/sane/sane-backends/doc/sane.tex,v
retrieving revision 1.17
diff -u -u -r1.17 sane.tex
--- doc/sane.tex	16 Oct 2005 17:20:06 -0000	1.17
+++ doc/sane.tex	23 Dec 2005 16:27:53 -0000
@@ -14,7 +14,7 @@
 
 \title{\huge SANE Standard Version 1.03}
 \author{}
-\date{2005-10-16} 
+\date{2005-12-23} 
 
 \makeindex
 
@@ -1459,8 +1459,8 @@
 \[
   c >= \left\{
   \begin{array}{ll}
-    \lceil B\cdot n / 8\rceil & \mbox{if $d=1$}\\
-    B\cdot n \cdot \lceil (d + 7)/8 \rceil & \mbox{if $d>1$}
+    B\cdot \lfloor (n + 7) / 8\rfloor & \mbox{if $d=1$}\\
+    B\cdot n \cdot d / 8 & \mbox{if $d>1$}
   \end{array}
   \right.
 \]