On Mon, Apr 07, 2008 at 01:12:11AM +0200, Joost Verburg wrote:
> Leuven, E. wrote:
> [...]
> This is really a great feature. I can now copy graphics from any Windows
> application!
>
> Joost
Cool stuff.
Nevertheless, could we try stick to LyX style also in Windows-only code?
E.g:
> +#ifdef Q_WS_WIN
> +
> +static FORMATETC setCf(int cf)
> +{
> + FORMATETC formatetc;
Tabs for indentation.
> + formatetc.cfFormat = cf;
> + formatetc.ptd = NULL;
> + formatetc.dwAspect = DVASPECT_CONTENT;
> + formatetc.lindex = -1;
> + if (cf == CF_ENHMETAFILE) formatetc.tymed = TYMED_ENHMF;
> + if (cf == CF_METAFILEPICT) formatetc.tymed = TYMED_MFPICT;
if (cf == CF_ENHMETAFILE)
formatetc.tymed = TYMED_ENHMF;
> + if (cf == CF_METAFILEPICT) formatetc.tymed = TYMED_MFPICT;
> + return formatetc;
> +}
> +
> +
> +static bool canGetData(int cf, IDataObject * pDataObj)
> +{
> + FORMATETC formatetc = setCf(cf);
> + return pDataObj->QueryGetData(&formatetc) == S_OK;
> +}
> +
> +
> +class QWindowsMimeMetafile : public QWindowsMime {
> +public:
> + QWindowsMimeMetafile()
> + : QWindowsMime()
> + {}
> + ~QWindowsMimeMetafile() {}
[Empty desctructors and constructors are not needed]
> + bool canConvertFromMime(const FORMATETC & formatetc, const QMimeData *
> mimeData) const;
> + bool canConvertToMime(const QString & mimeType, IDataObject * pDataObj)
> const;
> + bool convertFromMime(const FORMATETC & formatetc, const QMimeData *
> mimeData, STGMEDIUM * pmedium) const;
> + QVariant convertToMime(const QString & mimeType, IDataObject *
> pDataObj, QVariant::Type preferredType) const;
> + QVector<FORMATETC> formatsForMime(const QString & mimeType, const
> QMimeData * mimeData) const;
> + QString mimeForFormat(const FORMATETC &) const;
> +};
We have 'Type const & t';
> +
> +QVariant QWindowsMimeMetafile::convertToMime(
> + const QString & mimeType, IDataObject * pDataObj,
> + QVariant::Type preferredType) const
> +{
> + QVariant ret;
> +
> + if (canConvertToMime(mimeType, pDataObj)) {
Tab again...
Andre'