[sane-devel] dell1600n_net backend

2006-05-21 Thread Henning Meier-Geinitz
Hi,

On 2006-05-09 22:07, Jon Chambers wrote:
 Any idea where select.h is on HPUX?  (select is a POSIX call so I guess it 
 is somewhere?)  I suspect that these issues are likely to be common to 
 most network scanner backends so maybe sanei_tcp.h would be the correct 
 place for this kind of #ifdef?

Some Linux manual page says:
/* According to POSIX 1003.1-2001 */
#include sys/select.h 

/* According to earlier standards */
#include sys/time.h
#include sys/types.h
#include unistd.h

saned.c should already have a check for this.

Bye,
  Henning


[sane-devel] dell1600n_net backend

2006-05-15 Thread Henning Meier-Geinitz
Hi,

On 2006-05-11 21:47, Jon Chambers wrote:
 On Wed, 10 May 2006, Julien BLACHE wrote:
 Create an account on http://alioth.debian.org and give us your login
 name so you can be added to the sane project.
 
 My login is phap016-guest.

I just added you to the project. It can take some hours until CVS
write access works.

Bye,
  Henning


[sane-devel] dell1600n_net backend

2006-05-15 Thread Henning Meier-Geinitz
Hi,

On 2006-05-10 07:44, Julien BLACHE wrote:
  backend myself as a diff to the most recent major release until it is
  stable or can it be got into the main repository before that?
 
 Henning will correct me if I'm wrong, but the backend can probably go
 in if it has at least basic support for the scanner and won't cause
 data loss if something goes wrong.

Yes, my rule of thumb is something like if it does something useful,
complies with the rules in backend-writing.txt (and the SANE
standard), doesn't break compilation and doesn't cause hardware damage
it can go in. For more fine-grained control we have the status levels
for each scanner.

Bye,
  Henning


[sane-devel] dell1600n_net backend

2006-05-11 Thread Jon Chambers

Hi Julien,

On Wed, 10 May 2006, Julien BLACHE wrote:
 Create an account on http://alioth.debian.org and give us your login
 name so you can be added to the sane project.

My login is phap016-guest.

Thanks,
Jon

== Jon Chambers =
  http://www.jon.demon.co.uk, 020 8575 7097, 07931 961669
=


[sane-devel] dell1600n_net backend

2006-05-10 Thread Julien BLACHE
Jon Chambers j...@jon.demon.co.uk wrote:

Hi,

 Before including the code into CVS, please run indent -gnu on it.

 What is the arrangement for CVS access?  Should I maintain this

Create an account on http://alioth.debian.org and give us your login
name so you can be added to the sane project.

 backend myself as a diff to the most recent major release until it is
 stable or can it be got into the main repository before that?

Henning will correct me if I'm wrong, but the backend can probably go
in if it has at least basic support for the scanner and won't cause
data loss if something goes wrong.

JB.

-- 
Julien BLACHE   http://www.jblache.org 
j...@jblache.org  GPG KeyID 0xF5D65169


[sane-devel] dell1600n_net backend

2006-05-09 Thread Henning Meier-Geinitz
Hi,

On 2006-04-23 15:16, Jon Chambers wrote:
 I have a nominally functional backend for network scanning using a Dell 
 1600n MFP.  The source can be found at 
 http://www.jon.demon.co.uk/sane-dell1600n_net/dell1600n_net.c

