Having had a good close look, there is a little more that is needed before committing to CVS.
1) You've skipped at least one dialog - IDD_S_FROM_CWD - which is what was breaking local_dir. (Oh, and the functions weren't virtual in some child classes :}). 2) See download.cc - is next_dialog still used, and should a fail result in the previous behaviour? I've attached an updated patch for you with the virtual functions actually still virtual. Rob
Gary2.patch.bz2
Description: Binary data