[haiku-bugs] Re: [Haiku] #12653: Edit JPEG translator to remove warnings, allow -Werror

  • From: "MichaelCrawford" <trac@xxxxxxxxxxxx>
  • Date: Wed, 24 Feb 2016 12:00:43 -0000

#12653: Edit JPEG translator to remove warnings, allow -Werror
-------------------------------+----------------------------
   Reporter:  MichaelCrawford  |      Owner:  nobody
       Type:  enhancement      |     Status:  new
   Priority:  normal           |  Milestone:  Unscheduled
  Component:  - General        |    Version:  R1/Development
 Resolution:                   |   Keywords:
 Blocked By:                   |   Blocking:  9460
Has a Patch:  0                |   Platform:  All
-------------------------------+----------------------------

Comment (by MichaelCrawford):

 There was just one warning, line 766 of JPEGTranslator.cpp, outType might
 be clobbered by 'longjmp' or 'vfork'.

 outType is a uint32, and so is likely stored in a register.  The setjmp
 stores the registeres in the jmp_buf.  The solution appears to be to
 declare outType volatile, so it is stored on the stack.  In this
 particular routine doing so will have a negligible effect on performance

--
Ticket URL: <https://dev.haiku-os.org/ticket/12653#comment:1>
Haiku <https://dev.haiku-os.org>
Haiku - the operating system.

Other related posts: