spinhawk icon indicating copy to clipboard operation
spinhawk copied to clipboard

CTCE v2 Fix for CTCE_Recovery

Open Peter-J-Jansen opened this issue 4 years ago • 4 comments

The CTCE_Recovery routine, also used by command DEVINIT <CTCE_devnum> processing, could cause a Hercules crash. This is now fixed. Additionally, cosmetic changes were made in order to minimize diff changes when comparing with other Hercules versions.

Peter-J-Jansen avatar Mar 20 '20 16:03 Peter-J-Jansen

This change has some .vs files in it. Were these included by mistake?

rbowler avatar May 03 '20 14:05 rbowler

I hope you are well Roger !

Yes, the .vs files were a mistake and can be deleted. I think this was caused by me using the MS VS 2019 Git / GitHub support to update my fork.

Thanks for merging these CTCE fixes (and some others). I should also update the README.NETWORKING file.

Stay healthy !

Cheers,

Peter

Peter-J-Jansen avatar May 03 '20 15:05 Peter-J-Jansen

@Peter-J-Jansen : if you could resubmit this pull request with the .vs files removed I can progress it. Thanks.

rbowler avatar Nov 07 '20 14:11 rbowler

Will do Roger !

I will also remove some by now unnecessary comments to minimize diff output with the other Hercules version, and add support for the new CTCE optional ATTNDELAY parameter (developed whilst working on VTAM 3 with Bob Polmanter), and finally test it all again. This might take some time. OK ?

Cheers,

Peter

Peter-J-Jansen avatar Nov 08 '20 09:11 Peter-J-Jansen