This wiki has been moved to https://github.com/SuperTux/wiki into the mediawiki branch.

TODO-List

From SuperTux
Revision as of 02:48, 22 August 2013 by Giby (Talk | contribs)

Jump to: navigation, search

This is the actual thing to solve before Milestone 2

Major Issue

These Issus Need particular knowledge of coding.

  • SDL2 Migration
    • Rendering
    • Control
    • Files to edit:
      • joystickkeyboardcontroller.hpp
      • joystickkeyboardcontroller.cpp
      • gl_renderer.cpp
      • sdl_renderer.cpp
      • console.cpp
      • screen_manager.cpp
      • title_screen.cpp
      • keyboard_menu.hpp
      • addon_menu.cppjoystickkeyboardcontroller.hpp
      • joystickkeyboardcontroller.cppgl_renderer.cpp
      • sdl_renderer.cppfunctions.cpp
      • physfs_sdl.cpp
      • button_group.cpp
      • button.hpp
      • button.cpp
      • firefly.cpp
      • main.cpp
    • Fix the fullscreen/windows mode management
  • Fix the editor
    • adding function/debug
    • Re-writte/Translate code in C/C++
Why? If you want a C++ editor use flexlay.

Minor Issue

  • Non-fatal bug to fix:
    • sqapi.cpp
    • sqdebug.cpp
    • sqobject.h
    • sqfuncstate.cpp
    • sqvm.cpp
    • sqstdrex.cpp
    • brick.cpp
    • icecrusher.cpp
    • ispy.cpp
  • Localisation improve
    • Translation of menu and worlds [1]
    • Font design for non latin alphabet (traditional Chinese, Nepali, Arabic)

Misc.

From previous TODO list, some point should be rethink, or may have been corrected …

