Started by felo, March 10, 2010, 05:48:22 PM
0 Members and 1 Guest are viewing this topic.
Quote from: whoami on May 30, 2012, 07:09:04 PM@Eipi: since you work on the comments, your changes and mine are the ideal complement to each other, yours to be applied first. Do you keep this in sync with the ongoing development?
Quote from: Dwachs on June 06, 2012, 07:15:25 PMWhat is the point in renaming get_count to get_count_ first?
QuoteYou should also adjust the usage-comment, the command is check not checkall.
QuoteNow I do not know how to proceed.
QuoteTranslating all the code at once seems to be not sensible.
QuoteI do not want to end up with names with trailing underscores.
Quote from: whoami on June 06, 2012, 07:58:10 PMI looked for professional tools: MSVS 2010 seems to be available only as time-limited trial (nowadays), not as express edition, and tools like Visual Assist (which should do C++ refactoring) do not support the latter anyway. So I will stick with VS 2008.
Quote from: Markohs on June 14, 2012, 07:49:36 AMtriple empty line separating fabrikbauer_t::alles_geladen and fabrikbauer_t::finde_anzahl_hersteller (line 247, should be just 2 empty lines, this happens everywere)
Quote from: Dwachs on June 14, 2012, 06:26:38 AMDid you find any methods etc that could be deleted or moved to a different class (to provide cleaner interfaces) ?
/*! \brief Convenience function that creates the required objects to initialise the CEGUI system. The created Renderer will use the current OpenGL viewport as it's default surface size. This will create and initialise the following objects for you: - CEGUI::OpenGLRenderer - CEGUI::DefaultResourceProvider - CEGUI::System \param display_size Size object describing the initial display resolution. \param tt_type Specifies one of the TextureTargetType enumerated values indicating the desired TextureTarget type to be used. Defaults to TTT_AUTO. \return Reference to the CEGUI::OpenGLRenderer object that was created. */ static OpenGLRenderer& bootstrapSystem(const Size& display_size, const TextureTargetType tt_type = TTT_AUTO);
/// <summary>/// This functions documentation/// </summary>/// <param name="aa">blablabla</param>/// <returns></returns>bool function1(TypeX aa);
/*! * blablabla * \param */
/*! * Doc * \param a explanation * \param b explanation * \return explanation */
Quote from: Markohs on June 21, 2012, 10:40:49 PMAbout the documentation decisions we are taking, don't you think it's time to incorporate them to documentation/coding_styles.txt? looks like dwachs is already applying it in the code from what I just saw on last svn commits. I can make a draft if you want.
/** * This file is under <this particular license>. * See "license.txt". * * @file <filename> <One-sentence description> * @author <Author> * @date <Last edit date> */
Quote from: eipi on June 28, 2012, 11:06:18 AMThe translation patches have been updated against r5788. I made some small updates to ./bauer and ./besch.
Quote from: eipi on June 28, 2012, 11:06:18 AMAs to the file header comments: What about something like this: