thejasviv wrote:

> Dear All,
>
> Here is my code:
>
> void main()
> {
> char *p="Hello";
> *p='M'; //This is where the error occurs
> cout<<p<<endl;
> }
>
> This code compiles and executes perfectly in Boreland C++. When I run the
> same code in VC++ 6.0, however, I get the below error in debugging mode:
> Unhandled exception in Test.exe: 0xC0000005: Access Violation
>
> Can someone please provide me a solution to this?

"Ben Voigt" wrote:

>
> "David Wilkinson" <no-reply@effisols.com> wrote in message
> news:OXLFO3TLHHA.3668@TK2MSFTNGP02.phx.gbl...
> > thejasviv wrote:
> >
> >> Dear All,
> >>
> >> Here is my code:
> >>
> >> void main()
> >> {
> >> char *p="Hello";
> >> *p='M'; //This is where the error occurs
> >> cout<<p<<endl;
> >> }
> >>
> >> This code compiles and executes perfectly in Boreland C++. When I run the
> >> same code in VC++ 6.0, however, I get the below error in debugging mode:
> >> Unhandled exception in Test.exe: 0xC0000005: Access Violation
> >>
> >> Can someone please provide me a solution to this?
> >
> > thejasviv:
> >
> > When you write
> >
> > char *p = "Hello";
> >
> > you should really write
> >
> > const char *p = "Hello";
> >
> > because the string literal "Hello" is (or at least may be) in read-only
> > memory. Omitting the const is only permitted for legacy C reasons. If you
> > want to modify the string you should write
> >
> > char p[] = "Hello";
>
> Better, but still not correct. The compiler can still choose to use
> read-only memory, because it's still a string literal.
>
> Best is to use composite initializer syntax (an array is a composite):
> char p[] = { "Hello" }; // identical to char[] p = { 'H', 'e', 'l', 'l',
> 'o', '\0' };
>
> >
> > David Wilkinson
> >
> >
> >
> >
> >
> >
>
>