[bt-devel] bug

Martin Gruner bt-devel@crosswire.org
Tue, 26 Sep 2000 15:46:01 +0200


Hi Joachim,

how was your exam?

Yes, the backend needs some cleaning and IMO some simplification.
Let us talk about it when you're around.

My first thoughts: simplify the CModuleInfo Methods, maybe introduce a 
CModuleInfoConfig object to separate the config info from the text related 
functions.

Delete CKeyChain and ListCkeyChain, they are not really necessary.
Their place would be filled by 3 methods in CModuleInfo:

QStringList getBookList()
->chaches the result
->opening of lexicon presenters is speeded up

int getChapters(QString book)
->will compute the result when needed (no caching)
->links to a sword function

int getVerses(QString book, int chapter)
->see above

Martin

On Mon, 25 Sep 2000, you wrote:
> Hi Martin!
>
> This remebers me that we have to clean and restructure the backend.
> Sure, Torsten did it right, but we have to discuss which functions we need
> and we should realize it (at least parts) module independant.
>
> At the moment things are looked up in the module etc., sometimes it's
> possible to use only  a VerseKey or SWKey object.
>
> But now I have to learn for Math, we have our exam tomorrow.
> I'll have a look at it tomorrow, OK?
>
> --Joachim
>
> > Hey,
> >
> > how can I set the current key of a module? I use now lookupText(), but
> > that is bad.
> >
> > I commented out 2 completely buggy functions in CSWORDBibleModuleInfo:
> > getNextKey and getPrevKey. Why should the bibles use different methods
> > than commentaries?
> > These functions in CSWORDCommentaryModuleInfo do work, only not for
> > books. The getprevkey() in the lexicon seems to be buggy, too (goes to
> > key 00000, although this key doesn't exist).
> > Joachim, could you help me?
> >
> > You can test it with the keychoosers in the Searchdialog (only for
> > testing).
> >
> >
> > Martin