[...]

 I am keen to get the source into CVS and also to verify the build on some 
 other platforms (I suspect that some of the network headers may require 
 some #ifdef magic).

Ok, let's try (Linux i386):
dell1600n_net.c:78:20: error: tiffio.h: No such file or directory

I guess you must add some magic to not compile the backend if the jpeg
and tiff headers are not present. Or, if this is possible/makes sense,
to #ifdef out that functions calls in your backend.

With libtiff installed, I get these warnings:
dell1600n_net.c: In function 'sane_dell1600n_net_init':
dell1600n_net.c:225: warning: unused parameter 'authorize'
dell1600n_net.c: In function 'sane_dell1600n_net_get_devices':
dell1600n_net.c:263: warning: unused parameter 'local_only'
dell1600n_net.c: In function 'sane_dell1600n_net_get_option_descriptor':
dell1600n_net.c:498: warning: unused parameter 'handle'
dell1600n_net.c: In function 'sane_dell1600n_net_control_option':
dell1600n_net.c:522: warning: unused parameter 'handle'
dell1600n_net.c:522: warning: unused parameter 'info'
dell1600n_net.c: In function 'sane_dell1600n_net_read':
dell1600n_net.c:721: warning: comparison between signed and unsigned
dell1600n_net.c: In function 'sane_dell1600n_net_cancel':
dell1600n_net.c:749: warning: unused parameter 'handle'
dell1600n_net.c: In function 'sane_dell1600n_net_set_io_mode':
dell1600n_net.c:755: warning: unused parameter 'handle'
dell1600n_net.c:755: warning: unused parameter 'non_blocking'
dell1600n_net.c: In function 'sane_dell1600n_net_get_select_fd':
dell1600n_net.c:765: warning: unused parameter 'handle'
dell1600n_net.c:765: warning: unused parameter 'fd'
dell1600n_net.c: In function 'ClearKnownDevices':
dell1600n_net.c:787: warning: cast discards qualifiers from pointer target type
dell1600n_net.c:789: warning: cast discards qualifiers from pointer target type
dell1600n_net.c: In function 'JpegDecompTermSource':
dell1600n_net.c:1869: warning: unused parameter 'cinfo'

Doesn't look critical, but you could use __sane_unused__ to mark
unused parameters explicitely.

Ok, next test on HP-UX risc:
dell1600n_net.c:74:24: sys/select.h: No such file or directory
dell1600n_net.c:77:21: jpeglib.h: No such file or directory
dell1600n_net.c:78:20: tiffio.h: No such file or directory
[more errors deleted]


Before including the code into CVS, please run indent -gnu on it.

SANE_Status sane_get_select_fd (SANE_Handle handle,
SANE_Int * fd)
{

  return SANE_STATUS_ACCESS_DENIED;
  
Why is the access denied? I think the normal answer would be
SANE_STATUS_UNSUPPORTED as you don't implement the function. Same for
set_io_mode.

Bye,
  Henning


[sane-devel] dell1600n_net backend

2006-05-09 Thread Jon Chambers

Hi Henning,

On Tue, 9 May 2006, Henning Meier-Geinitz wrote:
 On 2006-04-23 15:16, Jon Chambers wrote:
 I have a nominally functional backend for network scanning using a Dell
 1600n MFP.
 I guess you must add some magic to not compile the backend if the jpeg
 and tiff headers are not present.

The joys of autconf yet to come...

 Ok, next test on HP-UX risc:
 dell1600n_net.c:74:24: sys/select.h: No such file or directory

Any idea where select.h is on HPUX?  (select is a POSIX call so I guess it 
is somewhere?)  I suspect that these issues are likely to be common to 
most network scanner backends so maybe sanei_tcp.h would be the correct 
place for this kind of #ifdef?

 Before including the code into CVS, please run indent -gnu on it.

What is the arrangement for CVS access?  Should I maintain this backend 
myself as a diff to the most recent major release until it is stable or 
can it be got into the main repository before that?

 Why is the access denied? I think the normal answer would be
 SANE_STATUS_UNSUPPORTED

Sorry - left over from when all the sane entry point functions were 
just placeholders!  I'll fix this.

Thanks,
Jon

== Jon Chambers =
  http://www.jon.demon.co.uk, 020 8575 7097, 07931 961669
=



[sane-devel] dell1600n_net backend

2006-04-23 Thread Jon Chambers

Hi,

I have a nominally functional backend for network scanning using a Dell 
1600n MFP.  The source can be found at 
http://www.jon.demon.co.uk/sane-dell1600n_net/dell1600n_net.c

The status is alpha:
- It builds ok on debian linux (i486) (with dependencies on libjpeg and
   libtiff).
- Single page scans are possible in mono and colour.
- There are still some wrinkles to be ironed out with multipage scans.

I am keen to get the source into CVS and also to verify the build on some 
other platforms (I suspect that some of the network headers may require 
some #ifdef magic).

cheers,
Jon

== Jon Chambers =
  http://www.jon.demon.co.uk, 020 8575 7097, 07931 961669
=