[Kstars-devel] TODO and FIXME for 4.3

Médéric Boquien mboquien at free.fr
Tue May 19 05:25:45 CEST 2009


Hi,

As we are in the freeze we can focus on fixing bugs. I have grepped TODO and 
FIXME in kstars' code to see what could be done beside the bugs already in the 
bug tracker.

Regards,

Médéric

kstars/skycomponents/customcatalogcomponent.h:28://JH: TODO: this class should 
only contain one custom catalog.
kstars/skycomponents/starblocklist.cpp:67:    // TODO: Remove staticity of 
BinFileHelper                       
kstars/skycomponents/starblocklist.cpp:123:     // TODO: Make this more 
general                                
kstars/skycomponents/deepstarcomponent.h:108:    // TODO: Find the right place 
for this method                 
kstars/skycomponents/starblockfactory.cpp:20:// TODO: Remove later                                             
kstars/skycomponents/starblockfactory.cpp:24:// TODO: Implement a better way 
of deciding this                  
kstars/skycomponents/starblockfactory.cpp:256:    Trixel curTrixel = 513; // 
TODO: Change if we change HTMesh level
kstars/skycomponents/starcomponent.cpp:428:    // TODO: Maybe we don't want to 
hardcode the filename?              
kstars/skycomponents/starcomponent.cpp:629:      // TODO: Should we multiply 
rBest by a factor < 1, so that we give higher priority to named stars?
kstars/skycomponents/starcomponent.cpp:715:            // NOTE: Assumes 2 
decimal places in magnitude field. TODO: Change if it ever does change   
kstars/skycomponents/starblock.cpp:30:    allocStars( 100 );                // 
TODO: Make this number user-specifiable                             
kstars/skycomponents/solarsystemsinglecomponent.cpp:107:    //TODO: default 
values for 2nd & 3rd arg. of SkyMap::checkVisibility()                 
kstars/skycomponents/solarsystemsinglecomponent.cpp:117:    //TODO: 
KSPlanetBase needs a color property, and someone has to set the planet's color 
kstars/skycomponents/solarsystemsinglecomponent.cpp:140:        StarObject 
fakeStar( 0, 0, 0, QString(), QString(), SpecType, 0.0, 0.0, 0.0, false, 
false, 0); // TODO: Choose the spectral type based on the colour                                                                                                                                                                                                        
kstars/skycomponents/skycomponent.h:45: *((TODO_DOX: Add outline of Components 
hierarchy))                                                                                                                    
kstars/skycomponents/skymapcomposite.cpp:251:    // TODO: Fix satellites by 
KDE 4.3                                                                                                                           
kstars/skycomponents/skymapcomposite.cpp:301:    // TODO: Add support for deep 
star catalogs                                                                                                                  
kstars/skycomponents/skymapcomposite.cpp:351:    // TODO: Add Deep Star 
Catalog support                                                                                                                       
kstars/skycomponents/deepstarcomponent.cpp:81:    // TODO: Read the 
multiplying factor from the dataFile                                                                                                      
kstars/skycomponents/deepstarcomponent.cpp:155:    // TODO: Work out the 
details                                                                                                                              
kstars/skycomponents/deepstarcomponent.cpp:170:// TODO: Optimize draw, if it 
is worth it.                                                                                                                     
kstars/skycomponents/deepstarcomponent.cpp:195:    // TODO: Enable hiding of 
faint stars                                                                                                                      
kstars/skycomponents/deepstarcomponent.cpp:259:        // TODO: Is there a 
better way? We may have to change the magnitude tolerance if the catalog 
changes                                                   
kstars/skycomponents/deepstarcomponent.cpp:367:    return 
m_CatalogNumber.value( HDnum, NULL ); // TODO: Maybe, make this more general.                                                                       
kstars/skycomponents/deepstarcomponent.cpp:442:            // NOTE: Assumes 2 
decimal places in magnitude field. TODO: Change if it ever does change                                                          
kstars/skyobjects/skypoint.h:469:    //TODO: move rise/set/transit stuff from 
SkyObject                                                                                                                       
kstars/skyobjects/ksplanet.h:70:     *TODO: Change the whole architecture for 
names later                                                                                                                     
kstars/skyobjects/ksplanetbase.cpp:315:       TODO: Switch to this if and when 
we make KSSun a singleton */                                                                                                   
kstars/skyobjects/ksplanet.cpp:150:// TODO: Get rid of this dirty hack post 
KDE 4.2 release                                                                                                                   
kstars/skyobjects/saturnmoons.h:35:  *TODO: make the moons SkyObjects, rather 
than just points.                                                                                                               
kstars/skyobjects/ksmoon.cpp:212:    KSSun *Sun = 
(KSSun*)KStarsData::Instance()->skyComposite()->findByName( "Sun" ); // TODO: 
Get rid of this ugly thing by making KSSun singleton.                         
kstars/skyobjects/jupitermoons.h:35:  *TODO: make the moons SkyObjects, rather 
than just points.                                                                                                              
kstars/skyobjects/skyobject.cpp:378://TODO: Should create a special UserLog 
widget that encapsulates the "default"                                                                                            
kstars/data/tools/binfiletester.c:173:            if( mag != -500 && ( ( 
data.mag - mag ) > 20 && mag < 1250 ) || data.mag < mag ) { // TODO: Make 
sensible magnitude limit (1250) user specifiable           
kstars/data/tools/binfiletester.c:174:                // TODO: Enable 
byteswapping                                                                                                                            
kstars/data/tools/mysql2bin.c:29:#define NTRIXELS 512                         
// TODO: Change if HTM Level Changes                                                                                            
kstars/data/tools/mysql2bin.c:320:    nsf_trix_begin = usf_trix_begin = 
dsf_trix_begin = 2 + 1 + 2; // The 2+1+2 is to leave space for deep magnitude 
limit, HTM Level and MSpT specification. TODO: Change this if things change.                                                                                                                                                                                          
kstars/data/tools/nomadmysql2bin-merge.c:25:#define NTRIXELS 32768                         
// TODO: Change if HTM Level Changes                                                                               
kstars/data/tools/nomadmysql2bin-merge.c:167:    usf_trix_begin = 2 + 1 + 2; 
// The 2 + 1 + 2 is to leave space for deep magnitude limit, HTM Level and 
MSpT specification. TODO: Change this if things change.
kstars/data/tools/tycdatatomysql.pl:66:my $level = 3;      # Note: If you 
change this, change at the TODO below as well.
kstars/data/tools/tycdatatomysql.pl:183:        $star->{Trixel} = $tid - 512; 
# TODO : Change if you change HTMesh level.
kstars/data/tools/Makefile:58:  cat ushf usdf > USNO-NOMAD.dat    # TODO: 
Change
kstars/data/tools/nomaddatatomysql.pl:176:        $star->{Trixel} = $tid - 
32768; # TODO: Change magic number if HTM level changes
kstars/data/tools/nomadmysql2bin.c:27:#define NTRIXELS 32768                         
// TODO: Change if HTM Level Changes
kstars/data/tools/nomadmysql2bin.c:195:    usf_trix_begin = 2 + 1 + 2; // The 
2 + 1 + 2 is to leave space for deep magnitude limit, HTM Level and MSpT 
specification. TODO: Change this if things change.
kstars/data/tools/parse-sac-to-mysql.pl:114:             ## NOTE: Ignoring 
fields NSTS, BRSTR and BCHM; TODO: Include them later
kstars/data/tools/nomadmysql2bin-split.c:30:#define NTRIXELS 32768                         
// TODO: Change if HTM Level Changes
kstars/data/tools/nomadbinfiletester.c:174:            if( mag != -5000 && ( ( 
new_mag - mag ) > 20 && mag < 1250 ) || new_mag < mag ) { // TODO: Make 
sensible magnitude limit (1250) user specifiable
kstars/data/tools/nomadbinfiletester.c:175:                // TODO: Enable 
byteswapping
kstars/kstars.h:335:    // TODO INDI Scripting to be supported in KDE 4.1
kstars/skymap.cpp:1525:    //TODO deal with alternate projections
kstars/indi/streamwg.cpp:229:    // TODO This is highly inefficient. Need to be 
replaced with a direct blit.
kstars/indi/indistd.cpp:965:        //TODO add text in the status bar "Slew 
aborted."
kstars/indi/indidriver.h:140:    //TODO void shutdownHost(int managersID);
kstars/fitsviewer/fitshistogram.cpp:233:    // TODO apply maximum compression 
against this buffer
kstars/fitsviewer/fitshistogram.cpp:260:     // TODO how to remove this item 
from redo/undo history?
kstars/fitsviewer/fitsimage.h:106:    /* TODO Make this stat PRIVATE
kstars/kstarsactions.cpp:317:   // TODO JM: redesign after KDE 4.2 is 
released.
kstars/ksnumbers.cpp:324:    // TODO Pasar a grados
kstars/kstarsdata.h:159:    //TODO JM: ADV tree should use XML instead
kstars/kstarsdata.h:173:    //TODO JM: Use XML instead; The logger should have 
more features
kstars/widgets/fovwidget.cpp:48:            // TODO: Check if decimal points 
in this are localized (eg: It should read 1,5 x 1,5 in German rather than 1.5 
x 1.5)
kstars/quaternion.h:77:    // TODO: Better add accessors...
kstars/dms.h:233:    //TODO: Either implement these operators, or delete the 
commented declarations
kstars/dialogs/detaildialog.h:115:    //TODO: showThumbnail() is only called 
in the ctor; make it private and not a slot.
kstars/dialogs/detaildialog.cpp:184:        else if ( ps->name() == 
i18n("Pluto") || ps->name() == "Ceres" || ps->name() == "Eris" ) // TODO: 
Check if Ceres / Eris have translations and i18n() them
kstars/dialogs/finddialog.cpp:280:    // TODO after KDE 4.1 release:
kstars/Messages.sh:40:# TODO: Move this thing to starnames.dat
kstars/skymapevents.cpp:47:// TODO: Remove if debug key binding is removed
kstars/kstarsdcop.cpp:538:// TODO JM: INDI Scripting to be supported in KDE 
4.1
kstars/kstarsinit.cpp:398:    // TODO redesign after KDE 4.2 is released
kstars/tools/conjunctions.cpp:67:    KStarsDateTime dtStop ( dtStart.djd() + 
365.24 ); // TODO: Refine
kstars/tools/conjunctions.cpp:150:                Object1 = 
KSPlanetBase::createPlanet( pNames.key( fd->selectedObject()->name() ) ); // 
TODO: Fix i18n issues.
kstars/tools/observinglist.h:60:        *TODO:
kstars/tools/scriptbuilder.cpp:174:    //TODO JM: INDI Scripting to be 
supported in KDE 4.1
kstars/tools/scriptbuilder.cpp:881:    // TODO: We have disabled the following 
two features. Enable them when feasible...
kstars/tools/scriptbuilder.cpp:1179:            // TODO INDI scripting to be 
supported under KDE 4.1
kstars/tools/scriptbuilder.cpp:1694:        //TODO JM: INDI Scripting to be 
included in KDE 4.1
kstars/tools/scriptbuilder.cpp:1984://TODO JM: INDI Scripting to be included 
in KDE 4.1
kstars/tools/scriptbuilder.cpp:2418://TODO JM: INDI Scripting to be included 
in KDE 4.1
kstars/tools/skycalendar.cpp:95:    int textFlags = Qt::AlignCenter; // TODO: 
See if Qt::SingleLine flag works better
kstars/tools/modcalceclipticcoords.cpp:231:    
fOut.open(QIODevice::WriteOnly); // TODO error checking


kstars/options/opscatalog.cpp:215:    //FIXME: need to add the ShowDeepSky 
meta-option to the config dialog!
kstars/skycomponents/saturnmoonscomponent.cpp:232:                //FIXME: 
map->rect() should return QRectF 
kstars/skycomponents/deepskycomponent.cpp:334:    //FIXME                                                   
kstars/skycomponents/constellationboundarylines.cpp:50:// FIXME: -jbb: need to 
update this information.     
kstars/skycomponents/solarsystemcomposite.cpp:131:    //FIXME: first draw the 
objects which are far away    
kstars/skycomponents/flagcomponent.h:150:     *FIXME: This should probably move 
to KStarsDateTime           
kstars/skycomponents/flagcomponent.h:159:     *FIXME: This should probably move 
to KStarsDateTime           
kstars/skycomponents/solarsystemsinglecomponent.cpp:149:             //FIXME:                                   
int(Options::zoomFactor()) >= int(zoommin) &&  //zoomed in enough,
kstars/skycomponents/solarsystemsinglecomponent.cpp:168:            //FIXME: 
Need to figure out why the size is sometimes NaN                                                     
kstars/skycomponents/solarsystemsinglecomponent.cpp:176:            //FIXME: 
resize_mult ??                                                                                       
kstars/skycomponents/solarsystemsinglecomponent.cpp:235:            //FIXME: 
map->rect() should return QRectF
kstars/skycomponents/jupitermoonscomponent.cpp:232:                //FIXME: 
map->rect() should return QRectF
kstars/skycomponents/deepskycomponent.h:89:     *@short Update the sky 
positions of this component.  FIXME -jbb does nothing now
kstars/kspopupmenu.cpp:52:      // FIXME: make logic less convoluted.
kstars/kspopupmenu.cpp:111:                     // FIXME: this should be some 
catalog name too
kstars/kspopupmenu.cpp:128:     // FIXME: information about angular sizes 
should be added.
kstars/kspopupmenu.cpp:153:             // FIXME: angular size is required.
kstars/kspopupmenu.cpp:249:    //FIXME: add key shortcut to menu items 
properly!
kstars/skyobjects/skypoint.cpp:97:    //        //FIXME: eventually, we will 
work with XYZ directly
kstars/skyobjects/skypoint.cpp:146:    tanDec = sinDec/cosDec;                    
// FIXME: -jbb div by zero?
kstars/skyobjects/skyobject.cpp:453:    //FIXME: we shouldn't be changing the 
font back and forth for every object!
kstars/skyobjects/skyobject.cpp:482:    //FIXME: Implement label background 
options
kstars/data/info_url.dat:229:## --==>> FIXME: Gaspra is missing from 
asteroids.dat!
kstars/thumbnailpicker.cpp:138:    uint pad = 0; /*FIXME LATER 4* 
KDialogBase::marginHint() + 2*ui->SearchLabel->height() + 
KDialogBase::actionButton( KDialogBase::Ok )->height() + 25;*/
kstars/thumbnailpicker.cpp:306:            uint pad = 0;/* FIXME later 
4*marginHint() + 2*ui->SearchLabel->height() + actionButton( Ok )->height() + 
25; */
kstars/kstars.h:733:    //FIXME: move to KStarsData
kstars/skymap.cpp:961:            if ( scaledRect().contains( p.toPoint() ) )  
//FIXME -jbb
kstars/indi/indiproperty.cpp:166:                // FIXME is this 
problematic??
kstars/indi/indidevice.cpp:313:            // FIXME for WO properties, only 
min/max needs to be updated
kstars/indi/telescopewizardprocess.cpp:70:    // FIXME is there a better way 
to check if a translated message is empty?
kstars/indi/telescopewizardprocess.cpp:359:     //FIXME this causes crash
kstars/indi/indidriver.cpp:419:    /* FIXME The following seems to cause a 
crash in KStars when we use
kstars/indi/telescopeprop.cpp:87:    /* FIXME This call causes KStars to crash 
under Qt 4.1
kstars/indi/indistd.cpp:217:    // FIXME It appears that FITSViewer causes a 
possible stack corruption, needs to investigate
kstars/indi/indielement.cpp:324:    /* FIXME make sure the size stuff work */
kstars/kstarsdata.cpp:125:    //FIXME: Do we still need this?
kstars/kstarsdata.cpp:128:    //FIXME: Verify list of deletes
kstars/kstarsdata.h:87:    //FIXME: can we avoid having so many friend 
classes?
kstars/kstarsdata.h:223:    //FIXME: can we live without this, or do we need a 
SkyMapComposite fcn?
kstars/widgets/thumbimage.cpp:30:    //FIXME: Deprecated?  Maybe we don't need 
this, since double-buffering is now built in
kstars/infobox.cpp:113:        //FIXME...
kstars/dialogs/focusdialog.h:67:        *FIXME: This should probably move to 
KStarsDateTime
kstars/dialogs/focusdialog.h:76:        *FIXME: This should probably move to 
KStarsDateTime
kstars/dialogs/finddialog.h:77:    //FIXME: Still valid for KDialog?  i.e., 
does KDialog have a slotOk() ?
kstars/skymapevents.cpp:55:    //FIXME: No equivalent for this line in Qt4 ??
kstars/skymapevents.cpp:809:    //FIXME: What to do about the visibility 
logic?
kstars/kstarsdcop.cpp:808://FIXME: DBUS: needs to be reimplemented without 
suspend/resume
kstars/kstarsinit.cpp:64://FIXME: New window disabled
kstars/kstarsinit.cpp:365://FIXME: implement glossary
kstars/tools/astrocalc.cpp:90:    //FIXME: Would be better to make the 
navigationPanel fit its contents,
kstars/tools/altvstime.cpp:84:    //FIXME:
kstars/tools/scriptbuilder.cpp:891:    //FIXME: This description is incorrect! 
Fix after strings freeze
kstars/tools/scriptbuilder.cpp:901:    //FIXME: This description is incorrect! 
Fix after strings freeze
kstars/tools/scriptbuilder.cpp:1156:    // FIXME Without --print-reply, the 
dbus-send doesn't do anything, why??
kstars/tools/skycalendar.cpp:89:// FIXME: For the time being, adjust with 
dirty, cluttering labels that don't align to the line
kstars/tools/observinglist.cpp:501://FIXME: This will open multiple Detail 
windows for each object;
kstars/tools/observinglist.cpp:528://FIXME: On close, we will need to close 
any open Details/AVT windows
kstars/skymapdraw.cpp:366://FIXME: Implement Options::boxBGMode()
kstars/htmesh/HTMesh.h:9:// FIXME? We are needlessly copying the trixel data 
into the buffer during
kstars/quaternion.cpp:183:              "movss %%xmm1, %%xmm4 \n\t" // FIXME: 
movaps -> movss (-1 Cylce)



More information about the Kstars-devel mailing list