22.02.2012 6:27, Claudio Valderrama C. wrote: > I think that playing here with streamList vs this->streamList is a recipe > for misreading the code and making mistakes:
This is why I prefer to prefix all member variables with "m_", e.g. m_streamList. Others may surely disagree with the chosen syntax but shadowing *must not* be allowed, IMO. > Besides, I wonder why we use a custom definition for this stream > list, > Firebird::Array<USHORT> streamList; > probably in this case, it's better than the canonical definition > typedef Firebird::HalfStaticArray<UCHAR, OPT_STATIC_ITEMS> StreamList; > that other places use? It seems so. Dmitry ------------------------------------------------------------------------------ Virtualization & Cloud Management Using Capacity Planning Cloud computing makes use of virtualization - but cloud computing also focuses on allowing computing to be delivered as a service. http://www.accelacomm.com/jaw/sfnl/114/51521223/ Firebird-Devel mailing list, web interface at https://lists.sourceforge.net/lists/listinfo/firebird-devel