oh, I didn't realize we needed to instantiate them ourselves, and that
there wasn't an `_new` thing. :+1:


On Sun, Oct 20, 2013 at 8:26 PM, Paul Kehrer <notifications@github.com>wrote:

> In cryptography/bindings/openssl/hmac.py:
>
> > +#    http://www.apache.org/licenses/LICENSE-2.0
> > +#
> > +# Unless required by applicable law or agreed to in writing, software
> > +# distributed under the License is distributed on an "AS IS" BASIS,
> > +# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or
> > +# implied.
> > +# See the License for the specific language governing permissions and
> > +# limitations under the License.
> > +
> > +INCLUDES = """
> > +#include <openssl/hmac.h>
> > +"""
> > +
> > +TYPES = """
> > +struct hmac_ctx_st { ...; };
> > +typedef struct hmac_ctx_st HMAC_CTX;
>
> If you make it fully opaque cffi won't allow it to be instantiated via ffi.new("HMAC_CTX
> *") because you cannot instantiate ctype 'HMAC_CTX' of unknown size
>
> A simpler declaration would be typedef struct { ...; } HMAC_CTX; though.
> I'll switch it to that.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/pyca/cryptography/pull/156/files#r7085246>
> .
>



-- 
"I disapprove of what you say, but I will defend to the death your right to
say it." -- Evelyn Beatrice Hall (summarizing Voltaire)
"The people's good is the highest law." -- Cicero
GPG Key fingerprint: 125F 5C67 DFE9 4084