> From [EMAIL PROTECTED] Fri Jul 21 14:18:07 2000
> Date: Fri, 21 Jul 2000 13:15:36 -0700 (PDT)
> From: [EMAIL PROTECTED]
> To: [EMAIL PROTECTED]
> Subject: Re: LPRng: IFHP: common filters and '+='
>
...
> >
> > What concerns me is the "bad pattern" error:
> >
> >   Filter_status: bad pattern in 'file_output_match' - ']' at 22:55:05.628
> >
> > This *only* occurs if I use '+=' in the above file_output_match...if I manually
> > include the common filters, it works just fine.  Is there an obvious
> > syntax error that I'm missing?

I have a better solution:  please apply the following patch. This treats the
] and [ correctly.



*** ifhp.c      2000/07/17 02:35:46     1.60
--- ifhp.c      2000/07/23 14:24:40
***************
*** 3205,3222 ****
                        if(DEBUGL2)Dump_line_list("Send_job: after files", &match );
                }
                for( i = 0; i < match.count; ++i ){
!                       DEBUG2("Send_job: [%d] '%s'", i, match.list[i] );
                        Free_line_list(&l);
!                       Split_count(2,&l,match.list[i],Whitespace, 0, 0, 0, 1, 1 );
                        if(DEBUGL2)Dump_line_list("Send_job: checking against", &l );
                        if( l.count == 0 ) continue;
                        s = l.list[0];
-                       if( !strcmp(s,"]") || !strcmp(s,"[") ){
-                               if( l.count != 1 ){
-                                       LOGMSG( "Send_job: bad pattern in 
'file_output_match' - '%s'", s );
-                               }
-                               continue;
-                       }
                        if( l.count != 2 && l.count != 3 ){
                                LOGMSG( "Send_job: wrong number of fields in 
'file_output_match' - '%s'", s );
                                continue;
--- 3205,3219 ----
                        if(DEBUGL2)Dump_line_list("Send_job: after files", &match );
                }
                for( i = 0; i < match.count; ++i ){
!                       s = match.list[i];
!                       DEBUG2("Send_job: [%d] '%s'", i, s );
!                       if( !s ) continue;
!                       while( isspace(cval(s)) || cval(s) == ']' || cval(s) == '[' ) 
++s;
                        Free_line_list(&l);
!                       Split_count(2,&l,s,Whitespace, 0, 0, 0, 1, 1 );
                        if(DEBUGL2)Dump_line_list("Send_job: checking against", &l );
                        if( l.count == 0 ) continue;
                        s = l.list[0];
                        if( l.count != 2 && l.count != 3 ){
                                LOGMSG( "Send_job: wrong number of fields in 
'file_output_match' - '%s'", s );
                                continue;

Patrick Powell                 Astart Technologies,
[EMAIL PROTECTED]            9475 Chesapeake Drive, Suite D,
Network and System             San Diego, CA 92123
  Consulting                   858-874-6543 FAX 858-279-8424 
LPRng - Print Spooler (http://www.astart.com)

-----------------------------------------------------------------------------
If you need help, send email to [EMAIL PROTECTED] (or lprng-requests
or lprng-digest-requests) with the word 'help' in the body.  For the impatient,
to subscribe to a list with name LIST,  send mail to [EMAIL PROTECTED]
with:                           | example:
subscribe LIST <mailaddr>       |  subscribe lprng-digest [EMAIL PROTECTED]
unsubscribe LIST <mailaddr>     |  unsubscribe lprng [EMAIL PROTECTED]

If you have major problems,  send email to [EMAIL PROTECTED] with the word
LPRNGLIST in the SUBJECT line.
-----------------------------------------------------------------------------

Reply via email to