Resolution units

[albertisi]

I have an image that i save with resolution and resolution unit like this:

Code: Select all

someImage.density(Geometry(resolution, resolution));
someImage.resolutionUnits(MagickCore::ResolutionType::PixelsPerCentimeterResolution);
someImage.write(imagePath);

Once i read the same image right after, my resolution units are set to unknown (0), why is that? I think that is the bug in magick++.
Read code:

Code: Select all

Image someImage2;
someImage2.read(imagePath);			
int resUnit2 = someImage2.resolutionUnits();

I am saving an image in PNG format. Please let me know if you need more information.

[dlemstra]

What is your OS and version of ImageMagick?

[albertisi]

My OS is Win 7. ImageMagick Version 6.8.9

[dlemstra]

What is the format of the image you are reading and writing?

[albertisi]

I am reading and writing PNG format image.

-Albert

[albertisi]

Anyone had this problem besides me?

[fmw42]

If the input image has no resolution, it will have unknown units. So you must specify both -density and -units.

see
http://www.imagemagick.org/script/comma ... hp#density
http://www.imagemagick.org/script/comma ... .php#units

PNG only supports pixelspercentimeter, but if you specify pixelsperinch, IM will convert that for you appropriately.

[albertisi]

@fmw42 i am setting density, and units, as in my original post.

I debugged image magic, and the problem is during the retrieval of resolutionUnits from the image.
After i read the image, i call to return resolution units In Image.cpp this code gets executed

Code: Select all

Magick::ResolutionType Magick::Image::resolutionUnits(void) const
{
  return(constOptions()->resolutionUnits());
}

Unfortunately constOptions() are not set, or updated during the read, so it contains default values for all attributes.
This code should be updated to this:

Code: Select all

Magick::ResolutionType Magick::Image::resolutionUnits(void) const
{
  return(constImage()->units());
}

Image attributes contains all the attributes setup after the read, constOptions doesnt.

Not sure how to proceed further...

[snibgo]

This code should be updated to this: ...

Are you saying IM has a bug, and this is your suggested correction? If so, then post to the bugs forum viewforum.php?f=3