incubator-wave-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ali Lown" <...@lown.me.uk>
Subject Re: Review Request: Adding final license headers
Date Fri, 14 Sep 2012 18:55:57 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/7077/#review11560
-----------------------------------------------------------

Ship it!


Thanks.

There are still a few spaces left floating, but we can change those when we commit it, other
than that it LGTM.

Might be worth waiting for Michael's feedback regarding keeping the 'Google header' in files.


/src/org/waveprotocol/box/webclient/widget/frame/FramedPanel.css
<https://reviews.apache.org/r/7077/#comment24849>

    Removing this space would be nice.



/src/org/waveprotocol/protobuf/extensions.proto
<https://reviews.apache.org/r/7077/#comment24848>

    And here.



/src/org/waveprotocol/pst/templates/jso/class.st
<https://reviews.apache.org/r/7077/#comment24850>

    And one here.



/src/org/waveprotocol/wave/client/wavepanel/view/dom/full/mock/ie_inline_thread.html
<https://reviews.apache.org/r/7077/#comment24847>

    Trailing space.


- Ali Lown


On Sept. 14, 2012, 12:30 a.m., Angus Turner wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/7077/
> -----------------------------------------------------------
> 
> (Updated Sept. 14, 2012, 12:30 a.m.)
> 
> 
> Review request for wave.
> 
> 
> Description
> -------
> 
> Patch ensuring all the code headers are right, as well as the license and notice files.
> 
> 
> Diffs
> -----
> 
>   /COPYING 1384205 
>   /DISCLAIMER PRE-CREATION 
>   /NOTICES PRE-CREATION 
>   /build-proto.properties.example 1384205 
>   /build.properties 1384205 
>   /check-certificates.sh 1384205 
>   /make_cert.sh 1384205 
>   /platforms/ubuntu/daemon-script.sh 1384205 
>   /platforms/ubuntu/server-wrapper.sh 1384205 
>   /process-script-args.sh 1384205 
>   /proto_src/org/waveprotocol/box/common/comms/WaveClientRpc.java 1384205 
>   /proto_src/org/waveprotocol/box/profile/ProfilesProto.java 1384205 
>   /proto_src/org/waveprotocol/box/search/SearchProto.java 1384205 
>   /proto_src/org/waveprotocol/box/server/persistence/protos/ProtoAccountStoreData.java
1384205 
>   /proto_src/org/waveprotocol/box/server/persistence/protos/ProtoDeltaStoreData.java
1384205 
>   /proto_src/org/waveprotocol/box/server/rpc/Rpc.java 1384205 
>   /proto_src/org/waveprotocol/protobuf/Extensions.java 1384205 
>   /proto_src/org/waveprotocol/wave/concurrencycontrol/ClientServer.java 1384205 
>   /proto_src/org/waveprotocol/wave/diff/Diff.java 1384205 
>   /proto_src/org/waveprotocol/wave/federation/FederationErrorProto.java 1384205 
>   /proto_src/org/waveprotocol/wave/federation/Proto.java 1384205 
>   /run-export.sh 1384205 
>   /run-import.sh 1384205 
>   /run-server.sh 1384205 
>   /server.config.example 1384205 
>   /server.federation.config.example 1384205 
>   /spec/waveid/Makefile 1384205 
>   /spec/waveid/waveidspec.html 1384205 
>   /src/org/waveprotocol/box/common/comms/waveclient-rpc.proto 1384205 
>   /src/org/waveprotocol/box/profile/profiles.proto 1384205 
>   /src/org/waveprotocol/box/search/search.proto 1384205 
>   /src/org/waveprotocol/box/server/frontend/WaveViewSubscription.java 1384205 
>   /src/org/waveprotocol/box/server/gxp/AuthenticationPage.gxp 1384205 
>   /src/org/waveprotocol/box/server/gxp/OAuthAuthorizeTokenPage.gxp 1384205 
>   /src/org/waveprotocol/box/server/gxp/RobotRegistrationPage.gxp 1384205 
>   /src/org/waveprotocol/box/server/gxp/RobotRegistrationSuccessPage.gxp 1384205 
>   /src/org/waveprotocol/box/server/gxp/TopBar.gxp 1384205 
>   /src/org/waveprotocol/box/server/gxp/UserRegistrationPage.gxp 1384205 
>   /src/org/waveprotocol/box/server/gxp/WaveClientPage.gxp 1384205 
>   /src/org/waveprotocol/box/server/persistence/protos/account-store.proto 1384205 
>   /src/org/waveprotocol/box/server/persistence/protos/delta-store.proto 1384205 
>   /src/org/waveprotocol/box/server/rpc/rpc.proto 1384205 
>   /src/org/waveprotocol/box/webclient/search/Search.css 1384205 
>   /src/org/waveprotocol/box/webclient/search/SearchPanel.css 1384205 
>   /src/org/waveprotocol/box/webclient/search/mock/digest.css 1384205 
>   /src/org/waveprotocol/box/webclient/widget/frame/FramedPanel.css 1384205 
>   /src/org/waveprotocol/box/webclient/widget/loading/Loading.css 1384205 
>   /src/org/waveprotocol/protobuf/extensions.proto 1384205 
>   /src/org/waveprotocol/pst/templates/api/api.st 1384205 
>   /src/org/waveprotocol/pst/templates/api/beans.st 1384205 
>   /src/org/waveprotocol/pst/templates/api/enum.st 1384205 
>   /src/org/waveprotocol/pst/templates/api/interface.st 1384205 
>   /src/org/waveprotocol/pst/templates/builder/beans.st 1384205 
>   /src/org/waveprotocol/pst/templates/builder/buildField.st 1384205 
>   /src/org/waveprotocol/pst/templates/builder/class.st 1384205 
>   /src/org/waveprotocol/pst/templates/builder/fieldDeclaration.st 1384205 
>   /src/org/waveprotocol/pst/templates/builder/importUtil.st 1384205 
>   /src/org/waveprotocol/pst/templates/builder/imports.st 1384205 
>   /src/org/waveprotocol/pst/templates/gson/class.st 1384205 
>   /src/org/waveprotocol/pst/templates/gson/clearField.st 1384205 
>   /src/org/waveprotocol/pst/templates/gson/fromGsonField.st 1384205 
>   /src/org/waveprotocol/pst/templates/gson/fromGsonFieldInner.st 1384205 
>   /src/org/waveprotocol/pst/templates/gson/importUtil.st 1384205 
>   /src/org/waveprotocol/pst/templates/gson/imports.st 1384205 
>   /src/org/waveprotocol/pst/templates/gson/toGsonField.st 1384205 
>   /src/org/waveprotocol/pst/templates/gson/toGsonFieldInner.st 1384205 
>   /src/org/waveprotocol/pst/templates/jso/beans.st 1384205 
>   /src/org/waveprotocol/pst/templates/jso/class.st 1384205 
>   /src/org/waveprotocol/pst/templates/jso/importUtil.st 1384205 
>   /src/org/waveprotocol/pst/templates/jso/imports.st 1384205 
>   /src/org/waveprotocol/pst/templates/pojo/addHashCode.st 1384205 
>   /src/org/waveprotocol/pst/templates/pojo/beans.st 1384205 
>   /src/org/waveprotocol/pst/templates/pojo/class.st 1384205 
>   /src/org/waveprotocol/pst/templates/pojo/clearField.st 1384205 
>   /src/org/waveprotocol/pst/templates/pojo/copyField.st 1384205 
>   /src/org/waveprotocol/pst/templates/pojo/fieldDeclaration.st 1384205 
>   /src/org/waveprotocol/pst/templates/pojo/importUtil.st 1384205 
>   /src/org/waveprotocol/pst/templates/pojo/imports.st 1384205 
>   /src/org/waveprotocol/pst/templates/pojo/returnFalseIfNotEqual.st 1384205 
>   /src/org/waveprotocol/pst/templates/proto/beans.st 1384205 
>   /src/org/waveprotocol/pst/templates/proto/class.st 1384205 
>   /src/org/waveprotocol/pst/templates/proto/copyField.st 1384205 
>   /src/org/waveprotocol/pst/templates/proto/enumHelpers.st 1384205 
>   /src/org/waveprotocol/pst/templates/proto/fromGsonField.st 1384205 
>   /src/org/waveprotocol/pst/templates/proto/importUtil.st 1384205 
>   /src/org/waveprotocol/pst/templates/proto/imports.st 1384205 
>   /src/org/waveprotocol/pst/templates/proto/messageHelpers.st 1384205 
>   /src/org/waveprotocol/pst/templates/proto/splitPoint.st 1384205 
>   /src/org/waveprotocol/pst/templates/proto/toGsonField.st 1384205 
>   /src/org/waveprotocol/pst/templates/util/addHashCode.st 1384205 
>   /src/org/waveprotocol/pst/templates/util/class.st 1384205 
>   /src/org/waveprotocol/pst/templates/util/imports.st 1384205 
>   /src/org/waveprotocol/pst/templates/util/returnFalseIfNotEqual.st 1384205 
>   /src/org/waveprotocol/wave/client/debug/logger/Logger.css 1384205 
>   /src/org/waveprotocol/wave/client/doodad/attachment/render/Thumbnail.css 1384205 
>   /src/org/waveprotocol/wave/client/doodad/experimental/htmltemplate/secureStyles.css
1384205 
>   /src/org/waveprotocol/wave/client/doodad/form/button/Button.css 1384205 
>   /src/org/waveprotocol/wave/client/doodad/form/check/CheckBase.css 1384205 
>   /src/org/waveprotocol/wave/client/doodad/form/check/Label.css 1384205 
>   /src/org/waveprotocol/wave/client/doodad/form/input/Input.css 1384205 
>   /src/org/waveprotocol/wave/client/doodad/form/input/Password.css 1384205 
>   /src/org/waveprotocol/wave/client/doodad/selection/CaretMarker.css 1384205 
>   /src/org/waveprotocol/wave/client/editor/Editor.css 1384205 
>   /src/org/waveprotocol/wave/client/editor/content/Default.css 1384205 
>   /src/org/waveprotocol/wave/client/editor/extract/Extract.css 1384205 
>   /src/org/waveprotocol/wave/client/editor/harness/public/EditorTest.css 1384205 
>   /src/org/waveprotocol/wave/client/editor/harness/public/EditorTest.html 1384205 
>   /src/org/waveprotocol/wave/client/editor/impl/DiffManager.css 1384205 
>   /src/org/waveprotocol/wave/client/editor/sugg/Suggestion.css 1384205 
>   /src/org/waveprotocol/wave/client/gadget/renderer/Gadget.css 1384205 
>   /src/org/waveprotocol/wave/client/scheduler/knobs/knobs.css 1384205 
>   /src/org/waveprotocol/wave/client/testing/public/UndercurrentHarness.html 1384205 
>   /src/org/waveprotocol/wave/client/wavepanel/impl/edit/KeepFocusInView.java 1384205

