<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">2017-06-01 9:49 GMT+02:00 Petri Hintukainen <span dir="ltr"><<a href="mailto:phintuka@users.sourceforge.net" target="_blank">phintuka@users.sourceforge.net</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="gmail-">ke, 2017-05-31 kello 12:33 +0200, Andreas Zelend kirjoitti:<br>
><br>
> 2017-05-31 10:29 GMT+02:00 Petri Hintukainen <phintuka@users.sourcefo<br>
> <a href="http://rge.net" rel="noreferrer" target="_blank">rge.net</a>>:<br>
> > I didn't find any limitation for field length. Recorded string<br>
> > length<br>
> > is limited to 255 (because of it is stored in Uint8).<br>
> > This would mean field length can be 256 (or even more if there's<br>
> > padding) ?<br>
> ><br>
><br>
> That's right, the caller has to make sure that he uses the correct<br>
> field length.<br>
<br>
</span>Field lengths are constants from specifications. Currently this is not<br>
an issue, it just feels safer to write code that can handle all<br>
possible values.<br>
<span class="gmail-"><br>
> I'll prepare another patch regarding off_t size problems.<br>
<br>
</span>in libudfread ?<br>
<br>
off_t is used in only one file, maybe you can just define it to<br>
uint64_t or something else. Or add native Windows API versions of the<br>
remaining functions.<br>
<span class="gmail-"><br></span></blockquote><div>Yes, maybe the following:<br><br>From 6ba9f18fbb4cfa9994a7c33d30750d30daff2fc4 Mon Sep 17 00:00:00 2001<br>From: ace20022 <<a href="mailto:ace20022@ymail.com">ace20022@ymail.com</a>><br>Date: Thu, 14 Jan 2016 15:46:43 +0100<br>Subject: [PATCH] Use 64-bit version of lseek and off_t for win32.<br><br>---<br> src/default_blockinput.c | 4 ++++<br> 1 file changed, 4 insertions(+)<br><br>diff --git a/src/default_blockinput.c b/src/default_blockinput.c<br>index 4daa6a3..aaa8c62 100644<br>--- a/src/default_blockinput.c<br>+++ b/src/default_blockinput.c<br>@@ -41,6 +41,10 @@<br> #include <stdio.h><br> #endif<br> #include <io.h><br>+# undef  lseek<br>+# define lseek _lseeki64<br>+# undef  off_t<br>+# define off_t int64_t<br> #endif<br> <br> #ifdef __ANDROID__<br>-- <br>2.12.0.windows.1<br><br></div><div>I've only tested it with msvc, not with mingw. The issue that needs fixing is the right shift at line 67 ov.OffsetHigh = (offset >> 32); <br>which is undefined (nop in my case) if offset is a 32-bit variable.<br></div><div></div><div><br> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="gmail-">
> btw do you plan to tag a release in the near future?<br>
<br>
</span>libudfread ?<br>
We could tag 1.0.0, there haven't been much changes recently.<br></blockquote><div></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
libbluray ?<br>
Probably sooner than later, there's already a pile of fixes since<br>
previous release.<br>
<div class="gmail-HOEnZb"><div class="gmail-h5"><br></div></div></blockquote><div>I've found a crash yesterday when calling bd_get_event between bd_init and bd_open* , since bd->disc is null. Are you interested?<br></div><div><br>Having a new release for both libs would be great.<br></div><div><br> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class="gmail-HOEnZb"><div class="gmail-h5">
> ______________________________<wbr>_________________<br>
> libbluray-devel mailing list<br>
> <a href="mailto:libbluray-devel@videolan.org">libbluray-devel@videolan.org</a><br>
> <a href="https://mailman.videolan.org/listinfo/libbluray-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/<wbr>listinfo/libbluray-devel</a><br>
______________________________<wbr>_________________<br>
libbluray-devel mailing list<br>
<a href="mailto:libbluray-devel@videolan.org">libbluray-devel@videolan.org</a><br>
<a href="https://mailman.videolan.org/listinfo/libbluray-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/<wbr>listinfo/libbluray-devel</a><br>
</div></div></blockquote></div><br></div></div>