‘memcopy’ was not declared in this scope

Question

#include <iostream>
#include <cstring>
#include <cstdlib>



using namespace std;
void printArray(int* arr, int size) {
    cout  << "Printing the array..." << endl;
    for (int index = 0; index < size; index++) {
        cout << arr[index] << endl;
    }
}

void populateArray(int* arr, int size) {
    for (int index = 0; index < size; index++) {
        arr[index] = index * 10 + 1;
    }
}

int main() {
    int size = 2;
    int* arr = new int[size];   
    populateArray(arr, size);
    size_t newSize = size * 2;
    int* newArr = new int[newSize];
    memcopy(newArr, arr, size);
    populateArray(newArr, newSize);
    return EXIT_SUCCESS;
}

Even after including cstring , I am getting 'VarArr.cpp:25:27: error: ‘memcopy’ was not declared in this scope' error while compiling. What am I missing here ?

When C was created they couldn't use long names: you'll need to remove the `o` from `memcopy()`. — Dietmar Kühl, Dec 17 '14 at 03:53
Screw me up. That was the error . Bit embarrassing. C/C++ is rusty after a long time of java programming — sakthisundar, Dec 17 '14 at 03:55
If your compiler is smart enough, it'll tell you. `error: use of undeclared identifier 'memcopy'; did you mean 'memcpy'?` — , Dec 17 '14 at 03:56

score 6 · Accepted Answer · answered Dec 17 '14 at 03:54

6

The correct name is memcpy, not memcopy.

answered Dec 17 '14 at 03:54

Shoe

74,840
36
166
272

score 4 · Answer 2 · answered Dec 17 '14 at 04:00

4

You made a typo and meant to use memcpy, but you should prefer to use the C++ facilities instead.

std::copy(arr, arr + size, newArr);

You seem to be forgetting to delete your new'd arrays, which isn't a problem since the program ends right there, but it's a good habit to remember to do this.

Let the standard library manage your memory for you. Use a container like std::vector or a smart pointer like std::unique_ptr.

answered Dec 17 '14 at 04:00

Thank you for the suggestions. Getting back to C++ after a long time. Just checking it with raw arrays rather going with vectors. But std::copy is a learning – sakthisundar Dec 17 '14 at 04:03
This answer is better than @jefffrey's. – caps Dec 17 '14 at 05:51

‘memcopy’ was not declared in this scope

2 Answers2