Todo List

Member rtop::BrowsePanelData< T >::capacity

: no explanation for using this. rationalize (because we are maintaining a vector of pointers?)

Member rtop::BrowsePanelData< T >::header

: header disappers when browsing a menu longer than screen height. add separate panel to hold header

Member rtop::BrowsePanelData< T >::object_list

: inefficient method of accessing data. replace either with linked list or some other mechanism

Member rtop::BrowsePanelData< T >::setHeader (std::string head_str) override

: will become redundant once header is implemented in separate panel. remove

Member rtop::Columns::markPid ()

: not implemented. implement

Member rtop::Columns::set (std::vector< int >)

: redundant. remove

Class rtop::Info

: need to investigate usefulness and performance. investigate and refactor

Class rtop::InfoProc< T >

: seems a bloated way to hold data. investigate & refactor

Member rtop::InfoProc< T >::longDesc ()

: redundant. remove

Member rtop::KeyDict::resolve (int, View *)

: not implemented. implement

Member rtop::ProcInfo::kill ()

: not implemented. implement it

Member rtop::ProcInfo::read ()

: modify loop to read all properties from /proc/pid/stat file, then compute different property values by combining entries

: hardcoded property name for Command. receive this value during initialization

Member rtop::ProcInfo::sort ()

: long and int sorting handled separately. investigate and refactor?

Member rtop::ProcViewPanel::createColumn (EditPanelData< InfoProc< std::string >> &, int, int, std::string)

: header color being set using hardcoded number. set properly.

Member rtop::ProcViewPanel::refresh () override

: no rationale for using mutex here. rationalize

: no mutex to restrict access to clm_names. rationalize and implement

Member rtop::ProcViewPanel::syncColors () override

: some bug is assigning header color to procviewpanel's columns. this function not doing its job. investigate

Member rtop::ProcViewPanel::y_start

: not being used. make use of

Member rtop::SimplePanelData::moveTo (int, int)

: may cause bugs. investigate

Member rtop::StateMachine::next (int)

: this is absurd as curr_state should always be one option in transition table. investigate, remove

Member rtop::StringParser::operator<< (std::string)

: no rational. rationalize

Member rtop::toChar (std::string str1)

: clarify the problem, and improve solution. refactor