Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tftp 1.1 #8

Merged
merged 1 commit into from Aug 14, 2019

Conversation

@ducasp
Copy link
Contributor

commented Jul 22, 2019

  • Do not wait extra tick after TCPIP_WAIT
  • Changes a bit the way to print transfer progress: do not print the .0 or .5, just print the text string first with the unit before the numbers, and then on each iteration just print the numbers, going back before doing so, this is faster and allow faster transfers
  • Changed the routine to convert size to ASCII to one that is simpler and faster, that one was overkill and not as fast I believe as it had to check lots of possibilities and use a lot IX registers, when no other options are being used... :-)
This version will not wait extra tick after TCPIP_WAIT and changes a …
…bit the way to print transfer progress (do not print the .0 or .5, just print the text string first with the unit before the numbers, and then on each iteration just print the numbers, going back before doing so, this is faster and allow faster transfers). Also changed the routine to convert size to ASCII to one that is simpler and faster, that one was overkill and not as fast I believe as it had to check lots of possibilities and use a lot IX registers, when no other options are being used... :-)

@Konamiman Konamiman force-pushed the ducasp:tftp_1.1 branch from 0481495 to f3c6c2e Jul 23, 2019

@Konamiman Konamiman merged commit d23c6ee into Konamiman:master Aug 14, 2019

@Konamiman Konamiman deleted the ducasp:tftp_1.1 branch Aug 14, 2019

@Konamiman Konamiman restored the ducasp:tftp_1.1 branch Aug 14, 2019

@Konamiman Konamiman deleted the ducasp:tftp_1.1 branch Aug 14, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.