davidllewellynjones added inline comments.

INLINE COMMENTS

> PictDestination.cpp:101
> +                }
> +            }
> +

The logic is confusing me here I'm afraid. If there's no `\picwgoal` control 
word, then `m_goalWidth` will default to 0 and the code above will be skipped. 
The image width will then be set to 0 on line 104: `m_imageFormat.setWidth( 0 
);`. Similarly for height. Is this correct? It feels like it might make more 
sense for the conditions to be reversed.

> denexter wrote in PictDestination.cpp:54
> I think picw and pich are source sizes and somewhat meaningless since that 
> information is encoded in the image. The goal size and scale combine to give 
> the display size.

Thanks for the explanation, that makes sense. Probably these control words are 
more important for WMFs and the like.

> pvuorela wrote in PictDestination.cpp:110
> Yea, could change these too. Though have been pondering of just feeding the 
> whole rtf-qt through astyle to get consistent formatting and whitespace.

I think it needs it, but I appreciate that's not really related to this change.

REPOSITORY
  R8 Calligra

BRANCH
  unicode

REVISION DETAIL
  https://phabricator.kde.org/D24943

To: pvuorela, davidllewellynjones
Cc: denexter, davidllewellynjones, Calligra-Devel-list, dcaliste, cochise, 
vandenoever

Reply via email to