Patrick,

Thanks for looking into this.  Actually, here's a better patch
than my first one.  This one will deal with all HASH values,
not just the pagecount one (eg. "ID = blah" -> "ID=blah").

Unfortunately, this still didn't resolve my problems with
the HP Business Inkjet 2250.  The pagecount is fine, but now
the end of job (waitend) is timing out.

I have left copies of test results in my home directory if you
want to have a look.  http://www.geog.ubc.ca/~kujala/lprng/

On Tue, Oct 08, 2002 at 12:58:15PM -0700, Patrick Powell wrote:
> I will add the patch.  Actually,  I need to be a bit more
> agressive and remove additional spaces as well...  I will fix this up.
> 
> Thanks!
> 
> Patrick Powell                 Astart Technologies
> [EMAIL PROTECTED]            6741 Convoy Court
> Network and System             San Diego, CA 92111
>   Consulting                   858-874-6543 FAX 858-751-2435
> LPRng - Print Spooler (http://www.lprng.com)
> 

-- 
later,
Vincent Kujala (http://www.geog.ubc.ca/~kujala)
*** ifhp.c.orig Mon Jul  8 08:58:41 2002
--- ifhp.c      Thu Sep 19 12:11:12 2002
***************
*** 828,834 ****
                        Clear_OBJ(l);
                }
        } else if( eq_line ){
!               /* now we check for xx=value entries */
                Check_device_status(str, infovar );
                if( pjlvar ) SAFEFREE(pjlvar); pjlvar = 0;
                infovar = 0;
--- 828,853 ----
                        Clear_OBJ(l);
                }
        } else if( eq_line ){
!                 /* now we check for xx\s*=\s*value entries VKVKVK */
!                 str[safestrlen(str)] = 0; /* should this be appended instead? */
!                 eq_line = safestrchr( str, '=' ); /* this is redundant */
!                 /* eat whitespace to left of '=' */
!                 for (s = eq_line-1; str <= s; s--) {
!                    if (isspace(cval(s))) { 
!                       memmove( s, s+1, safestrlen(s)+1); /* risky behaviour */
!                    } else {
!                       break; /* stop at first non-space */
!                    }
!                 }
!                 /* eat whitespace to right of '=' */
!                 eq_line = safestrchr( str, '=' ); /* this is necessary */
!                 for (s = eq_line+1; safestrlen(s)>0; ) {
!                    if (isspace(cval(s))) {
!                       memmove( s, s+1, safestrlen(s)+1); 
!                    } else {
!                       break; /* stop at first non-space */
!                    }
!                 }
                Check_device_status(str, infovar );
                if( pjlvar ) SAFEFREE(pjlvar); pjlvar = 0;
                infovar = 0;
***************
*** 3542,3547 ****
--- 3561,3567 ----
        if( (s = GET_HASH_STR_OBJ( Devstatus, "pagecount", MEMINFO )) ){
                pagecounter = atoi( s );
        }
+ 
        DEBUG1("Do_pagecount: pagecount at start %d", pagecounter );
        if( use_prog ){
                if( !pagecounter ){

Reply via email to