Code review comment for lp:~daggerstab/stellarium/comets-asteroids-importer

Hi

I compiled 4786 last night in Widows (4782 was OK). I causes a major lock up. It reaches the stellarium display and
then stops nothing but a total shut down unlocks the computer. As a restart then takes about 7 minutes I have not got very far but I suspect it may be in the config.ini or ssystem.ini files. The log.txt gives no indication

Barry Gerdes
Beaumont Hills Observatory
S 33' 41' 44" E 150' 56' 32"

> To: <email address hidden>
> From: <email address hidden>
> Subject: [Merge] lp:~daggerstab/stellarium/comets-asteroids-importer into lp:stellarium
> Date: Fri, 12 Nov 2010 17:23:10 +0000
>
> Bogdan Marinov has proposed merging lp:~daggerstab/stellarium/comets-asteroids-importer into lp:stellarium.
>
> Requested reviews:
> Stellarium (stellarium)
>
>
> This is the first version (0.0.1) of the Solar System Editor plug-in (formerly the Comets and Asteroids Importer plug-in).
>
> It's not a complete Solar System editor yet - the window that would allow manual editing of all parameters of Solar System objects is not finished yet, so it is hidden in this release. You can open manualImportWindow.ui in QtDesigner for a preview of what it will look like.
>
> In addition to the plug-in itself, the branch contains changes to Stellarium's core (SolarSystem + 2 new classes derived from Planet) that allow better handling of asteroids and comets, including name rendering and visual magnitude calculation.
>
> Feel free to comment on the interface and the texts used in it. :)
> --
> https://code.launchpad.net/~daggerstab/stellarium/comets-asteroids-importer/+merge/40744
> Your team Stellarium is requested to review the proposed merge of lp:~daggerstab/stellarium/comets-asteroids-importer into lp:stellarium.