Coding Standard Stuff

  • make code clean: "-O2", "-g3",
    • "-ansi",
    • "-pedantic",
    • "-Wall",
    • "-Wextra",
    • "-Wnon-virtual-dtor",
    • "-Weffc++",
    • "-Wconversion",
    • "-Werror",
    • "-Wshadow",
    • "-Wcast-qual",
    • "-Winit-self", # only works with >= -O1
    • "-Wno-unused-parameter",
  • remove overuse of multi-inheritance
  • remove overuse of friend'ship
  • maybe mark interfaces as interfaces (ISerializable or SerializableInterface)
  • split files with multiple classes into multiple files with one class each
  • Decide what to do with magic constants of objects (static vs anonymous namespace vs lisp property)
  • check the code with Valgrind and profilers
  • use Vector in Physics for 'a' and 'v'
  • replace random generator with mersenne twister and/or move to external/ (maybe use boost's random stuff)
  • md5.hpp and random_generator.hpp could go to external/
  • write/finish scripts to automatically:
**make all includes relative to top level dir
**sort includes (.hpp file, then system includes, then other project files)
**include guards proper and of the form HEADER_SUPERTUX_${PATH_TO_FILE}_HPP

Suggesions

  • implement a system that allows to attach comments to specific regions in a level
  • implement a tool to "screenshot" a complete level
  • GameObject::RemoveListenerListEntry: Ughs, somebody trying to implement a list class within in the GameObject?!
  • add --datadir DIR (data/) and --userdir DIR (~/.supertux/), allow multiple --datadir's
  • make gravity constant-> To be discussed…
  • rename Vector -> Vector2f
  • get rid of global SDL_Screen* screen variable -> in progress
  • get rid of SCREEN_WIDTH/SCREEN_HEIGHT overuse, give them a proper name at least -> to think when upgrading to SDL2
  • resolution menu entry moves the wrong way around
  • having dictionary_manager in Lisp is extremely ugly
  • enforce proper naming of files to match their class (SomeClass -> some_class.?pp or so)
  • file naming is inconsistent: some times we use '_' to separate words, sometimes we don't
  • collect all manager classes into globals.hpp
  • implement PNG screenshot
  • having hitbox in Sprite is fugly
  • write decal object that doesn't have hitbox
  • implement surface and/or sprite scaling (MipMaps?)
  • add code that compares the last Log line with the current, if they are the same reject them and just output something like:
 ** last line has been repeated X times

$ cat ~/.alsoftrc drivers = oss

  • peaking up/down doesn't work properly
  • peaking left/right should make Tux look into that direction (up/down to, needs new sprites)
  • keep possible future SDL1.3 upgrade in mind (what has changed?) -> in progress
  • cleanup scripting interface
  • replace cloud tiles with decals
  • option menu has text overlap in "aspect ratio"
  • jumping up from an enemy doesn't make a sound?
  • add support for automatic scrolling backgrounds
  • add direct reading of Vector2f to Reader/lisp
  • replace bell with 'reset block', that starts to glow once bumped into (or something different)
  • refactor Camera code, break ugly long functions into pieces and such
  • allow fully custom magnification levels from command line (maybe GUI to if there is a proper/easy way to let the user enter numbers) (--magnification or -g WIDTHxHEIGHT:ASPECTX:ASPECTY@MAGNIFICATION)
  • use AnchorPoint in Background instead of Alignment
  • allow gradients to parallax scroll like Background (make it optional)
  • add multicolored gradients (see Windstille source code, which can deal with Gimp gradients)
  • fix alpha blending in the SDL renderer, currently all sprites (Tux, etc.) appear transparent
  • position of statistics text on the worldmap doesn't scale properly with resolution it seems
  • gluBuild2DMipmaps leads to blurriness, maybe lack of magic offset (0.375f, 0.375f, 0.0f) or something else
  • font system has blending artifacts at non-1x magnifications, need to have an transparent pixel between letters
  • add a (border #t) flag to fonts, that allows to use fonts which have a 1px transparent border around glyphs, which is needed to get rid of blending artifacts in OpenGL
  • mouse cursor has blend artifact on high magnification
  • shadow font glyphs bleed into other glyphs
  • in DrawingRequest "void* request_data;" is only free'ed, but the destructor never gets called
  • sprite/sprite.cpp: frame should never get out of range:
 if((int)frame >= get_frames() || (int)frame < 0)
   log_warning << "frame out of range: " << (int)frame << "/" << get_frames() << " at " << get_name() << "/" << get_action() << std::endl;
  • Surface::hflip() is used exactly once, should probally be part of the constructor

Scenegraph and Physics Engine Restructuring

  • random idea to restructure engine stuff (might lead to nicer code and easier scriptability (and a need to rewrite lots of stuff...):

class SomeBadGuy : public PhysicsCallbackListener // or use boost::function { private:

     PhysicsPtr box;
     SpritePtr sprite;
       

public:

     SomeBadGuy(Engine& engine)  
     {
        box    = engine.physics().create_box(Rectf(0,0,32,32));
        box->register_listener(this);
        sprite = engine.graphics().create_and_add_sprite("Foobar");
     }
     void update(float delta)
     {
        // not much to do, as most stuff is done internally in the engine
        if (dead)
        {
                 sprite->replace_with("Foobar_dead");
        }
        else
        {
               sprite->hide();
               sprite->set_pos(box->get_pos());
       }
     }
     // no more draw(), done by the scene graph
     void on_collision(CollisionData data)
     {
       // respond
     }

};

Random Notes

  • calculate the size of an background image that should fill the screen:
 image_size = (1 - parallax_speed) * screen_size + level_size * parallax_speed

def calc(parallax, screen, tiles):

   return (1 - parallax) * screen + parallax * tiles * 32


Supported Resolutions

SuperTux shall support resolutions from 640x480 to 1280x800 at a magnification of 1x. For resolutions higher, such as 2560x1600, upscaling will be used. For resolutions smaller, like 320x240 downscaling will be used.

You expect people play testing the levels in more than one resolution? How many between 640x480 to 1280x800 should I try exactly?

Higher resolution graphics for 2x maginification might be provided. Lower res graphics for 0.5x maginification might be provided as well.

Resolution and magnification can be freely configured by the user within the given limits.

In tiles this means we have 40x25 (=1280x800px) tiles per screen.

Graphic Tasks

  • animate cave background torch: data/images/tiles/background/backgroundtile3.png
  • do parallax tiles (done, I think)
  • remove lightmap tiles
Why?
  • redraw/replace data/images/tiles/waterfall/
  • draw/redraw/replace for a ghostly word.


Whats the point in adding new features while the existing features and badguys can't be used due to lack of images or animations?

As long as all these points haven't been fixed, Milestone 2 will not be considered as Stable, so won't be released.