Page 1 of 1

Some fixes to pythonmagick

Posted: 2011-08-11T04:27:53-07:00
by cbusse
I have made a patch with some fixes for Problems with class inheritance for the Drawable* Classes in pythonmagick.

My problem now is, who I need to send them to for reviewing?

for those burning to be able to use the Drawable classes inside pythonmagick and like to try it out themself, here the uuencoded gzip of the diff file:

Code: Select all

begin 644 patch-cbusse.diff.gz
M'XL("$NZ0TX``W!A=&-H+6-B=7-S92YD:69F`+5<VW+<N!%]UU=,^<$E61?O
M6BH[I76V[)6=W:3DLF(KVLS3%,6A1HPIDD52EL>I_'L`$(,+@>X&!K(?5+OL
MTZ?!/@`(-#$\/#R<9<\OUL-M4W_(5F7^Y?EB4=;EL%@<M>N=_?W]V35L?O-F
M=OCSSP?'LWWV]\7)R]F;-SNSS;^BZHM3_;_\W\)D.OJM:JZ/-G2[?5'='#S+
MNE6_-SHMLR'[:]LU;=$-ZUW;=54,BVOFON"@`P_M?<LLQ=[._LY^7F5]/SMK
MJJ:;L(AK>Z<[^RQ:RT`F^JYM^G(H/K+HV>#QG-A!EJ[H^[*I+]=MX>$PK`!#
MTRW+FMV*X[PQ>/W>%7G#VN4/;!O]_EWVD%U7Q=N;F[)V_"TC[M_ED'.7HYZ_
M%=_+8IIVVXCZGY5=7D$M'XVX?U6V%]EP"S%(,\[AZ7.6C?"6?>SO=]D*O!,+
MA/*]R_K;MUV73<>28R=9/M[<],6`T(P`E.=]Q3+80[<EK2C#W\JJPC*L["3+
MQS;+2V>.\2!(ID_W8)_;F'&.IH;RRDVH[^]=]A6^"VE%&<[AL7Y.C?0/V>!,
M4I8-]V8]N#LO[TKH[C4`Y4%&+#E:+YH23+ZPT=Z?R^]0#I2=8*G6JZ8&.825
M9*A@'3=F@J,EIC\#03&QCM?>EOD9Z[[%-SB_$QS%>L&[5`<G:@/`>>[[6^I&
M#0C)%7:K#I#D)6Y6(U"F3T4^9/4*G*"4'6=I!K%T@$BDF>"XKY=D<RP0RO<Y
MSUB?7@%$THHS?"D>_@WY<QOI/4>\Y[CWT#5?BK?U4+)V9CW$8Z,"&+%GHX$(
M8.*3_UG6HEP2$\CV#S8;DG0<%,"'/[TM3`#;G^42G`P,!,IT"8_^2VJ\<P#5
M%RP,R::7^PB=!I%\_ZJ718=U+AN$\W59W5=HXS0"9;HJBX?KYAO`(JU>AO??
M\J+UM$!=]WJQY1R;<?D&:JJ28?%Z_EXT=\7033OLYK+?9UR]>39SAL7KR1]<
M;*_U]GK:3&U`_=A^''#D.W7$\U-1^1V9`?0[JYK>MXBS;+#W??>U&!K_O6HC
MZ>^_9\-*,?CO71M!?S[M03>@;(3W'TU7?F<KBZS">"Q4,*/_QCPH@A'CH;VO
MBFXH<_S^#$P@&]8F`P.R?6C@SJ=LA+>_#<H&>O_S/EORZ3M'AX`'%<[H'Q0^
M6#"G_VX]*)#Q\UW3$.-^"@GC\C=M"B&XPE0!L9'L6).#<UI^*RJGD>*B%\\?
M.Q_84ZN<EOFTP>MWY9GCKZ9S^Z&H3;<"<R<PB[[+GR_L^N-1WK:R3AT$Y37K
MD[\<O)SMB[]FQ?IH6=SL/AG63PYFNU^;<CG;'9MV>FKSG)X^V]O-F[H?9LOF
MGEW>F^T]A:!LU>F/,+JB,?9F(DH`^2\[ASO[Y5U;E6R=6ZT7S.\KG[,8]K7?
M^6!Z^=?=O5]V9O_;F=&)[_*PK$N<2/F)2/F)-^5%O7Q7K+JBZ)'4=[G("9)Q
MCM!43MKM*&#Z91PX]]XP40)T>4+VQ^)WD``&E&MP_))K(/YN-!##<?%Z-FW.
MZ'D@KT\(%W]V6=L6W>S7W2>VA266O\YY/>9.L>JW%>=E/SQE?GM3=7QN-K?I
M%I-M>2O;)WQ\6Q"4<`,J.OU/HM/_Y.WT+('EW1SN\",7T><WH)',Z?(Z!M3=
MC2A@C_<&B9%@)$B10);@PD0PP:+?B^G^V#_=YPR]:!D<44(2&O-^/RQ/3WOV
M?*M73[VZ*!?%[XAC138(B0;`,H$QH[22+"EJ\7U_F%0*2>K49F4]L"7%;;-$
ME.)\(DNCY:SI6+^]T*Y>K48G(X`[CNS@`#G6'E@U*'J4:)PD23'S-6*@=(X+
MU_#5"ZZA^.MJ./)@\IF<(F_&N/!K9WN,$1S]=&!\G+GQHX/&R682)>BGWMT&
M26>CJ0>5^62^K_MR51=+]I0?GL4^QE78;9_DBB`Q4^/KZ>!4&7`Q2XF5[+%_
M)=L(+-S#-5_0)L*$C]1.U]81H4?\)"8X$X'!8C4:21)$DJ__@Q0RL>1&(^OR
M]S7R_)!DQ*)+H48Z1Q(=!9+$C`/JX0\3(X9D2%!"':((TL)&TTLO#H7%4&S&
M4-$["G;=N_(RO`2_N^S:1+6XJ/"@2D"\&)D41:)0\BU7L%0FGI[91C`NEV0D
MQH^%E+SNO*;C0:-H&A'5"`@8JY.D252*'P0*EDF!28UN)!H7B2.<(34N9M4A
M)4"UT743QM',C.^C)=J#ZN<-':L>=TV1KJG#E@T**)Y(8DE\XE\2WS`D(A>S
M1NPZ1SBG=*61<=#UKPH'*S&-$*4`<TO(OGS5&22`B26'S6H$PS)(-F=C:;Y\
M]<BAW&0`1Q0C,$"+MP64"8@<(Y:D2-#K/+0Z?V[5YK&:65$OD8K9^:9N#C]Z
M1@BG\96'T4K9.5F5=\AC\GV>5I,79RR#LJV1CU-S$7RQ-1?I]"-J+KH]H%!@
M]!C!!$F*8NI(:YAL-IR<U.XXON)X1#K%*?)E[NZ]HAEP3>\66ZS()BG9`E@Q
M+'24:HHH0;K@4K1=A@Y__2(*L-.7+_RB[]7+Q5A"]I5>KL8C#%N\<^&.V]9I
M$LO'XJAT6'H5$LBO[I3(TUVP$,\-B7$?X>@&13.#'7M"')5F[IJ:9WXD/3S7
M"DU./^T&3N2=0T)R/^(4J_NT,..A>JB(N";^@-'Z<-\DC<21_T"%-#9VOAE=
MW2EGO.Z==4;3CWGC*\FWGH#D[:3EO0I=OUK@+3+/?;VIYP8H]]SVXY+/V5.R
M7Z6M9HW?EP0JT+IO?<6V^WBS[?XOI8-B.)A=-TW/)N6ZJ?.F77.S1QZ%]RND
MS$JDK=/9/L([6><7-J%Y]7AME5V;)S3'MA>0:1OT"/F>M#4IZ_*G.:'I-N%;
MY5D2A"98PH',2NLCI'33K(1<&C_%"LOFU"%V:C;\W=091F_R#/MDFC:K>-%S
MLT&[]?QLWE>:'MM,*7ZW+?JZ0Q36YQTW2+Y'GE;<YJ;E/FIBF>"W&`EJ7O'D
M$YM$M!E8K7B0Y+`PA]#!K&KJ%2\OX/^5H%OZY*5^R!@DEXTFJZ)5\U!TG\K5
M[8`41Q4GL=LR<`:OL]^:Q(1V7'94<,>%!HU12Q&E:"5_PQHFE0DF=\6B:8A&
MDHR22,'&W\:"44!5S#BP*-XP46I(AB0QS)\!!TKBN)`'0_*FJXONCX+W.TP?
MDYE4R0:;(3R'$NSXL')."Q#]R`;$B6G2)4@J?Y`=I*6)3:GQ21Y",H6*K/.9
M[*`<#GE,\J5S2M;YS]C#<JZ0J?.98*)R/F*VG,ET!#CO;H"HS'/WQ+S/@_,^
M?[2\SP/R/D_*^YS.^SPM[_.4O-L?3`A3P.-#'[JILA4BA4TI4L9V*957DBF6
M4[LG.60\SA(6#Y8("1BEE<V3K%KXB<,I/O7,H<$7=>K0\DL[=SAM`J%>XME#
M@R19-OD-D`CA3`]RH/&:<IZUE'B2TSF1L/E"":B><I2!W#V/;H"'.*0]A)9`
M"^+UE$2/HBC_#$NDI,HE2-/_,'2(J)S5JZKX4`PJZ^BZ">85=M,*'WE0JP*T
M]39C.W4Y5;*\,0>&78_T(\,6)[5>L;$)QX;=J(1TCW!TV")*UDU\@BA"-8TG
M-7O@4$HQP1>DET0*5D<K%0M72D<C=/($B]=(D"0H=!E:"+\T2]\GKT1QXI7_
MBP1C^1F0A/-$G!$>X>*[<U`<](RP"@>*X42($>$RK3!N?1HK6`;/BA];0^(K
M?HL07>]/D%NL]MU8J"KI*WV+)5$G_<VQ8*$F+J122X7']=*\S@)C\K5D0$:3
M0`=UU+3;`X<):B4J-=J@6,4U6:+D^K-PP9)/7%(W>#9EU!YOZIJVS?,T!!44
MBARKI>9)T5)_E2],R`D^I91K<!%+$`L96=*=1H&U\06)$D43)"@BOW`8I(:)
M)5\NKE_`4D@BD2#QFI4Z0:X<UB]<.40@AP:/"LKB1HK11'HC>OP?QNN:Q3EA
"````
`
end
For some reason the code will have some spaces to left side after pasting it from here .. so make sure to remove the spaces before you uudecode it!

As a side note, this patch was done with version pythonmagick_0.9.1-3ubuntu5. Any other source code version may need some modifications when applying this patch.

Re: Some fixes to pythonmagick

Posted: 2011-08-11T06:01:02-07:00
by magick
Thanks for the patch. We'll get them into the next point release of PythonMagick within a few days.