bugVarnamproject - Bugs: bug #41252, [libvarnam] Remove snprintf defines

 
 

bug #41252: [libvarnam] Remove snprintf defines

Submitter:  Navaneeth <navaneethkn>
Submitted:  Thu 16 Jan 2014 04:03:09 PM UTC
   
 
Category:  libvarnam Severity:  4 - Important
Item Group:  * Bug Status:  None
Privacy:  Public Assigned to:  None
Open/Closed:  Open
* Mandatory Fields

Add a New Comment Rich Markup
   

Thu 20 Mar 2014 04:43:24 AM UTC, comment #2: 

Varnam uses a portable snprintf implementation available at https://gitorious.org/varnamproject/libvarnam/source/455a2a5df151d93ace5f65de655efd6532e3e0a6:deps/snprintf.c.

To maintain compatibility with platforms which has snprintf, the authors of that library decided to use some MACROS which can be used to toggle the portable snprintf implementation. So systems which has support for it can just use the system version and those who don't will get the portable implementation.

Now this requires you define some MACROS while compile programs which uses Varnam. We don't have to expose this because this is libvarnam's implementation detail.

Try running

pkg-config --cflags varnam

Which will give you an output like,

-DHAVE_SNPRINTF -DPREFER_PORTABLE_SNPRINTF -DNEED_ASPRINTF -I/usr/local/include/libvarnam

This bug talks about removing the HAVE_SNPRINTF and other snprintf related defines so that everyone who needs to use varnam need not define all these. You can't simply remove it because they are in use. So need to find out places where this is in use, define these manually in the compilation unit and remove it from pkg-config flags.

Navaneeth <navaneethkn>
Group administrator
Thu 20 Mar 2014 04:07:50 AM UTC, comment #1: 

Where are these macros defined?

Kevin Martin Jose <lonesword>
Thu 16 Jan 2014 04:03:09 PM UTC, original submission:  

Currently to compile the code, you need to define SNPRINTF macros. This should be simplified

Navaneeth <navaneethkn>
Group administrator

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by lonesword (Posted a comment)
  • -email is unavailable- added by navaneethkn (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    No changes have been made to this item

    Back to the top

    Powered by Savane 3.13-3230.
    Corresponding source code