diff options
Diffstat (limited to 'pipermail/pycrypto/2011q1/000407.html')
| -rw-r--r-- | pipermail/pycrypto/2011q1/000407.html | 110 |
1 files changed, 110 insertions, 0 deletions
diff --git a/pipermail/pycrypto/2011q1/000407.html b/pipermail/pycrypto/2011q1/000407.html new file mode 100644 index 0000000..63dcdc9 --- /dev/null +++ b/pipermail/pycrypto/2011q1/000407.html @@ -0,0 +1,110 @@ +<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2//EN"> +<HTML> + <HEAD> + <TITLE> [pycrypto] Initial review of Thorsten's Py3k changes + </TITLE> + <LINK REL="Index" HREF="index.html" > + <LINK REL="made" HREF="mailto:pycrypto%40lists.dlitz.net?Subject=%5Bpycrypto%5D%20Initial%20review%20of%20Thorsten%27s%20Py3k%20changes&In-Reply-To=4D2C773B.8060605%40gmx.li"> + <META NAME="robots" CONTENT="index,nofollow"> + <META http-equiv="Content-Type" content="text/html; charset=us-ascii"> + <LINK REL="Previous" HREF="000395.html"> + <LINK REL="Next" HREF="000408.html"> + </HEAD> + <BODY BGCOLOR="#ffffff"> + <H1>[pycrypto] Initial review of Thorsten's Py3k changes</H1> + <B>Legrandin</B> + <A HREF="mailto:pycrypto%40lists.dlitz.net?Subject=%5Bpycrypto%5D%20Initial%20review%20of%20Thorsten%27s%20Py3k%20changes&In-Reply-To=4D2C773B.8060605%40gmx.li" + TITLE="[pycrypto] Initial review of Thorsten's Py3k changes">gooksankoo at hoiptorrow.mailexpire.com + </A><BR> + <I>Wed Jan 19 04:26:06 CST 2011</I> + <P><UL> + <LI>Previous message: <A HREF="000395.html">[pycrypto] Initial review of Thorsten's Py3k changes +</A></li> + <LI>Next message: <A HREF="000408.html">[pycrypto] Initial review of Thorsten's Py3k changes +</A></li> + <LI> <B>Messages sorted by:</B> + <a href="date.html#407">[ date ]</a> + <a href="thread.html#407">[ thread ]</a> + <a href="subject.html#407">[ subject ]</a> + <a href="author.html#407">[ author ]</a> + </LI> + </UL> + <HR> +<!--beginarticle--> +<PRE>I am not sure if I grasp all new stuff in Python 3, but would it possible to +keep the test vectors as they are, and possibly change the encoding at runtime? + +For instance, in test_AES.py, we have: + +test_data = [ +... +( '00112233445566778899aabbccddeeff', 'dda97ca4864cdfe06eaf70a0ec0d7191', + '000102030405060708090a0b0c0d0e0f1011121314151617', + 'FIPS 197 C.2 (AES-192)' ), +... ] + +The proposed change wraps each string but the last one with b(). + +Instead of changing the test vector line, we could have a separate +piece of code that dynamically +adjusts a tuple in the list in case python 3 is detected. + +Something along these lines, but possibly more pythonic: + +if python_version>2: + for t in test_data: + idx = test_data.index(t) + newt = tuple(b(t[0), b(t[1]), b(t[2]), t[3]) + test_data[idx] = newt + + +2011/1/11 Thorsten Behrens <<A HREF="http://lists.dlitz.net/cgi-bin/mailman/listinfo/pycrypto">sbehrens at gmx.li</A>>: +><i> Dwayne, +</I>><i> +</I>><i> upon further thought, how would the following work for you to address +</I>><i> your concerns: +</I>><i> +</I>><i> - b('something') vs 'something': Leave literals encased in b(). To +</I>><i> assure that no malicious changes have been made, use a 30-day trial +</I>><i> version of Beyond Compare 3, which will, in it's "Text Compare" +</I>><i> function, highlight the exact changes on each line. It exists for Linux +</I>><i> and Windows. vimdiff may work just as well, of course. +</I>><i> <A HREF="http://www.scootersoftware.com/download.php">http://www.scootersoftware.com/download.php</A> +</I>><i> +</I>><i> - Multiple commits, too much "history of work", not enough "this is the +</I>><i> change": Would you like me to create a py3kmk2 branch, and have a single +</I>><i> commit in it with all the changes made? +</I>><i>   I don't think I am going to be very successful with rebase. +</I>><i> +</I>><i> Yours +</I>><i> Thorsten +</I>><i> +</I>><i> _______________________________________________ +</I>><i> pycrypto mailing list +</I>><i> <A HREF="http://lists.dlitz.net/cgi-bin/mailman/listinfo/pycrypto">pycrypto at lists.dlitz.net</A> +</I>><i> <A HREF="http://lists.dlitz.net/cgi-bin/mailman/listinfo/pycrypto">http://lists.dlitz.net/cgi-bin/mailman/listinfo/pycrypto</A> +</I>><i> +</I></PRE> + + + +<!--endarticle--> + <HR> + <P><UL> + <!--threads--> + <LI>Previous message: <A HREF="000395.html">[pycrypto] Initial review of Thorsten's Py3k changes +</A></li> + <LI>Next message: <A HREF="000408.html">[pycrypto] Initial review of Thorsten's Py3k changes +</A></li> + <LI> <B>Messages sorted by:</B> + <a href="date.html#407">[ date ]</a> + <a href="thread.html#407">[ thread ]</a> + <a href="subject.html#407">[ subject ]</a> + <a href="author.html#407">[ author ]</a> + </LI> + </UL> + +<hr> +<a href="http://lists.dlitz.net/cgi-bin/mailman/listinfo/pycrypto">More information about the pycrypto +mailing list</a><br> +</body></html> |