[texworks] now r596 breaks compilation on Xubuntu 8.10 w. Qt 4.4.3

[Jonathan Kew]

On 5 Apr 2010, at 15:05, Stefan Löffler wrote:
> 
> I just noticed that QMap<QString, QVariant> is supported, which for our
> purposes should be equivalent.

Yes, I just changed the code. :)

> 
>> Yes, I considered creating a struct, and maybe this would be better in some ways. For now, after a bit of experimentation I felt that using a hash provided most flexibility for least effort, as source and PDF windows need different fields, and keys can be added or omitted without breaking compatibility or requiring us to maintain a "version" field, etc. But I'd be willing to consider a different approach; this is definitely experimental at the moment.
>> 
> 
> Yeah, compatibility would indeed be an issue with structs. But in what
> way do TeX and PDF windows need different fields? So far, I could only
> think of windows filename, window geometry, and possibly position (could
> be a line/page number and/or scrollbar values). Well, I think I'll have
> to have a look at the changes some time.

For editor windows, it might be nice to record the current settings for auto-indent, smart quotes, line numbers, syntax coloring, line wrap, etc.; for PDF windows, the zoom level, page number, scroll values. And the state of toolbars and docks is a different-sized chunk of data for each. The "bag of properties" model makes it easy to add these things at any point.

JK