Wireshark-commits: [Wireshark-commits] master b8b77ae: Clamp zooming so that we don't get zero or n

From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Sat, 12 Dec 2015 01:01:58 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=b8b77aecc38f8ada88de78939e4d35d0fa535bd4
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

b8b77ae by Guy Harris (guy@xxxxxxxxxxxx):

    Clamp zooming so that we don't get zero or negative font sizes.
    
    Those are obviously wrong.
    
    Also, clean up some stuff left over from the GTK+ 1.x days; GTK+ 2.x
    doesn't expose raw XLFD font names, it lets you specify a font by name
    and size, and font_zoom() doesn't determine whether the font is
    resizeable - it just constructs a new font name/size pair and leaves it
    up to its callers to try to load the font, so "there's no such font as
    Wingdings Gothic" and "you can't blow up Fraktur to 10 million points"
    both show up as errors loading the font by name.
    
    Bug: 8854
    Change-Id: I6af142c75c9ebabd1a95308c203f8cb1f36dd82f
    Reviewed-on: https://code.wireshark.org/review/12549
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  2ebfa30   IPv6: Dest/Hop extension header register_info fixup
    adds  b8b77ae   Clamp zooming so that we don't get zero or negative font sizes.


Summary of changes:
 ui/gtk/font_utils.c       |   28 +++++++++-------------------
 ui/gtk/font_utils.h       |    2 +-
 ui/gtk/main.c             |    8 +++++---
 ui/gtk/prefs_font_color.c |    5 ++---
 4 files changed, 17 insertions(+), 26 deletions(-)