macro has problem