Bug 162141 - 2 * operation is undefined
2 * operation is undefined
Status: CLOSED WONTFIX
Product: Fedora
Classification: Fedora
Component: gstreamer-plugins (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Monty
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2005-06-30 06:01 EDT by David Binderman
Modified: 2013-10-20 18:41 EDT (History)
2 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2007-02-01 09:07:44 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)

  None (edit)
Description David Binderman 2005-06-30 06:01:26 EDT
Description of problem:

I just tried to compile package gstreamer-plugins-0.8.8-6 from Redhat
Fedora Core development tree with the gcc 4.0.0 compiler and
the compiler flag -Wall.

The compiler said

1.

sndfile.cpp:1178: warning: operation on 'pSample' may be undefined

The source code is

	*pSample++ = bswapLE16(*pSample);

I agree with the compiler. Suggest better code

	*pSample = bswapLE16(*pSample);
	++pSample;

2.

sndfile.cpp:1369: warning: operation on 'pSampleW' may be undefined

Duplicate.



Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:


Additional info:
Comment 1 John (J5) Palmieri 2005-06-30 10:18:27 EDT
Thanks for the report.  Can you create a patch and attach it here and also send
it upstream.
Comment 2 David Binderman 2005-07-01 04:15:51 EDT
>Can you create a patch

I could, but since I have no way to test it, then
it would be a risky patch.

Suggest forward bug report to upstream developers,
who are in a much better position to accurately 
patch and test.
Comment 3 Bastien Nocera 2007-02-01 09:07:44 EST
GStreamer 0.8 series is long dead, and there doesn't seem to be any such plugins
anymore, so closing.

Note You need to log in before you can comment on or make changes to this bug.