473,756 Members | 5,660 Online
Bytes | Software Development & Data Engineering Community
+ Post

Home Posts Topics Members FAQ

operator += on short int values - strange compiler warning

I have some very simple code fragment:

short int n1, n2;
....
n1 += n2;

I get an warning from VS2003 - VS2005: warning C4244: '+=' : conversion
from 'int' to 'short', possible loss of data

n1 = n1 + n2;

Here I don't get the warning. Strange...

I'm using warning level 4. Is this "VC++ specific" or conformant to the
standard?

Greets

--
Henryk

Feb 7 '06 #1
11 1768
On 7 Feb 2006 06:12:57 -0800, "Henryk" <he************ @gmx.de> wrote:
I have some very simple code fragment:

short int n1, n2;
...
n1 += n2;

I get an warning from VS2003 - VS2005: warning C4244: '+=' : conversion
from 'int' to 'short', possible loss of data

n1 = n1 + n2;

Here I don't get the warning. Strange...

I'm using warning level 4. Is this "VC++ specific" or conformant to the
standard?


I would have expected the warning in both cases. The result of n1 + n2
is of type int according to the type promotion rules.

Best wishes,
Roland Pibinger
Feb 7 '06 #2

"Roland Pibinger" <rp*****@yahoo. com> wrote in message
news:43******** ******@news.uta net.at...
n1 += n2;

I get an warning from VS2003 - VS2005: warning C4244: '+=' : conversion
from 'int' to 'short', possible loss of data

n1 = n1 + n2;

Here I don't get the warning. Strange...

I'm using warning level 4. Is this "VC++ specific" or conformant to the
standard?


I would have expected the warning in both cases. The result of n1 + n2
is of type int according to the type promotion rules.


Why?

Feb 7 '06 #3
Duane Hebert wrote:
"Roland Pibinger" <rp*****@yahoo. com> wrote in message
news:43******** ******@news.uta net.at...

n1 += n2;

I get an warning from VS2003 - VS2005: warning C4244: '+=' : conversion

from 'int' to 'short', possible loss of data

n1 = n1 + n2;

Here I don't get the warning. Strange...

I'm using warning level 4. Is this "VC++ specific" or conformant to the
standard?


I would have expected the warning in both cases. The result of n1 + n2
is of type int according to the type promotion rules.

Why?


I guess I don't understand the question. Why what? Why the promotions
are performed on both operands? Because the Standard says so (5/9). Or
why is it so in the Standard? I am not sure, ask in comp.std.c++.

V
--
Please remove capital As from my address when replying by mail
Feb 7 '06 #4

"Victor Bazarov" <v.********@com Acast.net> wrote in message
news:0c******** *******@newsrea d1.mlpsca01.us. to.verio.net...
I guess I don't understand the question. Why what? Why the promotions
are performed on both operands? Because the Standard says so (5/9). Or
why is it so in the Standard? I am not sure, ask in comp.std.c++.


Sorry for the unclear question. I wanted to know if the standard required
addition of two shorts to be promoted to an int. I know if the two are
different
sizes, they're promoted to the size of the larger.

Thanks.

Feb 7 '06 #5
Duane Hebert wrote:
"Victor Bazarov" <v.********@com Acast.net> wrote in message
news:0c******** *******@newsrea d1.mlpsca01.us. to.verio.net...

I guess I don't understand the question. Why what? Why the promotions
are performed on both operands? Because the Standard says so (5/9). Or
why is it so in the Standard? I am not sure, ask in comp.std.c++.

Sorry for the unclear question. I wanted to know if the standard required
addition of two shorts to be promoted to an int.


Yes, that's how I read 5/9, the fourth bullet item.
I know if the two are
different
sizes, they're promoted to the size of the larger.


The Standard does not make it conditional. Both operands shall be
promoted. That means if 'int' can accommodate them, they will be made
'int'. If 'int' can't accommodate either of them, that one will be made
'unsigned int'. Then if one of them is 'unsigned' and the other isn't,
they are both made 'unsigned'.

V
--
Please remove capital As from my address when replying by mail
Feb 7 '06 #6

"Victor Bazarov" <v.********@com Acast.net> wrote in message
news:9w******** *******@newsrea d1.mlpsca01.us. to.verio.net...
The Standard does not make it conditional. Both operands shall be
promoted. That means if 'int' can accommodate them, they will be made
'int'. If 'int' can't accommodate either of them, that one will be made
'unsigned int'. Then if one of them is 'unsigned' and the other isn't,
they are both made 'unsigned'.


