C / C++ / MFC

9-May-13 6:27

Hi,

First all I want to thank all for the help with exception resulting from the CWnd::SendWIndow
seemed it had a errorous pointer as in the lparam

C++

my_event->buffer_ptr

I have done research on the Streamin Call Back function of rich edit but I haven't gotten it to work

I am trying to read into my RichEdit a MainFrame Assembler program Listing

I have a few questions

1) how do you terminate the callback the doc says set *pcb = 0; I did that call back function was still called after that

2) to set things on separate lines I add carriage return '/r' line feed '/n' at the end of each line is this correct

3) How does the framework set (or can I set the value of cb) number of bytes to read

Thanks

Richard MacCutchan9-May-13 7:06

9-May-13 7:06

It would help if you showed your actual code, as it's difficult to visualise exactly what you are doing. As to Q3, the documentation[^] implies that the framework sets the value of cb and the callback should transfer a mximum of that many bytes.

Use the best guess

ForNow9-May-13 18:54

9-May-13 18:54

C++

static DWORD CALLBACK readlisting(
         DWORD dwCookie,      // (in) pointer to CFile
         LPBYTE pbBuff,       // (in) pointer to the destination buffer
         LONG cb,             // (in) size in bytes of the destination buffer
        LONG FAR *pcb        // (out) number of bytes transfered
       )
 {
  UNREFERENCED_PARAMETER(cb);
          static BOOL flag = 0;
 //        LPCTSTR lpch;
		 int length;
        // CString buffer((LPCTSTR) lpch,135);
		 CString buffer; 
              //   char *holdptr;
                                 LPBYTE holdptr1;
                 int n;
                 CStdioFile* pFile = (CStdioFile*) dwCookie;

         switch (flag)
                 {
                 case 0:


                         flag = 1;


                                                do {
                                                     pFile->ReadString(buffer);
                                                    } while(buffer.Find((LPCTSTR)"CSECT") == -1);
                        length = buffer.GetLength();                                  
                        strcpy((char *) pbBuff,(char *)buffer.GetBuffer(length));
						holdptr1 = pbBuff + length;
						memcpy(holdptr1,"\r\n",2);
                          //     *pcb = 135;                // 87 btes transfer
							*pcb = length;
                        return 0;
                         break;
                 case 1:

                                                 while(TRUE)
                                                 {
                                                  pFile->ReadString(buffer);
												                                           length = buffer.GetLength();
                                                 if(buffer.Find((LPCTSTR)"END") > 0)
												 {
													                                        *pcb = 0;              // finish it up
                                                          return 0;          //
												 }
                  if(buffer.Find((LPCTSTR) "  ") < 2 || buffer.Find((LPCTSTR) "    ")    
                                                  ==7)          // no Location counter or opcode 
                                                 {
                                                  memset(pbBuff,0x20,135);
                                                  strcpy((char *)pbBuff,"  ");
                                                  holdptr1 =  pbBuff + 2;
                                                  memcpy(holdptr1,buffer.GetBuffer(length),length);
                                                  holdptr1 = holdptr1 + length;
                                                  memcpy(holdptr1,"\r\n",2);
                                                  *pcb = length + 2;
                                                         return 0;
                                                 }
                                                 else
                                                 {
												                                         n = buffer.Find((LPCTSTR) "    ");
                                                  memset(pbBuff,0x20,135);
                                                  strcpy((char *)pbBuff,"_ ");
                                                  holdptr1 =  pbBuff + 2;
                                                  memcpy(holdptr1,buffer.GetBuffer
                                                  (length),length);  // copy over location counter
                                                  holdptr1 = holdptr1 + length;
                                                   memcpy(holdptr1,"\r\n",2);
                                                   *pcb  = length + 2;
                                                   return 0;
                                                 }
                                                                                                 }
                 }
      return 0;
 }

Richard MacCutchan9-May-13 20:49

9-May-13 20:49

Apart from:

C++

memcpy(holdptr1,"\r\n",2);
//     *pcb = 135;                // 87 btes transfer
*pcb = length;  // should be length + 2

