Reply to thread

Message

<blockquote data-quote="gloomyandy" data-source="post: 90602" data-attributes="member: 24855">Firstly sorry if this is not the correct way to submit a potential bug fix! My first attempt so be gentle!Ok I finally got fed up enough with the problem I reported below:<a href="https://forum.team-mediaportal.com/showthread.php?t=12077" target="_blank">https://forum.team-mediaportal.com/showthread.php?t=12077</a>To try and do something about it. I grabbed a copy of VC# Express and the latest svn source and had a look. The code in question is in file GUIPictures.cs and is in methods CreateFolderThumb and LoadPicture.I found that on creating the thimbnails the DrawImage function was throwing an exception on the face of it there was no obvious reason as to why, the original image had been loaded ok and the background image was also ok. After a bit of digging I came across this web page:<a href="http://www.pixvillage.com/blogs/devblog/archive/2005/03/09/154.aspx" target="_blank">http://www.pixvillage.com/blogs/devblog/archive/2005/03/09/154.aspx</a>Which also has a pointer to a MS knowledge base article. Basically what these say is that when you load an image from a file or a stream the actual reading of the image may be delayed until the image is used. Destroying the file or the stream before the image may result in &quot;bad things happening&quot;. Now the code i(that loads the picture) is creating a stream (and then creating an image from this stream) using a &quot;using block&quot; so the stream will be disposed of when the block exits. The bad news is the function then goes on to return the image created from the stream to the calling function so the lifetime of the stream is &lt; the lifetime of the image.To fix this I re-factored the code a little so that the drawing of the image takes place within a new function AddPicture which replaces the old LoadPicture. The code is shown below (I've attached the complete file to this post as a text file):&nbsp;[CODE] &nbsp; void AddPicture(Graphics g, string strFileName, int x, int y, int w, int h)&nbsp; &nbsp; {&nbsp; &nbsp; &nbsp; // Add a thumbnail of the specified picture file to the image referenced by g, draw it at the&nbsp; &nbsp; &nbsp; // given location and size.&nbsp; &nbsp; &nbsp; Image img = null;&nbsp; &nbsp; &nbsp; using (PictureDatabase dbs = new PictureDatabase())&nbsp; &nbsp; &nbsp; {&nbsp; &nbsp; &nbsp; &nbsp; int iRotate = dbs.GetRotation(strFileName);&nbsp; &nbsp; &nbsp; &nbsp; using (FileStream stream = new FileStream(strFileName, FileMode.Open))&nbsp; &nbsp; &nbsp; &nbsp; {&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; img = Image.FromStream(stream, true, false);&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; if (img != null)&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; {&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; if (iRotate &gt; 0)&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; {&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; RotateFlipType flipType;&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; switch (iRotate)&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; {&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; case 1:&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; flipType = RotateFlipType.Rotate90FlipNone;&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; img.RotateFlip(flipType);&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; break;&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; case 2:&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; flipType = RotateFlipType.Rotate180FlipNone;&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; img.RotateFlip(flipType);&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; break;&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; case 3:&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; flipType = RotateFlipType.Rotate270FlipNone;&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; img.RotateFlip(flipType);&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; break;&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; default:&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; flipType = RotateFlipType.RotateNoneFlipNone;&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; break;&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; }&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; }&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; g.DrawImage(img, x, y, w, h);&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; }&nbsp; &nbsp; &nbsp; &nbsp; }&nbsp; &nbsp; &nbsp; }&nbsp; &nbsp; }[/CODE]This is then called as shown:[CODE]&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; using (Bitmap bmp = new Bitmap(width, height))&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; {&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; using (Graphics g = Graphics.FromImage(bmp))&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; {&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; g.CompositingQuality = System.Drawing.Drawing2D.CompositingQuality.HighQuality;&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; g.InterpolationMode = System.Drawing.Drawing2D.InterpolationMode.HighQualityBicubic;&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; g.SmoothingMode = System.Drawing.Drawing2D.SmoothingMode.HighQuality;&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; g.DrawImage(imgFolder, 0, 0, width, height);&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; int x, y, w, h;&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; x = 0; y = 0; w = thumbnailWidth; h = thumbnailHeight;&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; //Load first of 4 images for the folder thumb.&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; //Avoid crashes caused by damaged image files:&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; try&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; {&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; AddPicture(g, (string)pictureList[0], x + 10, y + 10, w, h);&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; }&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; catch (Exception)&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; {&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; Log.Info(&quot;Damaged picture file found: {0}. Try to repair or delete this file please!&quot;, (string)pictureList[0]);&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; }[/CODE]With the above changes I no longer get exceptions when the image is drawn to the thumbnail and thumbnails are created for all of my picture folders with no errors. It is worth noting that if the old LoadPicture function needed to rotate the picture then this would have resulted in the image being initialized from the stream while the stream still existed and so no exception would have been thrown. This fits perfectly with what I was seeing, some pictures used for thumbnails seemed to be ok others did not. On closer inspection the files that worked required rotation, the ones that failed did not.Hope this makes sense....AndyAdded to SF as patch[ 1597327 ] Blank thumbnail for directories in MyPictures<a href="https://sourceforge.net/tracker/index.php?func=detail&amp;aid=1597327&amp;group_id=107397&amp;atid=647927" target="_blank">https://sourceforge.net/tracker/index.php?func=detail&amp;aid=1597327&amp;group_id=107397&amp;atid=647927</a></blockquote>