>   /src/org/waveprotocol/wave/client/wavepanel/impl/toolbar/EditToolbar.css 1384205 
>   /src/org/waveprotocol/wave/client/wavepanel/impl/toolbar/attachment/AttachmentPopupWidget.css
1384205 
>   /src/org/waveprotocol/wave/client/wavepanel/view/dom/full/Blip.css 1384205 
>   /src/org/waveprotocol/wave/client/wavepanel/view/dom/full/BlipLinkPopupWidget.css 1384205

>   /src/org/waveprotocol/wave/client/wavepanel/view/dom/full/Collapsible.css 1384205 
>   /src/org/waveprotocol/wave/client/wavepanel/view/dom/full/ContinuationIndicator.css
1384205 
>   /src/org/waveprotocol/wave/client/wavepanel/view/dom/full/Conversation.css 1384205

>   /src/org/waveprotocol/wave/client/wavepanel/view/dom/full/FocusFrame.css 1384205 
>   /src/org/waveprotocol/wave/client/wavepanel/view/dom/full/FocusFrameIE.css 1384205

>   /src/org/waveprotocol/wave/client/wavepanel/view/dom/full/Participants.css 1384205

>   /src/org/waveprotocol/wave/client/wavepanel/view/dom/full/ReplyBox.css 1384205 
>   /src/org/waveprotocol/wave/client/wavepanel/view/dom/full/RootThread.css 1384205 
>   /src/org/waveprotocol/wave/client/wavepanel/view/dom/full/mock/blip.css 1384205 
>   /src/org/waveprotocol/wave/client/wavepanel/view/dom/full/mock/blip.html 1384205 
>   /src/org/waveprotocol/wave/client/wavepanel/view/dom/full/mock/focus.css 1384205 
>   /src/org/waveprotocol/wave/client/wavepanel/view/dom/full/mock/ie_inline_thread.css
1384205 
>   /src/org/waveprotocol/wave/client/wavepanel/view/dom/full/mock/ie_inline_thread.html
1384205 
>   /src/org/waveprotocol/wave/client/wavepanel/view/dom/full/mock/inline_thread.css 1384205

