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.

Member ConnectBar::_Layout ()
Should add a drop down box or something to determine the transfer protocol if this is going to be a general purpose file transfer tool.

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 FTPRemoteView. 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 FTPLocalView
This should be renamed since it works with any type of server, not just an FTPServer.

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

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

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

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

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

Member FTPRemoteView::ShowMessage (const String &zMsg)
Currently this doesn't have any functional purpose. It should do something besides output a message in the console.

Member FTPRemoteView::HideMessage ()
Should change this so that it hides the message created by ShowMessage() or remove it if ShowMessage() is no longer used.

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

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

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

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

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

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, Handler *pcTarget, int nParallelConnections)
This should be filled in with things that are specific to FTP Server Connections.

Class FTPSettings
FTPSettings class should be renamed to TransferrerSettings because it is not FTP protocol-specific.

Class FTPView
This should be renamed to TransferrerView to reflect that it works with generic views, not just FTP views.

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.

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 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 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::_CleanupNode (int nNode)
Notify GUI.

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

Generated on Sun Oct 26 21:16:25 2008 for Transferrer by  doxygen 1.5.1