[Psi-devel] a patch for auto-replace
mblsha-psi at affinix.com
Tue Feb 8 22:49:02 PST 2005
> >1. Why not to store the replace list in the config.xml? It would be alot
> >simplier that way. Store QMap<QString, QString> in Options structure, have
> >one AutoCorrect class inside PsiCon. Update the correction list on options
> >change. As only one user profile is active at a time, you could store
> >AutoCorrect in the PsiCon::Private class.
> What PsiCon stands for?
I don't really know, but it's the name of global Psi object, that exists when
Psi is working, and is used to combine different parts of the program
together. So I think, you'd better put common stuff there.
> Why hold a qmap and autocorrect - i.e. hold the
> data twice?
If you will just copy the QMap, I think the implicit data sharing will come
into play, and you'll have only once instance of QMap in the memory, so no
> Can i somehow get from PsiCon to options?
You need to change the API to receive PsiCon in the options. I think we
shoudn't do it yet. And, as additional note, have a look how different
OptionsDlg panes work. Storing strings in the Options structure would
actually be alot easier when you'll come to implementing GUI to configure
> About separate config file - r u sure you want to put it inside config.xml?
We already have too much files lying around. Do we really need more of them?
> Besides i'll probably have to read much more about signals and slots :).
There's a good overview in Qt Documentation, and you also could watch a video,
that's available from Trolltech's site.
More information about the Psi-devel-affinix.com