Input: post-code-review

[GoogleCodeExporter]

Bugs:
incorrect notify() handling in derived from input::Helper classes (as
Button,  and etc.)

Enhancements:
*use "boost::shared_ptr<Control>" instead of raw "Control*"
*InputImpl::Load change to LoadFromString
*std::list<CommandPtr> m_commands change to std::map<>

Remarks:
желательно использовать typedef для 
используемых сложных типов как Stl
контейнеры и итераторы
избыточное декарирование комментариев
избыточные комментарии (там где и так 
понятно, например на методы
Get,Destoy,Init)
использование паблик переменных в классе 
(Control)
в input::Input надо убрать сингельтон, заменив на 
статический Init(InputImpl*)
не аккуратность с форвард объявлениями. в 
результате InputImpl был всем виден.

Original issue reported on code.google.com by denis.o...@gmail.com on 12 May 2007 at 10:01

[GoogleCodeExporter]

принял

Original comment by atru...@gmail.com on 13 May 2007 at 7:52

[GoogleCodeExporter]

исправлен баг с notify

Original comment by atru...@gmail.com on 13 May 2007 at 8:25

• Changed state: Started

[GoogleCodeExporter]

[deleted comment]

[GoogleCodeExporter]

[deleted comment]

[GoogleCodeExporter]

closed

Original comment by denis.o...@gmail.com on 15 Oct 2009 at 3:33

• Changed state: Verified