Re: [xwiki-devs] [xwiki-notifications] r33309 - platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/plugin/image
Hi Sergiu, On 12/08/2010 02:50 PM, sdumitriu (SVN) wrote:
Author: sdumitriu Date: 2010-12-08 13:50:55 +0100 (Wed, 08 Dec 2010) New Revision: 33309
Modified: platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/plugin/image/ImagePlugin.java Log:
[cleanup] Applied codestyle
Did we set any code style rule regarding "this" usage? I prefer not to use it where it isn't needed. Thanks, Marius
Modified: platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/plugin/image/ImagePlugin.java =================================================================== --- platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/plugin/image/ImagePlugin.java 2010-12-08 11:51:33 UTC (rev 33308) +++ platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/plugin/image/ImagePlugin.java 2010-12-08 12:50:55 UTC (rev 33309) @@ -125,10 +125,10 @@ String defaultQualityParam = context.getWiki().Param("xwiki.plugin.image.defaultQuality"); if (!StringUtils.isBlank(defaultQualityParam)&& StringUtils.isNumeric(defaultQualityParam.trim())) { try { - defaultQuality = Math.max(0, Math.min(1, Float.parseFloat(defaultQualityParam.trim()))); + this.defaultQuality = Math.max(0, Math.min(1, Float.parseFloat(defaultQualityParam.trim()))); } catch (NumberFormatException e) { LOG.warn(String.format("Failed to parse xwiki.plugin.image.defaultQuality configuration parameter. " - + "Using %s as the default image quality.", defaultQuality), e); + + "Using %s as the default image quality.", this.defaultQuality), e); } } } @@ -160,16 +160,16 @@ String capacityParam = context.getWiki().Param("xwiki.plugin.image.cache.capacity"); if (!StringUtils.isBlank(capacityParam)&& StringUtils.isNumeric(capacityParam.trim())) { try { - capacity = Integer.parseInt(capacityParam.trim()); + this.capacity = Integer.parseInt(capacityParam.trim()); } catch (NumberFormatException e) { LOG.warn(String.format("Failed to parse xwiki.plugin.image.cache.capacity configuration parameter. " - + "Using %s as the cache capacity.", capacity), e); + + "Using %s as the cache capacity.", this.capacity), e); } } - lru.setMaxEntries(capacity); + lru.setMaxEntries(this.capacity);
try { - imageCache = context.getWiki().getLocalCacheFactory().newCache(configuration); + this.imageCache = context.getWiki().getLocalCacheFactory().newCache(configuration); } catch (CacheException e) { LOG.error("Error initializing the image cache.", e); } @@ -201,7 +201,7 @@ @Override public XWikiAttachment downloadAttachment(XWikiAttachment attachment, XWikiContext context) { - if (!imageProcessor.isMimeTypeSupported(attachment.getMimeType(context))) { + if (!this.imageProcessor.isMimeTypeSupported(attachment.getMimeType(context))) { return attachment; }
@@ -259,23 +259,23 @@ XWikiContext context) throws Exception { boolean keepAspectRatio = Boolean.valueOf(context.getRequest().getParameter("keepAspectRatio")); - if (imageCache == null) { + if (this.imageCache == null) { initCache(context); - if (imageCache == null) { + if (this.imageCache == null) { return shrinkImage(image, width, height, keepAspectRatio, quality, context); } } String key = String.format("%s;%s;%s;%s;%s;%s", image.getId(), image.getVersion(), width, height, keepAspectRatio, - quality); - byte[] data = imageCache.get(key); + quality); + byte[] data = this.imageCache.get(key); XWikiAttachment thumbnail; if (data != null) { thumbnail = (XWikiAttachment) image.clone(); thumbnail.setContent(new ByteArrayInputStream(data), data.length); } else { thumbnail = shrinkImage(image, width, height, keepAspectRatio, quality, context); - imageCache.set(key, thumbnail.getContent(context)); + this.imageCache.set(key, thumbnail.getContent(context)); } return thumbnail; } @@ -300,7 +300,7 @@ private XWikiAttachment shrinkImage(XWikiAttachment attachment, int requestedWidth, int requestedHeight, boolean keepAspectRatio, float requestedQuality, XWikiContext context) throws Exception { - Image image = imageProcessor.readImage(attachment.getContentInputStream(context)); + Image image = this.imageProcessor.readImage(attachment.getContentInputStream(context));
// Compute the new image dimension. int currentWidth = image.getWidth(null); @@ -314,15 +314,15 @@ if (dimensions[0] == currentWidth&& dimensions[1] == currentHeight) { return attachment; } - quality = defaultQuality; + quality = this.defaultQuality; }
// Scale the image to the new dimensions. - RenderedImage shrunkImage = imageProcessor.scaleImage(image, dimensions[0], dimensions[1]); + RenderedImage shrunkImage = this.imageProcessor.scaleImage(image, dimensions[0], dimensions[1]);
// Write the shrunk image to a byte array output stream. ByteArrayOutputStream bout = new ByteArrayOutputStream(); - imageProcessor.writeImage(shrunkImage, attachment.getMimeType(context), quality, bout); + this.imageProcessor.writeImage(shrunkImage, attachment.getMimeType(context), quality, bout);
// Create an image attachment for the shrunk image. XWikiAttachment thumbnail = (XWikiAttachment) attachment.clone(); @@ -395,7 +395,7 @@ */ public int getWidth(XWikiAttachment attachment, XWikiContext context) throws IOException, XWikiException { - return imageProcessor.readImage(attachment.getContentInputStream(context)).getWidth(null); + return this.imageProcessor.readImage(attachment.getContentInputStream(context)).getWidth(null); }
/** @@ -407,6 +407,6 @@ */ public int getHeight(XWikiAttachment attachment, XWikiContext context) throws IOException, XWikiException { - return imageProcessor.readImage(attachment.getContentInputStream(context)).getHeight(null); + return this.imageProcessor.readImage(attachment.getContentInputStream(context)).getHeight(null); } }
_______________________________________________ notifications mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/notifications
On 12/08/2010 04:02 PM, Marius Dumitru Florea wrote:
Hi Sergiu,
On 12/08/2010 02:50 PM, sdumitriu (SVN) wrote:
Author: sdumitriu Date: 2010-12-08 13:50:55 +0100 (Wed, 08 Dec 2010) New Revision: 33309
Modified: platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/plugin/image/ImagePlugin.java Log:
[cleanup] Applied codestyle
Did we set any code style rule regarding "this" usage? I prefer not to use it where it isn't needed.
It seems there's no standard. I prefer to always put it because it's obvious when reading a fragment of code outside an IDE which variables are local and which are class members. -- Sergiu Dumitriu http://purl.org/net/sergiu/
On Dec 9, 2010, at 12:46 AM, Sergiu Dumitriu wrote:
On 12/08/2010 04:02 PM, Marius Dumitru Florea wrote:
Hi Sergiu,
On 12/08/2010 02:50 PM, sdumitriu (SVN) wrote:
Author: sdumitriu Date: 2010-12-08 13:50:55 +0100 (Wed, 08 Dec 2010) New Revision: 33309
Modified: platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/plugin/image/ImagePlugin.java Log:
[cleanup] Applied codestyle
Did we set any code style rule regarding "this" usage? I prefer not to use it where it isn't needed.
It seems there's no standard. I prefer to always put it because it's obvious when reading a fragment of code outside an IDE which variables are local and which are class members.
I agree and I've seen errors in the past with scoping (and they're hard to find). -Vincent
On Thu, Dec 9, 2010 at 09:23, Vincent Massol <[email protected]> wrote:
On Dec 9, 2010, at 12:46 AM, Sergiu Dumitriu wrote:
On 12/08/2010 04:02 PM, Marius Dumitru Florea wrote:
Hi Sergiu,
On 12/08/2010 02:50 PM, sdumitriu (SVN) wrote:
Author: sdumitriu Date: 2010-12-08 13:50:55 +0100 (Wed, 08 Dec 2010) New Revision: 33309
Modified: platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/plugin/image/ImagePlugin.java Log:
[cleanup] Applied codestyle
Did we set any code style rule regarding "this" usage? I prefer not to use it where it isn't needed.
It seems there's no standard. I prefer to always put it because it's obvious when reading a fragment of code outside an IDE which variables are local and which are class members.
I agree and I've seen errors in the past with scoping (and they're hard to find).
+1
-Vincent _______________________________________________ devs mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/devs
-- Thomas Mortagne
participants (4)
-
Marius Dumitru Florea -
Sergiu Dumitriu -
Thomas Mortagne -
Vincent Massol