Thanks.
Feb 8 '06 #7
Thank you all,

I thought that the promotion occures to the largest type of all
operands involved.

The question is why I do get this warning only for the += (and possibly
for -= and others too) but not for the explizit operations.

I did not use a pendatic warning level before, but for the current
project I have to. So I was not aware of this problem.

This project contains a lot of algorithm stuff with mathematical
operations. The compiler would have to produce tons of warnings
according to the promotion rules. But I only get a few from this += ...

Btw, is there a good online source for the C++ Standard? I just
downloaded a pdf from open-std.org but it seems not up-to-date (its
from 2001).

Feb 8 '06 #8

Victor Bazarov wrote:
Duane Hebert wrote:
"Victor Bazarov" <v.********@com Acast.net> wrote in message
news:0c******** *******@newsrea d1.mlpsca01.us. to.verio.net...

I guess I don't understand the question. Why what? Why the promotions
are performed on both operands? Because the Standard says so (5/9). Or
why is it so in the Standard? I am not sure, ask in comp.std.c++.

Sorry for the unclear question. I wanted to know if the standard required
addition of two shorts to be promoted to an int.


Yes, that's how I read 5/9, the fourth bullet item.


But the fourth bullet directs to 4.5, and 4.5/1 reads:
"An rvalue of type char, signed char, unsigned char, short int, or
unsigned short int **can** be converted to an rvalue of type int if int
can represent all the values of the source type; otherwise, the source
rvalue **can** be converted to an rvalue of type unsigned int."
(the ** are mine)

As I interpret it, it is not *required* it is a possibility. Please,
correct me if I am wrong.

[snip]

TIA,

Marcelo Pinto

Feb 8 '06 #9
Marcelo Pinto wrote:
Victor Bazarov wrote:

Duane Hebert wrote:
"Victor Bazarov" <v.********@com Acast.net> wrote in message
news:0c***** **********@news read1.mlpsca01. us.to.verio.net ...

I guess I don't understand the question. Why what? Why the promotions
are performed on both operands? Because the Standard says so (5/9). Or
why is it so in the Standard? I am not sure, ask in comp.std.c++.
Sorry for the unclear question. I wanted to know if the standard required
addition of two shorts to be promoted to an int.


Yes, that's how I read 5/9, the fourth bullet item.

But the fourth bullet directs to 4.5, and 4.5/1 reads:
"An rvalue of type char, signed char, unsigned char, short int, or
unsigned short int **can** be converted to an rvalue of type int if int
can represent all the values of the source type; otherwise, the source
rvalue **can** be converted to an rvalue of type unsigned int."
(the ** are mine)

As I interpret it, it is not *required* it is a possibility. Please,
correct me if I am wrong.


Subclause 4.5 shows _how_ it's done *if* it's done. 5/9 requires that it
_shall_ be done. Read it again.

V
--
Please remove capital As from my address when replying by mail
Feb 8 '06 #10

This thread has been closed and replies have been disabled. Please start a new discussion.

Similar topics

