Im maknig macro format ..

Question

0.00/5 (No votes)

See more:

I make define header file.

The format make error..

C++

#define LOG_FORMAT(id, level, logformat, ...) \
{\
    if (zsummer::log4z::ILog4zManager::GetInstance()->PrePushLog(id,level)) \
    {\
       if(zsummer::log4z::ILog4zManager::GetInstance()->GetChangeFileStatus()==1)\
       {\
         zsummer::log4z::ILog4zManager::GetInstance()->PushLog(id, level, "Title");\

        }\

        char logbuf[LOG4Z_LOG_BUF_SIZE];\
        int ret = _snprintf_s(logbuf, LOG4Z_LOG_BUF_SIZE, _TRUNCATE, logformat, ##__VA_ARGS__);\
        zsummer::log4z::ILog4zManager::GetInstance()->PushLog(id, level, logbuf);\

    }\
 }

Posted 31-Aug-14 19:23pm

Sun-Mi Kang

Updated 1-Jun-15 17:41pm

v2

Add a Solution

Comments

Gihan Liyanage 1-Sep-14 1:38am

I do not familiar with this type of code. But I just think if you could have provided the error, then people may understand the problem clearly and you will be get the answer quickly.

[no name] 1-Sep-14 3:00am

1. You have not told us where or what the error is (please don't).
2. Your approach is wrong. What is need for macro? Why not use a static function and allow the compiler to do it's work.
http://stackoverflow.com/questions/14041453/why-are-preprocessor-macros-evil-and-what-are-the-alternatives

Richard MacCutchan 1-Sep-14 3:25am

That is really bad practice, use a proper function and make calls in the code.

Mohibur Rashid 1-Sep-14 3:42am

what is the definition of PushLog function
?

Sun-Mi Kang 1-Sep-14 4:28am

sorry bad question..
I remove this code.

And make function..

Sergey Alexandrovich Kryukov 1-Jun-15 23:20pm

Again, same thing: this is not C++/CLI. Please stop mystifying people. This is abuse, after all.
—SA

Sun-Mi Kang 1-Jun-15 23:41pm

Ok.. Sorry.!

Sergey Alexandrovich Kryukov 2-Jun-15 0:24am

All right. Please pay attention and use correct tags with your questions.
—SA

1 solution

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

nv3 · Accepted Answer · 2014-08-31T21:48:00

Solution 1

Note that you are not allowed to have empty lines in your macro. They also need to be terminated with a backslash. Otherwise they terminate the macro.

If this is not the obvious error, please let us know the exact error messages you get.

Posted 31-Aug-14 21:48pm

nv3