[QUOTE="gloomyandy, post: 90602, member: 24855"] Firstly sorry if this is not the correct way to submit a potential bug fix! My first attempt so be gentle! Ok I finally got fed up enough with the problem I reported below: [url]https://forum.team-mediaportal.com/showthread.php?t=12077[/url] To try and do something about it. I grabbed a copy of VC# Express and the latest svn source and had a look. The code in question is in file GUIPictures.cs and is in methods CreateFolderThumb and LoadPicture.I found that on creating the thimbnails the DrawImage function was throwing an exception on the face of it there was no obvious reason as to why, the original image had been loaded ok and the background image was also ok. After a bit of digging I came across this web page: [url]http://www.pixvillage.com/blogs/devblog/archive/2005/03/09/154.aspx[/url] Which also has a pointer to a MS knowledge base article. Basically what these say is that when you load an image from a file or a stream the actual reading of the image may be delayed until the image is used. Destroying the file or the stream before the image may result in "bad things happening". Now the code i(that loads the picture) is creating a stream (and then creating an image from this stream) using a "using block" so the stream will be disposed of when the block exits. The bad news is the function then goes on to return the image created from the stream to the calling function so the lifetime of the stream is < the lifetime of the image. To fix this I re-factored the code a little so that the drawing of the image takes place within a new function AddPicture which replaces the old LoadPicture. The code is shown below (I've attached the complete file to this post as a text file): [CODE] void AddPicture(Graphics g, string strFileName, int x, int y, int w, int h) { // Add a thumbnail of the specified picture file to the image referenced by g, draw it at the // given location and size. Image img = null; using (PictureDatabase dbs = new PictureDatabase()) { int iRotate = dbs.GetRotation(strFileName); using (FileStream stream = new FileStream(strFileName, FileMode.Open)) { img = Image.FromStream(stream, true, false); if (img != null) { if (iRotate > 0) { RotateFlipType flipType; switch (iRotate) { case 1: flipType = RotateFlipType.Rotate90FlipNone; img.RotateFlip(flipType); break; case 2: flipType = RotateFlipType.Rotate180FlipNone; img.RotateFlip(flipType); break; case 3: flipType = RotateFlipType.Rotate270FlipNone; img.RotateFlip(flipType); break; default: flipType = RotateFlipType.RotateNoneFlipNone; break; } } g.DrawImage(img, x, y, w, h); } } } } [/CODE] This is then called as shown: [CODE] using (Bitmap bmp = new Bitmap(width, height)) { using (Graphics g = Graphics.FromImage(bmp)) { g.CompositingQuality = System.Drawing.Drawing2D.CompositingQuality.HighQuality; g.InterpolationMode = System.Drawing.Drawing2D.InterpolationMode.HighQualityBicubic; g.SmoothingMode = System.Drawing.Drawing2D.SmoothingMode.HighQuality; g.DrawImage(imgFolder, 0, 0, width, height); int x, y, w, h; x = 0; y = 0; w = thumbnailWidth; h = thumbnailHeight; //Load first of 4 images for the folder thumb. //Avoid crashes caused by damaged image files: try { AddPicture(g, (string)pictureList[0], x + 10, y + 10, w, h); } catch (Exception) { Log.Info("Damaged picture file found: {0}. Try to repair or delete this file please!", (string)pictureList[0]); } [/CODE] With the above changes I no longer get exceptions when the image is drawn to the thumbnail and thumbnails are created for all of my picture folders with no errors. It is worth noting that if the old LoadPicture function needed to rotate the picture then this would have resulted in the image being initialized from the stream while the stream still existed and so no exception would have been thrown. This fits perfectly with what I was seeing, some pictures used for thumbnails seemed to be ok others did not. On closer inspection the files that worked required rotation, the ones that failed did not. Hope this makes sense.... Andy Added to SF as patch [ 1597327 ] Blank thumbnail for directories in MyPictures [url]https://sourceforge.net/tracker/index.php?func=detail&aid=1597327&group_id=107397&atid=647927[/url] [/QUOTE]

Verification