2
5883
by: victor75040 | last post by:
Before you all start flaming me, I am not a student and this is not for any homework. Just someone learing c++ on their own. I am now up to the chapter in my book that describes operator overloading. I just cannot find any explanation that clearly point out what the parts of the statement refer to. For example the book says: comp operator+(comp b)
18
2144
by: cppaddict | last post by:
Hi, Is it considered bad form to have the subscript operator return a const reference variable? If not, what is the proper way to do it? My question was prompted by the code below, my problematic attempt to implement a subscript operator that returns a const reference. The dubious code is marked at the end. <code>
99
9085
by: Glen Herrmannsfeldt | last post by:
I was compiling a program written by someone else about six years ago, and widely distributed at the time. It also includes makefiles for many different systems, so I know it has been compiled with many different compilers. I got compile errors when it used va_arg to fetch an argument of type short. That seemed a little strange to me, so I changed it to int and it compiled just fine. So now I wonder, just what is the rule for va_arg...
8
1695
by: NilsNilsson | last post by:
I wrote this: short s1 = 0; short s2 = 1; short s3 = s1 + s2; And gor this compile error message: Cannot implicitly convert type 'int' to 'short' What is wrong here?
4
10267
by: seb666fr2 | last post by:
hello, Anybody can tell me why it is impossible to use the '&' operator with operands of type 'long' or 'ulong' and what i must use instead of this? thanks.
12
2209
by: cody | last post by:
Why can I overload operator== and operator!= separately having different implementations and additionally I can override equals() also having a different implementation. Why not forbid overloading of == and != but instead translate each call of objA==objB automatically in System.Object.Equals(objA, objB). This would remove inconsistencies like myString1==myString2
16
2821
by: Norman Diamond | last post by:
In an antique obsolete version of MFC, a CString expression could be subscripted in order to retrieve one element. Visual Studio 2005 defines CSimpleStringT::operator. At first glance it looks like it might have been intended to provide backwards compatibility for antique programs. But there seems to be no way to use it. CString s = _T("ab"); short i = 1; _TCHAR c = s;
7
2084
by: email7373388 | last post by:
I'm working on a program which has a strange operator, :>. This is the syntax: ((unsigned short)( var1)):>((void __near *)( var2 )) Any clue?
3
2427
by: Thomas Lenz | last post by:
The code below should allow to use a comma instead of << with ostreams and include a space between two operands when comma is used. e.g. cout << "hello", "world", endl; should print the line "hello world". #include <iostream> using namespace std;
0
9487
marktang
by: marktang | last post by:
ONU (Optical Network Unit) is one of the key components for providing high-speed Internet services. Its primary function is to act as an endpoint device located at the user's premises. However, people are often confused as to whether an ONU can Work As a Router. In this blog post, we’ll explore What is ONU, What Is Router, ONU & Router’s main usage, and What is the difference between ONU and Router. Let’s take a closer look ! Part I. Meaning of...
0
9297
by: Hystou | last post by:
Most computers default to English, but sometimes we require a different language, especially when relocating. Forgot to request a specific language before your computer shipped? No problem! You can effortlessly switch the default language on Windows 10 without reinstalling. I'll walk you through it. First, let's disable language synchronization. With a Microsoft account, language settings sync across devices. To prevent any complications,...
0
10069
Oralloy
by: Oralloy | last post by:
Hello folks, I am unable to find appropriate documentation on the type promotion of bit-fields when using the generalised comparison operator "<=>". The problem is that using the GNU compilers, it seems that the internal comparison operator "<=>" tries to promote arguments from unsigned to signed. This is as boiled down as I can make it. Here is my compilation command: g++-12 -std=c++20 -Wnarrowing bit_field.cpp Here is the code in...
1
9884
by: Hystou | last post by:
Overview: Windows 11 and 10 have less user interface control over operating system update behaviour than previous versions of Windows. In Windows 11 and 10, there is no way to turn off the Windows Update option using the Control Panel or Settings app; it automatically checks for updates and installs any it finds, whether you like it or not. For most users, this new feature is actually very convenient. If you want to control the update process,...
0
9735
tracyyun
by: tracyyun | last post by:
Dear forum friends, With the development of smart home technology, a variety of wireless communication protocols have appeared on the market, such as Zigbee, Z-Wave, Wi-Fi, Bluetooth, etc. Each protocol has its own unique characteristics and advantages, but as a user who is planning to build a smart home system, I am a bit confused by the choice of these technologies. I'm particularly interested in Zigbee because I've heard it does some...
0
8736
agi2029
by: agi2029 | last post by:
Let's talk about the concept of autonomous AI software engineers and no-code agents. These AIs are designed to manage the entire lifecycle of a software development project—planning, coding, testing, and deployment—without human intervention. Imagine an AI that can take a project description, break it down, write the code, debug it, and then launch it, all on its own.... Now, this would greatly impact the work of software developers. The idea...
1
7285
isladogs
by: isladogs | last post by:
The next Access Europe User Group meeting will be on Wednesday 1 May 2024 starting at 18:00 UK time (6PM UTC+1) and finishing by 19:30 (7.30PM). In this session, we are pleased to welcome a new presenter, Adolph Dupré who will be discussing some powerful techniques for using class modules. He will explain when you may want to use classes instead of User Defined Types (UDT). For example, to manage the data in unbound forms. Adolph will...
0
5324
by: adsilva | last post by:
A Windows Forms form does not have the event Unload, like VB6. What one acts like?
3
2697
bsmnconsultancy
by: bsmnconsultancy | last post by:
In today's digital era, a well-designed website is crucial for businesses looking to succeed. Whether you're a small business owner or a large corporation in Toronto, having a strong online presence can significantly impact your brand's success. BSMN Consultancy, a leader in Website Development in Toronto offers valuable insights into creating effective websites that not only look great but also perform exceptionally well. In this comprehensive...

By using Bytes.com and it's services, you agree to our Privacy Policy and Terms of Use.

To disable or enable advertisements and analytics tracking please visit the manage ads & tracking page.