Avro
  1. Avro
  2. AVRO-418

avro.h generates errors when included in C++ code

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.3.2
    • Component/s: c
    • Labels:
      None

      Description

      avro.h cannot be included in C++ code without errors

      1. AVRO-418.patch
        2 kB
        Matt Massie
      2. avro_build_with_cpp.diff
        12 kB
        Bruce Mitchener

        Activity

        Hide
        Matt Massie added a comment -

        Patch provided by Eric Andersen. Thanks!

        Show
        Matt Massie added a comment - Patch provided by Eric Andersen. Thanks!
        Hide
        Doug Cutting added a comment -

        Matt, what's the rationale for resolving this? Isn't it still a valid issue?

        Show
        Doug Cutting added a comment - Matt, what's the rationale for resolving this? Isn't it still a valid issue?
        Hide
        Matt Massie added a comment -

        Keeping this issue open since it's still an issue that needs to be addressed

        Show
        Matt Massie added a comment - Keeping this issue open since it's still an issue that needs to be addressed
        Hide
        Matt Massie added a comment -

        This patch can be used as a workaround for this problem. We need someone better versed in C++ than myself to help design a correct solution to this problem.

        Show
        Matt Massie added a comment - This patch can be used as a workaround for this problem. We need someone better versed in C++ than myself to help design a correct solution to this problem.
        Hide
        Bruce Mitchener added a comment -

        I think this is a better approach to addressing the problems ... it also includes a test program that will fail to compile if the other fixes aren't in place. I tested the test under the CMake build but NOT under autotools.

        Show
        Bruce Mitchener added a comment - I think this is a better approach to addressing the problems ... it also includes a test program that will fail to compile if the other fixes aren't in place. I tested the test under the CMake build but NOT under autotools.
        Hide
        Matt Massie added a comment -

        Thanks for the patch, Bruce. Committed to trunk.

        Show
        Matt Massie added a comment - Thanks for the patch, Bruce. Committed to trunk.

          People

          • Assignee:
            Bruce Mitchener
            Reporter:
            Matt Massie
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development