I cannot see anything incorrect in your code. However, you should be checking that you do not transfer more than cb bytes at each call; currently your code does not do that.
Are you sure that it gets to the point where it sets *pcb = 0;?

Use the best guess

ForNow10-May-13 3:06

10-May-13 3:06

Cb is 4096

Thanks

Richard MacCutchan10-May-13 3:40

10-May-13 3:40

Some other issues I noticed from reviewing the above:

You are mixing strcpy() and memcpy() calls. This will cause problems if you want to convert this to Unicode in the future.
Similarly you are using a LPBYTE pointer instead of LPTSTR. Your code is manipulating characters so all pointers should be character types, not byte.
In the second case clause, in both blocks you are prepending two characters to the buffer, but you are not adding two to the *pcb value to take account of this.
In the same place, you are using memset() to initialise the buffer with 135 spaces. This serves no purpose, but to waste time and code.

Whether fixing the above gets you closer to a solution, only time will tell.

Use the best guess

ForNow10-May-13 9:33

10-May-13 9:33

Forgot the WS_VISIBLE style in my resource file sorry for wasting your time ill make the changes you
Pointed out

Thanks

Richard MacCutchan10-May-13 21:58

how to get self in c++ 21day book source code ?

10-May-13 21:58

I don't see how that has anything to do with either your original question, or the code you posted.

Use the best guess

reytc8-May-13 3:13

reytc

8-May-13 3:13

C++

how to get self in c++ 21day book source code ? my buy the book is 5th，
http://www.informit.com/store/sams-teach-yourself-c-plus-plus-in-21-days-adobe-reader-9780768666007[^]

please give me a source code download site,thanks.

My English is not very good, please understand

Re: how to get self in c++ 21day book source code ?

David Crow8-May-13 4:19

David Crow

8-May-13 4:19

If you own the book, why not just type the code in? Copy/paste bulk code is a sure fire way to not learn something.

Re: how to get self in c++ 21day book source code ?

Richard MacCutchan8-May-13 4:57

Re: how to get self in c++ 21day book source code ?

8-May-13 4:57

Look in the book and there will be a section somewhere (often near the end) which gives the link for the source code.

Use the best guess

Malli_S9-May-13 3:27

Malli_S

9-May-13 3:27

Also some books have CDs with them. Check whether the book you're referencing provides one.

[Delegates] [Virtual Desktop] [Tray Me !]

-Malli...!

Re: how to get self in c++ 21day book source code ?

scottgp9-May-13 3:46

scottgp

9-May-13 3:46

Looks like it's here, http://www.informit.com/store/sams-teach-yourself-c-plus-plus-in-21-days-9780672327117[^], under "Downloads"?
Scott

Re: how to get self in c++ 21day book source code ?

Amarnath S9-May-13 21:58

Amarnath S

9-May-13 21:58

21 days for C++ - really enough?

Look at this viewpoint from Peter Norvig - one of the world gurus in Artificial Intelligence ... Teach yourself Programming in Ten Years[^].

intialisation of charecter array

Coder Block7-May-13 19:54

Coder Block

7-May-13 19:54

hi guys,
If i create one character array of some size like,
char* TempArray[10];
how do i intialise it???

Richard MacCutchan7-May-13 20:45

7-May-13 20:45

That is not a character array, it's an array of pointers. You need something like:

C++

char foo[10] = "string";
// or
char* foo = "string";

Note: Your signature is too big.

Use the best guess

Coder Block7-May-13 20:54

Coder Block

7-May-13 20:54

nop i need array of charecter pointer??
not normal pointer above example that you have given is normal character string i need array of characters string.
And i will change my sign;.

Richard MacCutchan7-May-13 21:56

7-May-13 21:56

Coder Block wrote:
nop i need array of charecter pointer??

That's not what your original question asked for.

C++

char* foo[10] =
{
    { "string1" },
    { "string2" },
    { "string3" },
// ...
    NULL // use to signify end of array if length is not known at compile time.
};

And your signature is still too big; use the standard font size.

Use the best guess

rxylinux8-May-13 4:55

rxylinux

8-May-13 4:55

you can initialise the array just like this
char *TempArray[10] = {NULL};