Wow! It is a touch of master!
Despite I prefer TASM conventions your code is much cleaner than mine. I
like your use of AAM instruction :-)
The only thing I think was better in my code was common procedure for
converting number to string for both numeric systems :-/

>Note: In dosemu on my 2 GHz PC, 1 msec is only 600 delay loop iterations
And how many for pure DOS system?

>probe all printers and ignore command line?
No, I don't like this idea. Better would be to add switch "/A" for all
ports. However this is not needed because you can call LPTtest from batch
file for all ports.
Are you really, absolutely sure that PC can't have LPT4? My eletronical
manual about PC says (translation from czech): 0:040Eh - port LPT4, PS/2:
Extended BIOS data segment
BTW: My computer shows on 0:040Eh value 208h - it doesn't look as BIOS data
segment, IMO

> - tune delay handling further?
I think it is not necessary to more tune delay routine.
> - make "show for all printers" the default?
No.

> - out baseport,4 should be re-enabled after somebody figures out how to
check for ieee1284 printer presence before that
Or we can add some command line switch "/C" (compatibility)

----- Original Message -----
From: "Eric Auer" <[EMAIL PROTECTED]>
To: <freedos-devel@lists.sourceforge.net>
Sent: Friday, July 13, 2007 5:22 PM
Subject: Re: [Freedos-devel] LPTtest v1.3


>
> Hi all,
>
> > I added help screen, possibility to scan other LPT ports and a little
> > modyfied delay routine. For help screen write lpttest /? link:
> > http://www.laaca-mirror.ic.cz/lpttest.zip
> (compiles with TASM and TLINK)
>
> I have thrown Michael Devore's NoMySo Perl script at the source
> and made a NASM version for people who prefer free assemblers :-)
>
>
http://www.coli.uni-saarland.de/~eric/stuff/soft/by-others/lpttest-nasm.zip
>
>
>
> In addition, I messed with the code and modified a few things:
>
> - made the code nicely indented for easy reading
> - buffer is in BSS again now (and btw, 1024h is not 1024 ;-)
>   so binary size is 2 kilobytes again now
> - multi line strings are only one string, not multi
>
> - modified the port number / address display, new hex2str
> - modified int2str for fun, now uses 1x div and 2x aam ;-)
> - there is no lpt4, only lpt1 to lpt3. suggestion: probe all
>   printers and ignore command line? depends on whether and
>   how you want to use lpttest in batch files...
> - commented out the "out baseport,4" for testing. GOOD: this
>   means you no longer print a diamond in dosemu and non-pnp
>   printers. BAD: it is quite possible that some printers now
>   no longer show their pnp string, please test
> - modified the delay and delay calibration functions: now
>   waits 0.5 sec for calibration and 1 msec for each delay,
>   using a 48 bit counter. Note: In dosemu on my 2 GHz PC,
>   1 msec is only 600 delay loop iterations (because the loop
>   contains I/O port access which is slow). Suggestion: use a
>   32 bit counter and calibrate during only 0.054 sec?
>
> I hope you like the changes, Laaca :-).
>
>
>
> As said, open issues are:
>
> - tune delay handling further?
> - make "show for all printers" the default?
> - maybe even remove command line handling?
>
> - out baseport,4 should be re-enabled after somebody figures
>   out how to check for ieee1284 printer presence before that
>
> The last point has most priority now if you ask me :-).
>
> Eric
>
>
>
> -------------------------------------------------------------------------
> This SF.net email is sponsored by DB2 Express
> Download DB2 Express C - the FREE version of DB2 express and take
> control of your XML. No limits. Just data. Click to get it now.
> http://sourceforge.net/powerbar/db2/
> _______________________________________________
> Freedos-devel mailing list
> Freedos-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/freedos-devel


-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
_______________________________________________
Freedos-devel mailing list
Freedos-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/freedos-devel

Reply via email to