Sony's Changes to GNUstep GUI Library: Adding Touch

http://blog.deliciousrobots.com/2010/11/27/sonys-changes-to-gnustep-gui-library-adding-touch/

by postfuturist • 16 years ago

41 7 16 years ago

blog.deliciousrobots.com

dizm2• 16 years ago

Pretty obvious that whoever was working on that was mostly trying to reverse engineer UIKit vs. update GNUStep for touch. There are even commented out lines that are what would be output by class-dump.

I wonder if the straight reverse engineering/possible decompiling is why it was pulled so fast. Also, it's subtle, but there is a reason Apple didnt simply graft touch onto AppKit like this project is/was attempting.

postfuturist3• 16 years ago

I didn't see that at all. The gesture recognition stuff all looks written from scratch and the integration code doesn't match anything from UIKit. There are thousands of lines of code added by Sony with only a few commented out sections, mostly well-commented code also. The only thing I could see that remotely looks like what you are talking about is this

    // Adding and removing targets and actions
    //- (void)addTarget:(id)targetaction:(SEL)action;
    //- (void)removeTarget:(id)targetaction:(SEL)action;


    // Adding and removing targets and actions
    /***
    - (void)addTarget:(id)targetaction:(SEL)action
    {
    }

    - (void)removeTarget:(id)targetaction:(SEL)action
    {
    }
    ***/

This is an unused interface in SNGestureRecognizer. The name of one of the methods, addTarget, is the same as something from UIKit's UIControl. It's hardly a smoking gun, as it is unused and doesn't even match the method signature of the one from UIKit:

    - (void)addTarget:(id)target action:(SEL)action forEvents:(int)eventMask;

It's certainly not the result of a reverse engineering dump. If anything, Sony engineers thought for a second that they would mimic UIKit's interfaces and then quickly scrapped the idea. The other commented regions look like you might expect from a normal codebase:

    /*** Cannot assume touches will come simultaneously
    // Not a valid pan if # of touches on view doesn't match
    if ([touches count] != numberOfTouchesRequired ||
        //[_view touchCount] != numberOfTouchesRequired)
        [_hitTestedView touchCount] != numberOfTouchesRequired)
    {
       if (_state == SNGestureRecognizerStatePossible)
          [self changeState: SNGestureRecognizerStateFailed];
       else
          [self changeState: SNGestureRecognizerStateCancelled];  

       return;
    }
    ***/

panic• 16 years ago

I believe dizm was referring to the header and class interface, not the source. The header is more or less ripped directly from UIKit, with some puzzling additions. What is the point of adding a "vulture" without also adding an "obstacle", for example? It seems like these methods should be private, not exposed in the public header.

dizm2• 16 years ago

SNTouch.m is one example with stubbed out class-dump methods.

The naming in every place that it can is lifted directly from UIKit, and then their changes use different naming conventions. For example the new instance variables in NSView.h.

Sony's Changes to GNUstep GUI Library: Adding Touch

Related Stories