19: BUG, moving the mouse to the send button is an anoyance
REMOVE: Send button and the Clear button
ADD: code to collect and post message from the message textview
ADD: code in MainWindow::HandleMessage to clear and send message text
18: BUG, code is hard to read with mainwindowLayout files
ADD: move all the code into mainwindow source & header files
17: BUG, tab between input textviews, to speed up typing input settings, use set tab order
ADD: m_pcTextUserName->SetTabOrder( iTabOrder++ ) ... m_pcTextServerChannel->SetTabOrder( iTabOrder++ )
16b:BUG, applications store their settings in Settings folder
REMOVE: genenv("HOME")/IRC-Bookmarks/
ADD: genenv("HOME")/Settings/bsIRC/
16a:BUG, check bookmark is valid, else the app will crash
ADD: SetBookmarkPath() to ensure the folders exist and/or create
15: BUG, on selected bookmark the delete button should also clear the textview fields
ADD: MainWindow::RemoveBookmarkFromList(), m_pcTextUserName->Clear() ... m_pcTextBookmarkName->Clear()
14: BUG, os::Variant is the return type for textview GetValue()
ADD: GetValue().AsString() to return an os::String type
13: BUG, remove reduntant variables from mainwindow header file
REMOVE: cMessage, cBookmarkPath, cBookmarkName, cName, cPassword, cRealName, cServerName, cServerPort, cServerChannel
12: BUG, need timestamp on textview lines
ADD: os::DateTime::Now() in MainWindow::AddStringToTextView()
11: BUG, remove libc file functions, and use libsyllable storage functions
ADD:
10: BUG, hard to read bookmark code
MOVE: MainWindow::HandleMessage() bookmark code to descriptive methods
ADD: SetBookmarkPath(), LoadBookmarkList(), AddBookmarkToList(), RemoveBookmarkFromList(), SelectBookmarkFromList()
9: BUG, remove CR & LF chars everywhere, and only put once in commthread send method
REMOVE: "\r\n" in all the code
ADD: "\r\n" in CommThread::Send()
8: BUG, intermittent loss of ping command, no post these to mainwindow, test in commthread
MOVE: MainWindow::PingPong() to CommThread::PingPong(), no postmessage to mainwindow gives more reliable ping command
7: BUG, use libsyllable message labels os::M_ABOUT_REQUESTED, os::M_QUIT_REQUESTED
REMOVE: M_APP_QUIT, M_MENU_ABOUT, M_MENU_QUIT,
6: BUG, NO need to display the PING messages
ADD: MainWindow::PingPong()
5: BUG, Vertical Scrollbar is NOT set to the bottom of the server text
ADD: MainWindow::AddStringToTextView()
4: BUG, CommThread class will call libnet methods
ADD: www.jarvoll.se/syllable/files/looper.zip, CommThread class files ( commthread.cpp/commthread.h )
REMOVE: class CommThead : public os::Looper
ADD: class CommThead : public os::Thread, a thread is better choice for polling recv()
3: BUG, /* libnet.so - the highlevel network API library for Syllable
* Copyright (C) 2004 Kristian Van Der Vliet */
libnet.so not found in Syllable 0.6.3/0.6.4/0.6.6/0.6.7
ADD: libnet source files
ADD: Makefile, static linked libnet to libnet.a
2: BUG, Numeric sIRC bugs, libnet is more capable.
REMOVE: sIRC code from mainwindow.cpp/mainwindow.h
1b: BUG, Bookmark Connect button -> app crash if no bookmark selected
ADD: if(nSelected == -1) break;
1a: BUG, Bookmark Remove button -> app crash if no bookmark selected
ADD: if(nSelected == -1) break;