[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-commits
Subject: extragear/graphics/gwenview
From: Angelo Naselli <anaselli () linux ! it>
Date: 2007-03-31 13:34:47
Message-ID: 1175348087.849199.29255.nullmailer () svn ! kde ! org
[Download RAW message or body]
SVN commit 648473 by anaselli:
Fixed code style
CCMAIL:gwenview-general@lists.sourceforge.net
M +2 -2 gvcore/imageloader.cpp
M +12 -31 imageutils/jpegcontent.cpp
M +3 -2 imageutils/jpegcontent.h
--- trunk/extragear/graphics/gwenview/gvcore/imageloader.cpp #648472:648473
@@ -673,8 +673,8 @@
QSize size = content.size();
d->mProcessedImage = QImage(size, d->mDecoder.image().depth());
}
- d->mProcessedImage.setDotsPerMeterX(content.getDotsPerMeterX());
- d->mProcessedImage.setDotsPerMeterY(content.getDotsPerMeterY());
+ d->mProcessedImage.setDotsPerMeterX(content.dotsPerMeterX());
+ d->mProcessedImage.setDotsPerMeterY(content.dotsPerMeterY());
} else {
kdWarning() << "ImageLoader::changed(): JPEGContent could not load '" << \
d->mURL.prettyURL() << "'\n"; }
--- trunk/extragear/graphics/gwenview/imageutils/jpegcontent.cpp #648472:648473
@@ -290,45 +290,26 @@
}
-int JPEGContent::getDotsPerMeterX() const {
- Exiv2::ExifKey keyResUnit("Exif.Image.ResolutionUnit");
- Exiv2::ExifData::iterator it = d->mExifData.findKey(keyResUnit);
- if (it == d->mExifData.end()) {
- return 0;
- }
- int XRes = it->toLong();
- Exiv2::ExifKey keyXResolution("Exif.Image.XResolution");
- it = d->mExifData.findKey(keyXResolution);
- if (it == d->mExifData.end()) {
- return 0;
- }
- // The unit for measuring XResolution and YResolution. The same unit is used for \
both XResolution and YResolution.
- // If the image resolution in unknown, 2 (inches) is designated.
- // Default = 2
- // 2 = inches
- // 3 = centimeters
- // Other = reserved
- const float INCHESPERMETER = (100. / 2.54);
- switch (XRes) {
- case 3: // dots per cm
- return (it->toLong() * 100);
- default: // dots per inch
- return (it->toLong() * INCHESPERMETER);
- }
+int JPEGContent::dotsPerMeterX() const {
+ return dotsPerMeter("XResolution");
+}
- return 0;
+
+int JPEGContent::dotsPerMeterY() const {
+ return dotsPerMeter("YResolution");
}
-int JPEGContent::getDotsPerMeterY() const {
+int JPEGContent::dotsPerMeter(const QString& keyName) const {
Exiv2::ExifKey keyResUnit("Exif.Image.ResolutionUnit");
Exiv2::ExifData::iterator it = d->mExifData.findKey(keyResUnit);
if (it == d->mExifData.end()) {
return 0;
}
- int YRes = it->toLong();
- Exiv2::ExifKey keyYResolution("Exif.Image.YResolution");
- it = d->mExifData.findKey(keyYResolution);
+ int res = it->toLong();
+ QString keyVal = "Exif.Image." + keyName;
+ Exiv2::ExifKey keyResolution(keyVal.ascii());
+ it = d->mExifData.findKey(keyResolution);
if (it == d->mExifData.end()) {
return 0;
}
@@ -339,7 +320,7 @@
// 3 = centimeters
// Other = reserved
const float INCHESPERMETER = (100. / 2.54);
- switch (YRes) {
+ switch (res) {
case 3: // dots per cm
return (it->toLong() * 100);
default: // dots per inch
--- trunk/extragear/graphics/gwenview/imageutils/jpegcontent.h #648472:648473
@@ -44,8 +44,8 @@
Orientation orientation() const;
void resetOrientation();
- int getDotsPerMeterX() const;
- int getDotsPerMeterY() const;
+ int dotsPerMeterX() const;
+ int dotsPerMeterY() const;
QSize size() const;
@@ -69,6 +69,7 @@
JPEGContent(const JPEGContent&);
void operator=(const JPEGContent&);
void applyPendingTransformation();
+ int dotsPerMeter(const QString& keyName) const;
};
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic