wine-commits April 2013

wine-commits@winehq.org

2 participants
579 discussions

Victor Martinez : gdiplus: Fix returned rect in GdiGetRegionBoundsI.
by Alexandre Julliard 03 Apr '13

03 Apr '13

Module: wine Branch: master Commit: e891fe4b8e995ddd707af727137609533ec64093 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e891fe4b8e995ddd707af7271… Author: Victor Martinez <vmartinez(a)reactos.org> Date: Wed Apr 3 00:51:47 2013 +0200 gdiplus: Fix returned rect in GdiGetRegionBoundsI. --- dlls/gdiplus/region.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/gdiplus/region.c b/dlls/gdiplus/region.c index 99c5d35..d797459 100644 --- a/dlls/gdiplus/region.c +++ b/dlls/gdiplus/region.c @@ -716,7 +716,7 @@ GpStatus WINGDIPAPI GdipGetRegionBoundsI(GpRegion *region, GpGraphics *graphics, status = GdipGetRegionBounds(region, graphics, &rectf); if(status == Ok){ rect->X = gdip_round(rectf.X); - rect->Y = gdip_round(rectf.X); + rect->Y = gdip_round(rectf.Y); rect->Width = gdip_round(rectf.Width); rect->Height = gdip_round(rectf.Height); }

1 0

Francois Gouget : testbot/Users: Turn the DeleteNonPermanentSessions() method into a function.
by Alexandre Julliard 03 Apr '13

03 Apr '13

Module: tools Branch: master Commit: e90220247e4612c26bb8b8f7b2be7bbd4f2e3644 URL: http://source.winehq.org/git/tools.git/?a=commit;h=e90220247e4612c26bb8b8f7… Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Wed Apr 3 16:58:57 2013 +0200 testbot/Users: Turn the DeleteNonPermanentSessions() method into a function. Methods should not add filters to the collection they are called on. --- testbot/lib/WineTestBot/CGI/Sessions.pm | 12 ++++++------ 1 files changed, 6 insertions(+), 6 deletions(-) diff --git a/testbot/lib/WineTestBot/CGI/Sessions.pm b/testbot/lib/WineTestBot/CGI/Sessions.pm index 6cf8b71..97e33b5 100644 --- a/testbot/lib/WineTestBot/CGI/Sessions.pm +++ b/testbot/lib/WineTestBot/CGI/Sessions.pm @@ -75,14 +75,14 @@ sub CreateItem return WineTestBot::CGI::Session->new($self); } -sub DeleteNonPermanentSessions +sub DeleteNonPermanentSessions($) { - my $self = shift; my $User = $_[0]; - $self->AddFilter("User", [$User]); - $self->AddFilter("Permanent", [!1]); - map { $self->DeleteItem($_); } @{$self->GetItems()}; + my $Sessions = CreateSessions(); + $Sessions->AddFilter("User", [$User]); + $Sessions->AddFilter("Permanent", [!1]); + map { $Sessions->DeleteItem($_); } @{$Sessions->GetItems()}; } sub NewSession @@ -90,7 +90,7 @@ sub NewSession my $self = shift; my ($User, $Permanent) = @_; - CreateSessions()->DeleteNonPermanentSessions($User); + DeleteNonPermanentSessions($User); my $Session = $self->Add(); my $Existing = $Session;

1 0

Francois Gouget : testbot/Users: Fix UserDetails. pl to reuse WaitForApproval() instead of incompletely duplicating it.
by Alexandre Julliard 03 Apr '13

03 Apr '13

Module: tools Branch: master Commit: 5a6b0e8323ea7ef2056ccec29d57d37fc56d8c55 URL: http://source.winehq.org/git/tools.git/?a=commit;h=5a6b0e8323ea7ef2056ccec2… Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Wed Apr 3 16:58:34 2013 +0200 testbot/Users: Fix UserDetails.pl to reuse WaitForApproval() instead of incompletely duplicating it. --- testbot/web/admin/UserDetails.pl | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/testbot/web/admin/UserDetails.pl b/testbot/web/admin/UserDetails.pl index 9e79aa4..bc56eec 100644 --- a/testbot/web/admin/UserDetails.pl +++ b/testbot/web/admin/UserDetails.pl @@ -1,6 +1,7 @@ # User details page # # Copyright 2009 Ge van Geldorp +# Copyright 2013 Francois Gouget # # This library is free software; you can redistribute it and/or # modify it under the terms of the GNU Lesser General Public @@ -53,8 +54,7 @@ sub GetActions my $self = shift; my @Actions; - if ($self->{Item}->Active && length($self->{Item}->Password) == 1 && - ! defined($LDAPServer)) + if (!defined $LDAPServer and $self->{Item}->WaitingForApproval()) { $Actions[0] = "Approve"; }

1 0

Francois Gouget : testbot/Users: Remove a redundant check in SendResetCode( ).
by Alexandre Julliard 03 Apr '13

03 Apr '13

Module: tools Branch: master Commit: 783f7dc5e0793c1853fc31988d807940e3e7687b URL: http://source.winehq.org/git/tools.git/?a=commit;h=783f7dc5e0793c1853fc3198… Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Wed Apr 3 16:57:58 2013 +0200 testbot/Users: Remove a redundant check in SendResetCode(). An account which has been activated is no longer waiting for approval. --- testbot/lib/WineTestBot/Users.pm | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/testbot/lib/WineTestBot/Users.pm b/testbot/lib/WineTestBot/Users.pm index cb3725b..a163a17 100644 --- a/testbot/lib/WineTestBot/Users.pm +++ b/testbot/lib/WineTestBot/Users.pm @@ -166,7 +166,7 @@ sub SendResetCode { return "This account has been suspended"; } - if (! $self->Activated() && $self->WaitingForApproval()) + if ($self->WaitingForApproval()) { return "Your account has not been approved yet"; }

1 0

Francois Gouget : testbot/Users: Disabled accounts are not waiting for approval.
by Alexandre Julliard 03 Apr '13

03 Apr '13

Module: tools Branch: master Commit: b98cb3a916297bee4f33e2dd44cf58dbe9442c0b URL: http://source.winehq.org/git/tools.git/?a=commit;h=b98cb3a916297bee4f33e2dd… Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Wed Apr 3 16:57:39 2013 +0200 testbot/Users: Disabled accounts are not waiting for approval. --- testbot/lib/WineTestBot/Users.pm | 8 ++------ 1 files changed, 2 insertions(+), 6 deletions(-) diff --git a/testbot/lib/WineTestBot/Users.pm b/testbot/lib/WineTestBot/Users.pm index f29fd5e..cb3725b 100644 --- a/testbot/lib/WineTestBot/Users.pm +++ b/testbot/lib/WineTestBot/Users.pm @@ -1,4 +1,5 @@ # Copyright 2009 Ge van Geldorp +# Copyright 2013 Francois Gouget # # This library is free software; you can redistribute it and/or # modify it under the terms of the GNU Lesser General Public @@ -88,12 +89,7 @@ sub WaitingForApproval { my $self = shift; - if ($self->Activated()) - { - return !1; - } - - return ! $self->ResetCode; + return $self->Active && !$self->Activated() && !$self->ResetCode; } sub GenerateResetCode

1 0

Francois Gouget : testbot/Users: Remove the unused current user tracking code.
by Alexandre Julliard 03 Apr '13

03 Apr '13

Module: tools Branch: master Commit: c2b91aa379043a5555bc2eaafe47b2bc35f9acc9 URL: http://source.winehq.org/git/tools.git/?a=commit;h=c2b91aa379043a5555bc2eaa… Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Wed Apr 3 16:57:25 2013 +0200 testbot/Users: Remove the unused current user tracking code. Only the web modules have a concept of a 'current user' and they track him through his session and the PageBase.pm module. --- testbot/lib/WineTestBot/Users.pm | 15 ++------------- 1 files changed, 2 insertions(+), 13 deletions(-) diff --git a/testbot/lib/WineTestBot/Users.pm b/testbot/lib/WineTestBot/Users.pm index 54c07bf..f29fd5e 100644 --- a/testbot/lib/WineTestBot/Users.pm +++ b/testbot/lib/WineTestBot/Users.pm @@ -310,12 +310,11 @@ use WineTestBot::Config; use WineTestBot::UserRoles; use WineTestBot::WineTestBotObjects; -use vars qw (@ISA @EXPORT @PropertyDescriptors $CurrentUser); +use vars qw (@ISA @EXPORT @PropertyDescriptors); require Exporter; @ISA = qw(WineTestBot::WineTestBotCollection Exporter); -@EXPORT = qw(&CreateUsers &SetCurrentUser &GetCurrentUser &GetBatchUser - &Authenticate); +@EXPORT = qw(&CreateUsers &GetBatchUser &Authenticate); BEGIN { @@ -414,14 +413,4 @@ sub GetBatchUser return $class->CreateUsers()->GetItem("batch"); } -sub SetCurrentUser -{ - $CurrentUser = shift; -} - -sub GetCurrentUser -{ - return $CurrentUser; -} - 1;

1 0

Francois Gouget : testbot/ddl: Fix the SQL code that creates the 'batch' user.
by Alexandre Julliard 03 Apr '13

03 Apr '13

Module: tools Branch: master Commit: c8699e6985d0c9c89d651589ec7440dd7114b265 URL: http://source.winehq.org/git/tools.git/?a=commit;h=c8699e6985d0c9c89d651589… Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Wed Apr 3 16:57:03 2013 +0200 testbot/ddl: Fix the SQL code that creates the 'batch' user. --- testbot/ddl/winetestbot.sql | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/testbot/ddl/winetestbot.sql b/testbot/ddl/winetestbot.sql index 4fbde28..c8820cc 100644 --- a/testbot/ddl/winetestbot.sql +++ b/testbot/ddl/winetestbot.sql @@ -162,7 +162,7 @@ INSERT INTO Roles (Name, IsDefaultRole) VALUES('admin', 'N'); INSERT INTO Roles (Name, IsDefaultRole) VALUES('wine-devel', 'Y'); INSERT INTO Users (Name, EMail, Password, Active, RealName) - VALUES('batch', '/dev/null', '*', 'Batch user for internal jobs', NULL); + VALUES('batch', '/dev/null', '*', 'Y', 'Batch user for internal jobs'); INSERT INTO Branches (Name, IsDefault) VALUES('master', 'Y');

1 0

Francois Gouget : testbot/web: Standardize the OnAction() methods.
by Alexandre Julliard 03 Apr '13

03 Apr '13

Module: tools Branch: master Commit: a7e0a1d56ec80b437d85755631963a642041cc51 URL: http://source.winehq.org/git/tools.git/?a=commit;h=a7e0a1d56ec80b437d857556… Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Wed Apr 3 16:56:42 2013 +0200 testbot/web: Standardize the OnAction() methods. Also note that FormPage::Save() sets the 'ErrField' and 'ErrMessage' properties directly and returns a simple boolean. --- testbot/lib/ObjectModel/CGI/ItemPage.pm | 9 +++------ testbot/web/admin/UserDetails.pl | 22 +++++++++------------- 2 files changed, 12 insertions(+), 19 deletions(-) diff --git a/testbot/lib/ObjectModel/CGI/ItemPage.pm b/testbot/lib/ObjectModel/CGI/ItemPage.pm index a3310ec..16736fe 100644 --- a/testbot/lib/ObjectModel/CGI/ItemPage.pm +++ b/testbot/lib/ObjectModel/CGI/ItemPage.pm @@ -157,12 +157,9 @@ sub OnAction if ($Action eq "OK") { - if ($self->Save()) - { - $self->RedirectToList(); - exit; - } - return !1; + return !1 if (!$self->Save()); + $self->RedirectToList(); + exit; } elsif ($Action eq "Cancel") { diff --git a/testbot/web/admin/UserDetails.pl b/testbot/web/admin/UserDetails.pl index f49a522..9e79aa4 100644 --- a/testbot/web/admin/UserDetails.pl +++ b/testbot/web/admin/UserDetails.pl @@ -64,21 +64,13 @@ sub GetActions return \@Actions; } -sub OnApprove +sub OnApprove($) { my $self = shift; - ($self->{ErrField}, $self->{ErrMessage}) = $self->Save(); - if (defined($self->{ErrMessage})) - { - return !1; - } + return !1 if (!$self->Save()); $self->{ErrMessage} = $self->{Item}->Approve(); - if (defined($self->{ErrMessage})) - { - return !1; - } - + return !1 if (defined $self->{ErrMessage}); $self->RedirectToList(); exit; } @@ -88,8 +80,12 @@ sub OnAction my $self = shift; my $Action = $_[0]; - return ($Action eq "Approve" ? $self->OnApprove() : - $self->SUPER::OnAction(@_)); + if ($Action eq "Approve") + { + return $self->OnApprove(); + } + + return $self->SUPER::OnAction(@_); } package main;

1 0

Jacek Caban : ieframe: Improved debug traces.
by Alexandre Julliard 02 Apr '13

02 Apr '13

Module: wine Branch: master Commit: 64ead391d8afb1342efaeb2a0cdc27ac19f1f7af URL: http://source.winehq.org/git/wine.git/?a=commit;h=64ead391d8afb1342efaeb2a0… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue Apr 2 17:39:57 2013 +0200 ieframe: Improved debug traces. --- dlls/ieframe/dochost.c | 12 ++++++++++++ 1 files changed, 12 insertions(+), 0 deletions(-) diff --git a/dlls/ieframe/dochost.c b/dlls/ieframe/dochost.c index f95c4bd..7522c38 100644 --- a/dlls/ieframe/dochost.c +++ b/dlls/ieframe/dochost.c @@ -363,6 +363,16 @@ static IStream *get_travellog_stream(DocHost *This) return stream; } +void dump_travellog(DocHost *This) +{ + unsigned i; + + for(i=0; i < This->travellog.length; i++) + TRACE("%d: %s %s\n", i, i == This->travellog.position ? "=>" : " ", debugstr_w(This->travellog.log[i].url)); + if(i == This->travellog.position) + TRACE("%d: =>\n", i); +} + static void update_travellog(DocHost *This) { travellog_entry_t *new_entry; @@ -407,6 +417,8 @@ static void update_travellog(DocHost *This) } if(This->travellog.position > This->travellog.length) This->travellog.length = This->travellog.position; + + dump_travellog(This); } void create_doc_view_hwnd(DocHost *This)

1 0

Jacek Caban : ieframe: Added GoForward implementation.
by Alexandre Julliard 02 Apr '13

02 Apr '13

Module: wine Branch: master Commit: 0d30e51084afa88e25dad2ef59a7acf82f33cfee URL: http://source.winehq.org/git/wine.git/?a=commit;h=0d30e51084afa88e25dad2ef5… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue Apr 2 17:40:14 2013 +0200 ieframe: Added GoForward implementation. --- dlls/ieframe/ie.c | 4 ++-- dlls/ieframe/ieframe.h | 1 + dlls/ieframe/navigate.c | 10 ++++++++++ dlls/ieframe/webbrowser.c | 4 ++-- 4 files changed, 15 insertions(+), 4 deletions(-) diff --git a/dlls/ieframe/ie.c b/dlls/ieframe/ie.c index da0e99c..afbd2e9 100644 --- a/dlls/ieframe/ie.c +++ b/dlls/ieframe/ie.c @@ -149,8 +149,8 @@ static HRESULT WINAPI InternetExplorer_GoBack(IWebBrowser2 *iface) static HRESULT WINAPI InternetExplorer_GoForward(IWebBrowser2 *iface) { InternetExplorer *This = impl_from_IWebBrowser2(iface); - FIXME("(%p)\n", This); - return E_NOTIMPL; + TRACE("(%p)\n", This); + return go_forward(&This->doc_host->doc_host); } static HRESULT WINAPI InternetExplorer_GoHome(IWebBrowser2 *iface) diff --git a/dlls/ieframe/ieframe.h b/dlls/ieframe/ieframe.h index 4140e05..bbb3e4c 100644 --- a/dlls/ieframe/ieframe.h +++ b/dlls/ieframe/ieframe.h @@ -269,6 +269,7 @@ void call_sink(ConnectionPoint*,DISPID,DISPPARAMS*) DECLSPEC_HIDDEN; HRESULT navigate_url(DocHost*,LPCWSTR,const VARIANT*,const VARIANT*,VARIANT*,VARIANT*) DECLSPEC_HIDDEN; HRESULT go_home(DocHost*) DECLSPEC_HIDDEN; HRESULT go_back(DocHost*) DECLSPEC_HIDDEN; +HRESULT go_forward(DocHost*) DECLSPEC_HIDDEN; HRESULT refresh_document(DocHost*) DECLSPEC_HIDDEN; HRESULT get_location_url(DocHost*,BSTR*) DECLSPEC_HIDDEN; HRESULT set_dochost_url(DocHost*,const WCHAR*) DECLSPEC_HIDDEN; diff --git a/dlls/ieframe/navigate.c b/dlls/ieframe/navigate.c index 703e32c..d730b10 100644 --- a/dlls/ieframe/navigate.c +++ b/dlls/ieframe/navigate.c @@ -1099,6 +1099,16 @@ HRESULT go_back(DocHost *This) return navigate_history(This, This->travellog.position-1); } +HRESULT go_forward(DocHost *This) +{ + if(This->travellog.position >= This->travellog.length) { + WARN("No history available\n"); + return E_FAIL; + } + + return navigate_history(This, This->travellog.position+1); +} + HRESULT get_location_url(DocHost *This, BSTR *ret) { FIXME("semi-stub\n"); diff --git a/dlls/ieframe/webbrowser.c b/dlls/ieframe/webbrowser.c index 0a9f2a5..02edfd7 100644 --- a/dlls/ieframe/webbrowser.c +++ b/dlls/ieframe/webbrowser.c @@ -259,8 +259,8 @@ static HRESULT WINAPI WebBrowser_GoBack(IWebBrowser2 *iface) static HRESULT WINAPI WebBrowser_GoForward(IWebBrowser2 *iface) { WebBrowser *This = impl_from_IWebBrowser2(iface); - FIXME("(%p)\n", This); - return E_NOTIMPL; + TRACE("(%p)\n", This); + return go_forward(&This->doc_host); } static HRESULT WINAPI WebBrowser_GoHome(IWebBrowser2 *iface)

1 0

← Newer
1
...
44
45
46
47
48
49
50
...
58
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

wine-commits April 2013