ZXing.Delphi icon indicating copy to clipboard operation
ZXing.Delphi copied to clipboard

PDF417 Support

Open adearmasIAP opened this issue 5 years ago • 18 comments

Hello, I have finished PDF417 translation from C# to Delphi. It compiles just fine althought there's a lot of work to do left:

  • First of all Camera demo it does not work beause it cancels with AVs when it detects a PDF417 code. I cleaned up some errors but, obviusly some remain.
  • There will be thousands of memory leaks because I tried to keep it as similar to the original as possible (Hoping to make it work fast but I failed).
  • I used a lot of compiler new goodies and I have to remove them to make ir compatible with older Delphi compilers.

I'd like to know if anyone with more experience than me would like to help to make ir work.

Regards.

adearmasIAP avatar Jan 25 '21 14:01 adearmasIAP

Did you continue developing your translation?

Spelt avatar Apr 25 '21 13:04 Spelt

Hi!... The translation is complete. I've been chasing an AV raised when a QR is detected but I couldn't fix it. The code compiles. I had to change some classes that were not well translated (AFAIK)

Do you have time to help me? It is still written in 10.4.2 ( I guess it compiles well in 10.4.x) because I wanted to keep it as original as the C# code looks like in order to make it work.

Regards


De: E Spelt @.> Enviado: domingo, 25 de abril de 2021 10:18 Para: Spelt/ZXing.Delphi @.> Cc: DE ARMAS ADRIAN ALFREDO @.>; Author @.> Asunto: Re: [Spelt/ZXing.Delphi] PDF417 Support (#115)

Did you continue developing your translation?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/Spelt/ZXing.Delphi/issues/115#issuecomment-826323385, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKDF76PS2UD57BNBFI46GN3TKQJEFANCNFSM4WR3NFZQ.

adearmasIAP avatar Apr 25 '21 14:04 adearmasIAP

Short term, I have zero time to spare. Long term I wil help but that won’t help you now.

On 25 Apr 2021, at 16:16, adearmasIAP @.***> wrote:

Hi!... The translation is complete. I've been chasing an AV raised when a QR is detected but I couldn't fix it. The code compiles. I had to change some classes that were not well translated (AFAIK)

Do you have time to help me? It is still written in 10.4.2 ( I guess it compiles well in 10.4.x) because I wanted to keep it as original as the C# code looks like in order to make it work.

Regards


De: E Spelt @.> Enviado: domingo, 25 de abril de 2021 10:18 Para: Spelt/ZXing.Delphi @.> Cc: DE ARMAS ADRIAN ALFREDO @.>; Author @.> Asunto: Re: [Spelt/ZXing.Delphi] PDF417 Support (#115)

Did you continue developing your translation?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/Spelt/ZXing.Delphi/issues/115#issuecomment-826323385, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKDF76PS2UD57BNBFI46GN3TKQJEFANCNFSM4WR3NFZQ. — You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/Spelt/ZXing.Delphi/issues/115#issuecomment-826331901, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADAS3VKSLGT5SVU5VMHPUNLTKQP2ZANCNFSM4WR3NFZQ.

Spelt avatar Apr 25 '21 16:04 Spelt

It is OK. I made the translation when I had the time. It is not a functionality that I need for my own, I just wanted to collaborate with something that needed a good count of hours. Every now and then, when I have the time, I keep chasing that AV.

Regards

adearmasIAP avatar Apr 25 '21 16:04 adearmasIAP

@adearmasIAP Do you have any prototype source files with your PDF417 implementations?

Tajmi avatar Nov 01 '21 10:11 Tajmi

Hello @Tajmi . I have the source code that I translated but it doesn't work at all. The project I was working bought some PDF417 reading devices so I had no need to keep investigating the implementation.

If anyone wants to take a look I will happily upload it.

adearmasIAP avatar Nov 01 '21 12:11 adearmasIAP

Hi @adearmasIAP I would realy like to look at it and try to fix it. Can you contact me on [email protected] or upload the source files anywhere? Thank you.

Tajmi avatar Nov 01 '21 16:11 Tajmi

Hello @Tajmi I have uploaded my code. I needed an implementation of bigintegers so I used http://rvelthuis.de/programs/bigintegers.html

The code does not work, it raises an AV. There's a lot of work to do: 1.Make it work 2.To keep backward compability lots of inline variables should be removed 3.Memory leaks... I tried to translate the code without changes (that's the reason of inline variables) and that produced lots of ML. My idea was remove them before uploading the result.

https://drive.google.com/file/d/1a1GCIu4Btke0wBXTDSPtjOOs9dW1hkRr/view?usp=sharing

Hope you can make it work

adearmasIAP avatar Nov 04 '21 22:11 adearmasIAP

Did anyone could make PDF417 decode work?

wilchaia avatar Jun 08 '22 14:06 wilchaia

Yes I kinda make it. But it was hell work.

Tajmr avatar Jun 08 '22 14:06 Tajmr

I got it running on delphi XE4, but it is still quit buggy :/

Tajmr avatar Jun 08 '22 14:06 Tajmr

Is it possible to get the code and see if I can help?


De: Tajmr @.> Enviado: miércoles, 8 de junio de 2022 11:34 Para: Spelt/ZXing.Delphi @.> Cc: DE ARMAS ADRIAN ALFREDO @.>; Mention @.> Asunto: Re: [Spelt/ZXing.Delphi] PDF417 Support (#115)

I got it running on delphi XE4, but it is still quit buggy :/

— Reply to this email directly, view it on GitHubhttps://github.com/Spelt/ZXing.Delphi/issues/115#issuecomment-1150002549, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKDF76KHZ326JARW3ID7DULVOCVOXANCNFSM4WR3NFZQ. You are receiving this because you were mentioned.Message ID: @.***>

adearmasIAP avatar Oct 11 '22 09:10 adearmasIAP

I can post the code at the evening somewhere. I am no longer maintaining it. But if it can help anyone else.

Tajmi avatar Oct 11 '22 10:10 Tajmi

Yes please


De: Tajmi @.> Enviado: martes, 11 de octubre de 2022 7:10 Para: Spelt/ZXing.Delphi @.> Cc: DE ARMAS ADRIAN ALFREDO @.>; Mention @.> Asunto: Re: [Spelt/ZXing.Delphi] PDF417 Support (#115)

I can post the code at the evening somewhere. I am no longer maintaining it. But if it cant help anyone else.

— Reply to this email directly, view it on GitHubhttps://github.com/Spelt/ZXing.Delphi/issues/115#issuecomment-1274453455, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKDF76KXNBSUPWAWAZB4VFLWCU4JBANCNFSM4WR3NFZQ. You are receiving this because you were mentioned.Message ID: @.***>

adearmasIAP avatar Oct 11 '22 10:10 adearmasIAP

Unfortunately I cannot help due to lack of knowledge. But I need PDF417. Thanks. Thanks. Thank you very much

balger avatar Nov 09 '22 23:11 balger

Hi Tajmi

Can you post the code?

Spelt avatar Nov 18 '22 23:11 Spelt