#19297 closed Cleanup/optimization (wontfix)

refactor settings setup code

Reported by: ptone Owned by: nobody
Component: Core (Other) Version: master
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no


There is an increasing amount of code duplication in conf.init.LazySettings between _setup and configure

There are some differences in the way the actual settings holder is setup - but there should be a way to refactor this so that _setup calls configure, and put the common code there.

Attachments (0)

Change History (1)

comment:1 Changed 17 months ago by ptone

  • Resolution set to wontfix
  • Status changed from new to closed

So after poking around this area I've concluded two things:

1) the savings gained aren't worth the gymnastics of a refactor here
2) configure really is doing a different enough thing to stay different

Add Comment

Modify Ticket

Change Properties
<Author field>
as closed
as The resolution will be set. Next status will be 'closed'
The resolution will be deleted. Next status will be 'new'

E-mail address and user name can be saved in the Preferences.

Note: See TracTickets for help on using tickets.