Todo List

Member AddressField::_Private::_AddButton (String label, String path)
Use proper icons.

Class AddressFieldButton
This should be linked to the current EFileBrowser code listed in the source directory so that they do not become too distinct from each other.

Class App
This should be changed so that it doesn't create the window until the arguments have been verified. This will avoid the output of any extra information except for the usage information.

Class ConnectBar
Need to add a function for retrieving the value of the DropdownMenu that contains the desired connection type.

Member DirListNode::Write (void *pBuf, size_t nSize)
The dirlist parsing might be better in Server, but we need to do it here because we need to save any fragments of the last line.

Member DirListNode::Write (void *pBuf, size_t nSize)
Fix up initial, final flags that are sent to the RemoteIconView. Maybe remove them. Currently the 'final' flag isn't being sent.

Member DownloadNode::Seek (curl_off_t nOffset, int nOrigin)
Make sure that File::Seek() returns the same as libcurl expects.

Class FTPServer
Functionality specific for FTP style connections should be moved here in order to better abstract the Server object.

Member FTPServer::FTPServer (const String &zURL, Window *pcTarget, AppSettings *pcSettings)
This should be filled in with things that are specific to FTP Server Connections.

Member LocalIconView::LocalIconView (const os::Rect &, const String &zName)
zName is currently ignored because of a limitation with libsyllable (to be fixed in later version).

Member MainWindow::HandleMessage (os::Message *)
Need code to determine the type of server connection to create depending on the type of connection (This will become more important when there are more server types).

Member MainWindow::HandleMessage (os::Message *)
Check that the new url is actually different to the old one, and only proceed if so

Member MainWindow::ConstructURL (const String &zHost, int nPort, const String &zUser, const String &zPassword)
This should be changed so that it can decide whether to create a a URL string for ftp or whatever other protocol that is desired.

Class ProgressWindow
When this window is closed it closes the whole program. It should just hide until the Terminate method is called.

Member ProgressWindow::ProgressWindow (MainWindow *mainWindow, AppSettings *pcSettings)
Must change the panels so that when the window is resized, the ListView is resized as well.

Member ProgressWindow::ProgressWindow (MainWindow *mainWindow, AppSettings *pcSettings)
Need to set up the ListView so that it's scroll bars will become visible when necessary.

Member QueueNode::m_nType
Should this be an enum type?

Member QueueNode::m_nStatus
Should this be an enum type?

Member QueueNode::GetStatus ()
Should this be here since it is a GUI access methods?

Member QueueNode::GetType ()
Should this be here since it is a GUI access methods?

Member QueueNode::GetProgress ()
Currently this just returns 0.

Member QueueNode::GetProgress ()
Should this be here since it is a GUI access methods?

Member QueueNode::WriteCallback (void *pBuf, size_t nSize, size_t nMult, void *pCookie)
Verify descriptions of callback parameters with libcurl documentation.

Class RemoteIconView
Once this is under CVS all ifdef 0 code should be removed.

Class RemoteIconView
This should be renamed so that it doesn't sound like it's unique to FTP servers only.

Member RemoteIconView::RemoteIconView (const Rect &cFrame, const String &zName)
Should cancel any previous dirlistings if a new listing is requested

Member RemoteIconView::SetServer (Server *pcServer)
Find out from the server what the current dir is via PWD.

Member RemoteIconView::ShowMessage (const String &zMsg)
Fix the crashes (may be libsyllable bugs?).

Member RemoteIconView::HideMessage ()
Not implemented yet since ShowMessage() doesn't work.

Member RemoteIconView::SetContents (std::vector< RemoteNode > *pacContents, bool bInitial, bool bFinal)
Perform periodic layouts while downloading large dirlisting.

Member RemoteIconView::SetContents (std::vector< RemoteNode > *pacContents, bool bInitial, bool bFinal)
Display file sizes in a human readable format?

Member RemoteIconView::SetContents (std::vector< RemoteNode > *pacContents, bool bInitial, bool bFinal)
Move the entry path stuff somewhere else?

Member RemoteIconView::GetNodeImage (RemoteNode *pcNode, bool bSmall)
Ask the registrar for a suitable icon, based on the file extension.

Member RemoteIconView::GetNodeImage (RemoteNode *pcNode, bool bSmall)
Add basic file/dir icons as resources, and fall back to them if the icon file is missing.

Member RemoteIconView::Invoked (uint nIcon, IconData *pcData)
Should attempt to dowload and open the remote file if it's not a directory?

Member RemoteIconView::Invoked (uint nIcon, IconData *pcData)
What should be the default action when the file type is unknown?

Member RemoteIconView::MouseUp (const Point &cPoint, uint32 nButtons, Message *pcData)
Implement recursive transfer when a folder is dropped.

Class Server
ParseDirListing function should be removed if it is unnecessary.

Member Server::~Server ()
Is it true that all other cleanup is done in TransferThread.

Member Server::~Server ()
I want this to be private but can't be bothered debugging the error messages yet.

Member Server::_SetupCurlHandle (CURL *pHandle)
This should be a virtual function that is called by the particular server type. Currently it directly implements FTP style transfers which defeates the purpose of having separate Server and FTPServer classes.

Class SettingsMenu
Currently some of the information in the menu is specific to FTPServer objects.

Class SettingsMenu
Should have a way to show and hide the progress window.

Class TransferThread
The class needs to be cleaned up properly.

Member TransferThread::m_apcMessages
Could use std::pair< Transfer*, int > instead?

Member TransferThread::m_pCurlMultiHandle
This needs better documentation.

Member TransferThread::~TransferThread ()
Other curl cleanup - remove all easy handles from the multi handle

Member TransferThread::_ProcessMessages ()
Move the code for each case into separate functions.

Member UploadNode::Seek (curl_off_t nOffset, int nOrigin)
Make sure that File::Seek() returns the same as libcurl expects.

Generated on Fri Jan 2 17:22:53 2009 for Transferrer by  doxygen 1.5.1