Re:[PATCH 4/5] gdiplus: Implement SetPageTransform playback.

[Akira Nakagawa]

You have written the following:
but you missed break;at the end of case {}.
+        case EmfPlusRecordTypeSetPageTransform:
+        {
+            EmfPlusSetPageTransform *record = (EmfPlusSetPageTransform*)header;
+            GpUnit unit = (GpUnit)flags;
+
+            if (dataSize + sizeof(EmfPlusRecordHeader) < sizeof(EmfPlusSetPageTransform))
+                return InvalidParameter;
+
+            real_metafile->page_unit = unit;
+            real_metafile->page_scale = record->PageScale;
+
+            return METAFILE_PlaybackUpdateWorldTransform(real_metafile);
+        }
         default:

[PATCH 4/5] gdiplus: Implement SetPageTransform playback.

[Bruno Jesus]

On Wed, Jan 22, 2014 at 10:49 AM, Akira Nakagawa  wrote:
> You have written the following:
> but you missed break;at the end of case {}.

There is a return, so there is no need for a break.
> -
>    case EmfPlusRecordTypeSetPageTransform:
> -
>    {
> -
>        EmfPlusSetPageTransform *record = (EmfPlusSetPageTransform*)header;
> -
>        GpUnit unit = (GpUnit)flags;
> -
> -
>        if (dataSize + sizeof(EmfPlusRecordHeader) < sizeof(EmfPlusSetPageTransform))
> -
>            return InvalidParameter;
> -
> -
>        real_metafile->page_unit = unit;
> -
>        real_metafile->page_scale = record->PageScale;
> -
> -
>        return METAFILE_PlaybackUpdateWorldTransform(real_metafile);
> -
>    }
>    default: