Bug 137566 - bmp-plugins-0.9.7-12: undefined C code
Summary: bmp-plugins-0.9.7-12: undefined C code
Status: RESOLVED WONTFIX
Alias: None
Product: SUSE LINUX 10.0
Classification: openSUSE
Component: Basesystem (show other bugs)
Version: Final
Hardware: All Other
: P5 - None : Minor
Target Milestone: ---
Assignee: Hendrik Vogelsang
QA Contact: E-mail List
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-12-08 12:43 UTC by David Binderman
Modified: 2006-05-18 11:10 UTC (History)
0 users

See Also:
Found By: Other
Services Priority:
Business Priority:
Blocker: ---
Marketing QA Status: ---
IT Deployment: ---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description David Binderman 2005-12-08 12:43:02 UTC 
I just tried to compile package bmp-plugins-0.9.7-12 with a prerelease of the
GNU C compiler version 4.1. It said

1.

sndfile.cpp:1174: warning: operation on "pSample" may be undefined

The source code is

                                *pSample++ = bswapLE16(*pSample);


I agree with the compiler - this code seems to be undefined. 

AFAIK, there are no guarantees in C that the side effects on the left of an
assignment happens before or after the right side effects.

Suggest code rework.

2.

sndfile.cpp:1365: warning: operation on "pSample" may be undefined

Duplicate.


BTW, I've already tried emailing who I thought was the author 
[ jeremytan@users.sourceforge.net ], without success.
Comment 1 Hendrik Vogelsang 2006-01-16 14:49:28 UTC 
please talk to the authors of libmodplug

http://sourceforge.net/projects/modplug-xmms/

so we can pick it up from there
Comment 2 David Binderman 2006-01-16 22:09:10 UTC 
>please talk to the authors of libmodplug

I tried that, and it didn't work.

It looks like this bug report goes to /dev/null
Comment 3 Hendrik Vogelsang 2006-05-18 11:10:31 UTC 
we will drop bmp and bmp-plugins for the next version.