SCM: Automate formatting of references

[10110111]

Description

Now users will edit an actual list of references: a QTreeWidget, not having to deal with the Markdown syntax we use. This way we'll hopefully get a higher rate of syntactically correct SCs in new PRs.

Type of change

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)
• This change requires a documentation update
• Housekeeping

How Has This Been Tested?

Test Configuration:

• Operating system: Ubuntu 20.04

Checklist:

• My code follows the code style of this project.
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation (header file)
• I have updated the respective chapter in the Stellarium User Guide
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes
• Any dependent changes have been merged and published in downstream modules

[xLPMG]

My changes generate no new warnings

Hm on my system they do: (although very minor, but the goal should be no warnings :) )

SkyCultureMaker/src/gui/ScmSkyCultureDialog.cpp:40:33: warning: unused parameter 'parent' [-Wunused-parameter]
   40 |         QWidget* createEditor(QWidget* parent, const QStyleOptionViewItem& option, const QModelIndex& index) const 
SkyCultureMaker/src/gui/ScmSkyCultureDialog.cpp:40:69: warning: unused parameter 'option' [-Wunused-parameter]
   40 |         QWidget* createEditor(QWidget* parent, const QStyleOptionViewItem& option, const QModelIndex& index) const 
SkyCultureMaker/src/gui/ScmSkyCultureDialog.cpp:40:96: warning: unused parameter 'index' [-Wunused-parameter]
   40 |         QWidget* createEditor(QWidget* parent, const QStyleOptionViewItem& option, const QModelIndex& index) const

I'm not sure if this is a good practice, but one way to get rid of this is to not name the parameters:

QWidget* createEditor(QWidget*, const QStyleOptionViewItem&, const QModelIndex&) const override

[gzotti]

You can use Q_UNUSED(param_not_used_elsewhere_in_body) to make it clearer.

[10110111]

Hm on my system they do

Fixed in f180b2a.

[github-actions]

Hello @10110111!

Please check the fresh version (development snapshot) of Stellarium:
https://github.com/Stellarium/stellarium-data/releases/tag/weekly-snapshot

[github-actions]

Hello @10110111!

Please check the latest stable version of Stellarium:
https://github.com/Stellarium/stellarium/releases/latest