>   /src/org/waveprotocol/wave/client/wavepanel/view/dom/full/mock/inline_thread.html 1384205

>   /src/org/waveprotocol/wave/client/wavepanel/view/dom/full/mock/participation.html 1384205

>   /src/org/waveprotocol/wave/client/wavepanel/view/dom/full/mock/tmp_inline_thread.css
1384205 
>   /src/org/waveprotocol/wave/client/wavepanel/view/dom/full/mock/tmp_inline_thread.html
1384205 
>   /src/org/waveprotocol/wave/client/widget/button/icon/IconButtonTemplate.css 1384205

>   /src/org/waveprotocol/wave/client/widget/button/icon/IconButtonTemplate1.css 1384205

>   /src/org/waveprotocol/wave/client/widget/button/text/TextButton.css 1384205 
>   /src/org/waveprotocol/wave/client/widget/menu/Menu.css 1384205 
>   /src/org/waveprotocol/wave/client/widget/popup/DesktopTitleBar.css 1384205 
>   /src/org/waveprotocol/wave/client/widget/popup/DesktopUniversalPopup.css 1384205 
>   /src/org/waveprotocol/wave/client/widget/popup/MobileUniversalPopup.css 1384205 
>   /src/org/waveprotocol/wave/client/widget/popup/desktopchrome/DesktopPopupChrome.css
1384205 
>   /src/org/waveprotocol/wave/client/widget/popup/desktopchrome/DropdownPopupChrome.css
1384205 
>   /src/org/waveprotocol/wave/client/widget/profile/ProfilePopupWidget.css 1384205 
>   /src/org/waveprotocol/wave/client/widget/progress/ProgressWidget.css 1384205 
>   /src/org/waveprotocol/wave/client/widget/toolbar/SubmenuToolbarWidget.css 1384205 
>   /src/org/waveprotocol/wave/client/widget/toolbar/ToplevelToolbarWidget.css 1384205

>   /src/org/waveprotocol/wave/client/widget/toolbar/buttons/HorizontalToolbarButtonWidget.css
1384205 
>   /src/org/waveprotocol/wave/client/widget/toolbar/buttons/VerticalToolbarButtonWidget.css
1384205 
>   /src/org/waveprotocol/wave/communication/proto/Int52.java 1384205 
>   /src/org/waveprotocol/wave/concurrencycontrol/clientserver.proto 1384205 
>   /src/org/waveprotocol/wave/diff/diff.proto 1384205 
>   /src/org/waveprotocol/wave/federation/federation.protodevel 1384205 
>   /src/org/waveprotocol/wave/federation/federation_error.protodevel 1384205 
>   /test/com/google/wave/api/AttachmentRobotTest.java 1384205 
>   /test/org/waveprotocol/box/common/ExceptionalIteratorTest.java 1384205 
>   /test/org/waveprotocol/box/server/robots/RobotsTestBase.java 1384205 
>   /test/org/waveprotocol/box/server/rpc/WebSocketChannelTest.java 1384205 
>   /test/org/waveprotocol/wave/client/doodad/link/LinkTest.java 1384205 
>   /test/org/waveprotocol/wave/client/wavepanel/impl/diff/DiffControllerTest.java 1384205

>   /tools/eclipse.sh 1384205 
>   /war/static/auth.css 1384205 
>   /whitepapers/Makefile 1384205 
> 
> Diff: https://reviews.apache.org/r/7077/diff/
> 
> 
> Testing
> -------
> 
> n/a
> 
> 
> Thanks,
> 
> Angus Turner
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message