buglwIP - A Lightweight TCP/IP stack - Bugs: bug #43103, conditional compilation about ipv6...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #43103: conditional compilation about ipv6 header file

Submitted by:  yuanbin <ffddybz>
Submitted on:  Fri 29 Aug 2014 02:55:49 AM UTC  
 
Category: IPv6Severity: 3 - Normal
Item Group: Change RequestStatus: Invalid
Privacy: PublicAssigned to: Ivan Delamer <idelamer>
Open/Closed: ClosedPlanned Release: None
lwIP version: git head

Fri 29 Aug 2014 04:12:27 PM UTC, comment #1:

The headers have conditional compilation included, so if you disable IPv6 there will be no additional code compiled in.

It is standard LwIP practice to put conditional markers inside the header files, vs. wrapping the include directive.

You can use your patch in your code if you don't want to keep the .h files in your project environment.

I suggest you can post this in the patch section if you think it may be useful for others, but it is not a bug.

Cheers
Ivan

Ivan Delamer <idelamer>
Project MemberIn charge of this item.
Fri 29 Aug 2014 02:55:49 AM UTC, original submission:

I use git head because of the support of ipv6 , but in many occasions, I only want to use ipv4. So I hope that it does not compile any source file and header file about ipv6. But in many files it did not use conditional compilation to handle ipv6 header. So, I submit the patch.
And also, when will it release lwip-1.5.0?

yuanbin <ffddybz>

 

Attached Files
file #31996:  ipv6.patch added by ffddybz (8KiB - application/octet-stream)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by idelamer (Posted a comment)
  • -unavailable- added by ffddybz (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 4 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Fri 29 Aug 2014 04:12:27 PM UTCidelamerStatusNone=>Invalid
      Assigned toNone=>idelamer
      Open/ClosedOpen=>Closed
    Fri 29 Aug 2014 02:55:49 AM UTCffddybzAttached File-=>Added ipv6.patch, #31996

    Back to the top


    Powered by Savane 3.1-cleanup1