[c++]const 멤버를 가지고 있는 구조체 초기화에 대하여..

leboum의 이미지

AudioHardwareALSA.h
---------------------------------------------

struct alsa_properties_t
{
    const AudioSystem::audio_devices device;
    const char         *propName;
    const char         *propDefault;
    mixer_info_t       *mInfo;
};
 
#define ALSA_PROP(dev, name, out, in) \
    {\
        {dev, "alsa.mixer.playback." name, out, NULL},\
        {dev, "alsa.mixer.capture." name, in, NULL}\
    }
 
 
class ALSAMixer
{
    public:
        ALSAMixer(const char *sndcard = "hw:00");
        status_t                getPlaybackMuteState(uint32_t device, bool *state);
 
    private:
        snd_mixer_t *           mMixer[SND_PCM_STREAM_LAST+1];
        alsa_properties_t       mixerMasterProp[SND_PCM_STREAM_LAST+1];
        alsa_properties_t       mixerProp[SND_MIXER_OUT_DEVICE][SND_PCM_STREAM_LAST+1];
};
 
 
====ALSAMixer.cpp====
    namespace android_audio_legacy
    {
       ...
 
106: ALSAMixer::ALSAMixer(const char *sndcard)
107: {
108:    int err;
109:    LOGI(" Init ALSAMIXER for SNDCARD : %s",sndcard);
110:
111:    mixerMasterProp =
112:        ALSA_PROP(AudioSystem::DEVICE_OUT_ALL, "master", "PCM", "Capture");
113:
114:    mixerProp = {
115:        ALSA_PROP(AudioSystem::DEVICE_OUT_EARPIECE, "earpiece", "Earpiece", "Capture"),
116:        ALSA_PROP(AudioSystem::DEVICE_OUT_SPEAKER, "speaker", "Speaker",  ""),
117:        ALSA_PROP(AudioSystem::DEVICE_OUT_WIRED_HEADSET, "headset", "Headphone", "Capture"),
118:        ALSA_PROP(AudioSystem::DEVICE_OUT_BLUETOOTH_SCO, "bluetooth.sco", "Bluetooth", "Bluetooth Capture"),
119:        ALSA_PROP(AudioSystem::DEVICE_OUT_BLUETOOTH_A2DP, "bluetooth.a2dp", "Bluetooth A2DP", "Bluetooth A2DP Capture"),
120:        ALSA_PROP(static_cast<AudioSystem::audio_devices>(0), "", NULL, NULL)
        };
 
     .....
 
    } 

hardware/alsa_sound/ALSAMixer.cpp: In constructor 'android_audio_legacy::ALSAMixer::ALSAMixer(const char*)':
hardware/alsa_sound/ALSAMixer.cpp:106:41: error: uninitialized const member in 'struct android_audio_legacy::alsa_properties_t' In file included from hardware/alsa_sound/ALSAMixer.cpp:35:0:
hardware/alsa_sound/AudioHardwareALSA.h:112:38: note: 'android_audio_legacy::alsa_properties_t::device' should be initialized
hardware/alsa_sound/ALSAMixer.cpp:106:41: error: uninitialized const member in 'struct android_audio_legacy::alsa_properties_t' In file included from hardware/alsa_sound/ALSAMixer.cpp:35:0:
hardware/alsa_sound/AudioHardwareALSA.h:112:38: note: 'android_audio_legacy::alsa_properties_t::device' should be initialized
hardware/alsa_sound/ALSAMixer.cpp:112:9: error: assigning to an array from an initializer list
hardware/alsa_sound/ALSAMixer.cpp:121:5: error: assigning to an array from an initializer list
target StaticLib: libmedia_helper (out/target/product/xxxxxxx/obj/STATIC_LIBRARIES/libmedia_helper_intermediates/libmedia_helper.a)
make: *** [out/target/product/xxxxxxx/obj/SHARED_LIBRARIES/libaudio_intermediates/ALSAMixer.o] Error 1

기존에 동작하던 안드로이드 ICS 버전 소스를 제리빈버전으로 포팅 중에있는데요.
지금까지 잘 동작하던 소스에서 에러가 나네요.
JB AOSP소스에 들어있는 최신 툴체인을 이용하였는데, 이 툴체인에서는 에러로 처리를 하는것 같습니다.

제가 생각하는 원인은 구조체 alsa_properties_t의 const 멤버 변수들을 초기화 하지 않아서 생기는 문제라고 보여지는데요.
C++를 잘 모르는 상황인지라, 어떻게 수정을 해야할지 모르겠네요.

조언 부탁드리겠습니다.

HDNua의 이미지

C++에서 구조체는, 기본 영역이 public인 클래스와 다를 것이 없으므로
생성자를 정의하면서 멤버 이니셜라이저에서 초기화해주면 됩니다.
포인터의 경우 인자가 있다면 인자로, 없다면 초기 값으로 0을 넣어주는 방식으로요.

그리고 에러 메시지를 보니... 이니셜라이저에서 필드 초기화 시
필드에 배열처럼 값을 대입하려는 부분이 있는 것 같습니다.

자세한 내용은 잘 모르겠으나 ALSAMixer.cpp 파일의 112, 121줄 등을 확인해보셔야할 것 같아요.

저는 이렇게 생각했습니다.