Mud-H Posted February 19, 2015 Posted February 19, 2015 I have been working with T3D for years now and I never reported issues I found. Most are minor but I still think they should be fixed in the engine for future. Where I should report those? Here, on GitHub or on GG forum?For example, I'm currently experimenting for the first time with GuiMenuBar and just found a minor issue with GuiMenuBar::ClearMenus:From Online Reference: void GuiMenuBar::clearMenus ( int param1, int param2 ) Clears all the menus from the menu bar. Example: // Inform the GuiMenuBar control to clear all menus from itself. %thisGuiMenuBar.clearMenus();You can see tro parameters in function: param1, and param2. Unline the example clearMenus(); don't work, it required the 2 unused parameters. (clearMenus("",""); works fine. I had a look into the code and found this:DefineEngineMethod( GuiMenuBar, clearMenus, void, ( S32 param1, S32 param2),, "@brief Clears all the menus from the menu bar.\n\n" "@tsexample\n" "// Inform the GuiMenuBar control to clear all menus from itself.\n" "%thisGuiMenuBar.clearMenus();\n" "@endtsexample\n\n" "@see GuiTickCtrl") { object->clearMenus(); }I don't know why those 2 parameters are there but they should be removed...So, how and where I should report this? I have some others to report. Quote
Azaezel Posted February 19, 2015 Posted February 19, 2015 Reproduceable bugs would go https://github.com/GarageGames/Torque3D/issues Quote
Mud-H Posted February 19, 2015 Author Posted February 19, 2015 Posted: https://github.com/GarageGames/Torque3D/issues/1209Thanks for pointing me to right place Quote
buckmaster Posted February 19, 2015 Posted February 19, 2015 I should note that with bug reports, unless the code is obviously wrong, we'd really appreciate instructions on how to reproduce the issue, with stock script templates like Empty or Full if possible. It means we can more quickly validate the problem/solution than having to work it out ourselves! Quote
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.