Sukender

In the root CMakeLists.txt there is this code

IF(WIN32)
  SET(CURL_LDAP_HYBRID OFF)
  OPTION(CURL_LDAP_WIN "Use W$ LDAP implementation" ON)
  MARK_AS_ADVANCED(CURL_LDAP_WIN)
ELSE()
  OPTION(CURL_LDAP_HYBRID "W$ LDAP with non-W$ compiler" OFF)
  MARK_AS_ADVANCED(CURL_LDAP_HYBRID)
  SET(CURL_LDAP_WIN OFF)
ENDIF()


I think that for consistency that should be rather

IF(WIN32)
  OPTION(CURL_LDAP_WIN "Use W$ LDAP implementation" ON)
  MARK_AS_ADVANCED(CURL_LDAP_WIN)
  SET(CURL_LDAP_HYBRID OFF)
ELSE()
  OPTION(CURL_LDAP_HYBRID "W$ LDAP with non-W$ compiler" OFF)
  MARK_AS_ADVANCED(CURL_LDAP_HYBRID)
  SET(CURL_LDAP_WIN OFF)
ENDIF()


Sorry for this nitpicking but that's my way of thinking. I'm trying to
point out everything that seems to be not right no matter how important
it is. :)


-- 
Piotr Dobrogost
*** curlpp.org - c++ wrapper for libcurl ***

Reply via email to