gnash-dev
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Gnash-dev] patch for curl_adapter.


From: Andrea Palmatè
Subject: Re: [Gnash-dev] patch for curl_adapter.
Date: Wed, 29 Apr 2009 21:49:48 +0200

This patch is perfect..
but it also needs from line 53

void
URL::normalize_path(std::string& path)
{

#if defined(_WIN32) || defined(WIN32) || defined(__OS2__) || defined(__amigaos4__)
        return;
#endif

Attached the two last patches (please someone commit them for me..)
I've tried to make a bzr commit and a bzr push but it seems there is something wrong with my trunk
Can someone explain me what could be the problem?

This is the error i receive

bzr: ERROR: Cannot lock LockDir(http://bzr.savannah.gnu.org/r/gnash/trunk/.bzr/branch/lock ): Transport operation not possible: http does not support mkdir()

Andrea

Il giorno 29/apr/09, alle ore 16:16, Richard Wilbur ha scritto:

On Tue, 2009-04-28 at 23:14 +0200, Andrea Palmatè wrote:
just to be clear. look at this piece of code:

URL::URL(const std::string& absolute_url)
{
        //cerr << "URL(" << absolute_url << ")" << endl;
        if ( ( absolute_url.size() && absolute_url[0] == '/' )
                || absolute_url.find("://") != std::string::npos
|| ( absolute_url.size() > 1 && absolute_url[1] == ':' )) //for win32

this is what i mean..

What about something like this?

=== modified file 'libbase/URL.cpp'
--- libbase/URL.cpp     2009-03-17 16:14:19 +0000
+++ libbase/URL.cpp     2009-04-29 13:54:31 +0000
@@ -153,7 +153,9 @@
        //cerr << "URL(" << absolute_url << ")" << endl;
        if ( ( absolute_url.size() && absolute_url[0] == '/' )
                || absolute_url.find("://") != std::string::npos
- || ( absolute_url.size() > 1 && absolute_url[1] == ':' )) //for win32 + || ( absolute_url.size() > 1 && absolute_url[1] == ':' ) //for win32
+             || ( absolute_url.size() > 2 && absolute_url.find(':',
2) != std::string::npos ) //for aos4
+             )
        {
                //cerr << "It's absolute" << endl;
                init_absolute(absolute_url);


I believe this would work.  It recognizes the AmigaOS paths with
variable length assignments as absolute.  Am I remembering correctly
that the translation of assignments happens at the OS level? If that is
the case, then the invocation:
aos4-gnash DH2:Gnash/SWF_TEST/test.swf

should work with 'DH2:Gnash/SWF_TEST/test.swf' being recognized as an
absolute path.

Richard Wilbur



_______________________________________________
Gnash-dev mailing list
address@hidden

Attachment: curl_adapter.diff
Description: Binary data

Attachment: URL.cpp.diff.cpp
Description: Binary data


http://lists.gnu.org/mailman/listinfo/gnash-dev



reply via email to

[Prev in Thread] Current Thread [Next in Thread]