Just want to create a FBO with different size from game screen. There is no other meanings.


If you are cleaning up in general, then I might mention a few other weaknesses of this code:

1) `cb()` -- if there is an error, some sort of message should be passed to callback; also probably good to emit "error".

2) According to the postgres DOC:

> It is possible for NoticeResponse and ParameterStatus messages to be interspersed between CopyData messages; frontends must handle these cases, and should be prepared for other asynchronous message types as well (see Section 49.2.6). Otherwise, any message type other than CopyData or CopyDone may be treated as terminating copy-out mode.

(See: [http://www.postgresql.org/docs/9.4/static/protocol-flow.html#COPY-Operations])

so in theory the code should check for these and at the least ignore them. However, "NoticeResponse" at least could be propagated to the saved "data" listener(s), which are unplugged by "submit".


exuberant


I don't think you needed this one since strings are immutable. normally I wouldn't care about the redundancy but on Py2k this could actually be a problem if `self.name` has unicode chars.


@marcysutton: e29cb807 :goat:


should we make a backup for tracing? for example mv the path as an old path using time suffix?log and data may be cleared by external crontab task?


LGTM. We didn't create a singleton holder on the viewer side, so I think you don't have to mention it in your commit message


"github area" can be removed now.


Initial commit


Thanks.


@moya6 - Patryk, While ValidationException probably makes more sense here than AccessControlException, it also would break backward compatibility so I am not in particularly in favor of it. One certainly can conceive of cases where developers are already explicitly catching AccessControlException but not any superclass exception and would be forced to change their code. While we can do this with a major point release, we would not normally do so during a minor release or patch release. Also is I am not mistaken, we cannot simply deprecating this method and just add a new one that throws ValidationException rather than AccessControlException because I do not believe that Java allows two identical function prototypes to differ only in their throws clause. Therefore, IMO, we are pretty much screwed. Instead, the best we can do would be to change the javadoc, indicate that we plan to change it next major release, and be done with it. Otherwise, we are potentially forcing code changes. If there is enough use of this particular method in a application's code base, they may very well simply decide to forgo upgrading to this next ESAPI patch release rather than fixing all the occurrences that we have unintentionally broken. Thus, my inclination is to reject these particular commits and instead just focus on some relevant javadoc changes explaining under what circumstances that AccessControlException may be thrown (and acknowledging at that point that ValidationException probably would have been a better choice).

Please respond when you have a chance. Also note that at the very least when you DO decide to make an interface change, you should probably check with myself or Chris Schmidt first. If we were to approve it, we would also insist on appropriate javadoc changes / updates.


It's Ok to me. But I'd like to keep decoder interface platform independent. So we can support more platform such as DXVA. I will remove this getDisplayID in near future. Hope we can get better abstract then.


Actually, what do you think about @poulter7's existing solution in #7?

This implementation spawns a lot more schedulers, where his still only uses one. I think the recommended way to use watchdog is to use fewer. And like you pointed out, this only works with the top-level for now.


```js
const {item} = this.props;
const title = item.get('title');
```


That looks good. I had to hack around this in my book. I recall it was a simple fix. You can find it in custom styles.

I suppose ideally you would show the full size image in lightbox, separate tab (just a link) or something but that feels like a plugin. We might want to activate async loading of images on the official site btw.


These should be properties... also look for other places that could be using properties instead of member variables


:+1:
What time the document translation project?


Speaking of plugins although it's nice each plugin is separate it doesn't exactly help performance if they all have to traverse all of the content (ie. through cheerio). We could proxy some basic ops (ie. cheerio) to get more performance out of it. That's a separate issue, though.


These got added and i think they are unnecessary


I just reproduced what is done with `ipython` and `sage-notebook`. It did work for me, that's rather strange.


課題5はまだタブとスペースが混在しています。修正するなら、直して欲しかったです...。


Looking at this Google Code issue #190 (which is a different # here on GitHub), I see that the bug looks like it had already been changed to throw AccessControlException rather than IOException. I haven't checked if the JUnit test case(s) were also updated accordingly (assuming there was even a test case to cover this). Anyway, at first glance, it appears as though we accepte karansanwal's pull request without marking this issue as closed. If so, we dropped the ball on that.


Зачем? Это токены, следовательно присвоить значения переменным true / false не получиться - не распарсит.


It would have been clearer to do this as 2 commits. One for gofmt & another for the actual changes.


I get this
```
Detected file collision(s):

        /usr/share/sage/src/sage

Searching all installed packages for file collisions...

Press Ctrl-C to Stop

None of the installed packages claim the file(s).

Package 'sci-mathematics/sage-9999' NOT merged since it has one or
more collisions between symlinks and directories, which is explicitly
forbidden by PMS section 13.4 (see bug #326685). If necessary, refer
to your elog messages for the whole content of the above message.
```
Now this may be a corner case when `/usr/share/sage/src/sage` is left.


yes, we can use a cmd flag controlling this. but now let it run first. :-)


first commint


@moschel This does not belong in the description.


FizzBuzzOutという関数名は出力までしそうなのに、値を返すだけなのですね。もう少し名前に気を付けた方がいいかもしれませんね。（値編集はFormat~とか）


Yes, I'd say it is a remnant from another installation and it isn't recognised as being a part of any packages. Pity `pkg_preinst` doesn't cover that case, I am afraid you will have to remove it manually.


is managing の目的語にDOM elementが来るので、"Vue インスタンスが管理している DOM 要素"ですかね。


オブザーブというカタカナ表記に少し違和感感じました。全部統一でそう表記されているならようですが、「監視」でも良い気がします。また、こちらもobservingの目的語にdata objectが来るので、Vue インスタンス「を」ではなく「が」が本来の意図だと思います。


try 4 ok


@moya6 - Have you looked at Google Issue 194 (https://code.google.com/p/owasp-esapi-java/issues/detail?id=194) which is what the AntiSamy failure check that you commented out is related to? There was a claim that there was a fix there, although neither Chris nor I ever substantiated it. I guess if we are going to keep using AntiSamy and AntiSamy itself is unlikely to be fixed, this is the best that we can do if the patches proposed for Google issue 194 do not actually fix / workaround this issue somehow.


Here's how I would do it ... just a tip no judgement ... from the beginning:

* fork and new branch
* commit my changes in this new fork branch
* pull upstream master into local master
* rebase local changes in fork branch on top of local master
* force push (`--force-with-lease`) local branch to rewrite history on github

+ve:
Good history at all times

-ve:
Unstable commit hashes (due to force pushing).

YMMV :rose:


closes #94


This needs to be a date, like "2000-01-01", not "x".  That's what's making the tests fail.  The other fields work with "x" because they have string types, but you can't set a date to "x".


Prueba de combinación de branches


Closes #94


@TomBebbington Just FYI doing this might cause some weird behavior because the sizes of these arguments are actually 32 bit. See the [arduino source](https://github.com/arduino/Arduino/blob/master/hardware/arduino/sam/cores/arduino/wiring_digital.c#L73).


This Variable name should be something that represents the locale code
I would prefer naming it `translated_locale_code`, so future developer can understand it easily


トランスクルージョンコンテンツではなくテンプレートでは？？


コミットは出来れば、課題ごとに分けてしてください。コメントしづらいです...。

課題7ですが、Getter、Setterとして、```string GetValue()```と```void Set(string outPutValue)```を用意しているのであれば、誤りです。Getter、Setterが何かわからないのであれば、質問してください。また、C#では出来ればプロパティを使用して実装してください。
参考:[MSDN:プロパティ (C# プログラミング ガイド)](https://msdn.microsoft.com/ja-jp/library/x9fsa0sw.aspx)

```void OutPutValuse()```は綴り間違いですか？バルス！


It would be great to have this line split in two


I think it's better to keep the names exposed to the user shorter, i.e., without the `CUSTOM_` prefix.  Then it seems we'll have to name default values differently, e.g., with `DEFAULT_` prefix or so.
```bash
read -p "Enter number of data nodes [$DEFAULT_NUM_DATA_NODES]: " NUM_DATA_NODES
: ${NUM_DATA_NODES:=$DEFAULT_NUM_DATA_NODES}
```
Later, a helper function could allow us to use a single name, and validate values, etc.  Here's a sketch:
```bash
enter() {
  local name=$1
  local value=${!name:-}
  local desc=${2:-$name}
  local prompt="Enter $desc ${value:+[$value]}: "
  read -p "$prompt"
  eval "$name=\${REPLY:-\${$name:-}}"
}

NUM_DATA_NODES=2
enter NUM_DATA_NODES "number of data nodes"
```


With respect to the fact this was written almost two years ago, following MSDN it seems that `SendTo()` can return immediately and actually send less bytes than the sender requested, due to possible constraints. It's not clear whether `Send()` does this but I would expect it can do the same, i.e. not send the entire buffer.
Are we basically saying here, "send what you can and move on"?


Break and add alt attribute into img tag.


Does not its better to use the `else' logic only?


@thedarkone I think that works!


This is not going to work for two reasons:

1. If a RubyGems doesn't put `ext/` in the load path this will still fail (more on this below).
2. This requires `./lib` to be in the load path during development as otherwise the code might end up loading an older version of the extension (if present). I _do not_ want to force myself (or others) to run everything using `ruby -Ilib` (or set this in a Rakefile). In fact, I don't want to mess with `$LOAD_PATH`, ever.

Regarding case 1, the only reason I can think of as to why RubyGems might not love the extension in to the `lib` directory is what I mentioned in https://github.com/YorickPeterse/ruby-ll/issues/15#issuecomment-107291321. If so it's odd that this works for some installations and not others, but if changing that fixes it (the patch does still work on my laptop) then that's the way to go.


I've changed all this data, nefarious people.


Just a comment about the last parameter and your readme.   You need to update the Extract section in the readme with this new commit: The readme.md says that that line creates a LevelDB, but it doesn't.  Also the last param isn't the number of mini batches like the its says.  Thanks.


I'd say there was definitely something obvious I was missing. I think I looked at this about 4 different times and didn't realize my mistake. 😔 That's my bad. Thank you for being patient with me. 🍻

I'm going to reopen your previous PR and pull it in. Thanks again!


This method and the one below need to be within the class declaration for `User`.

Your class should look like this:

```ruby
class User < ActiveRecord::Base
  # Include default devise modules. Others available are:
  # :confirmable, :lockable, :timeoutable and :omniauthable
  devise :database_authenticatable, :registerable,
         :recoverable, :rememberable, :trackable, :validatable, :confirmable
  has_many :posts

  def admin?
    role == 'admin'
  end

  def moderator?
    role == 'moderator'
  end
end
```


I see. Didn't even know that was possible.

Currently, we are not showing the variable names to the user. The user would simply see the message and the default value, e.g:
Enter number of data nodes [2]:

I think it's fine as it is, but feel free to change if you like.


Oh! there is an Typo BTW! So your code is not running!


Doing what chatWindowDashboard does. Is there a common file this can be placed in?


Yes, the sketched version mimics the same thing.  I also think it's good as it is, and we can introduce more utilities for writing installer scripts as we go.  This will be one of the first things to add.


Hi Christopher,

Mea culpa.

First off, no harm meant from any of this. The origin of this rewrite was my need to move quickly and get something together in a module that I could package and get running right away. I'm totally happy to work with you to merge this back into a mainline distribution, and apologize for not getting in touch sooner (again, speed was the issue).

Regarding the License: Not to split hairs, and as far as I've read: The way I updated the License was within the bounds for the Apache 2.0, and, additionally, given that it was a rewrite, it was to further distance your potential liability for code you didn't product or distribute. That said, if you'd prefer the format you had originally, I'll rewrite the License... I was just trying to stick to my interpretation of the convention.

So, given all of this, I'm happy to collaborate moving forward, and hope this didn't get us off on the wrong foot.


After correcting the Typo, this page shows like this! :sob:
![firefox_screenshot_2015-06-01t04-11-33 931z](https://cloud.githubusercontent.com/assets/7114151/7906138/afed7490-0846-11e5-873a-58125b3eda6e.png)


Nice, 2 things

Always initialize variables with the var keyword, including variables inside the for statement (var i = 0;)

The function should "return" the result, then you can console log your execution of it.


All fixed up. Your commit is now part of #504.


The lack of proper indentation in the above makes it hard to read, redo with good indentation, also, try to not use a for loop, utilize the .split() feature on a string.


Hi, is this a implementation of Recurrent Neural Network?


@zathras-crypto: leftover


Oh my, I didn't notice this before (and merged it nevertheless, even though it was *only* this one-line-change. >_<


Very roughly: how ready is this branch?


Single-stepping (i.e. when we hit our basic block terminators) causes RIP to increase by 1.  That `jmp r12` is turning into `jmp rsp` when we start stepping again with RIP increased by 1.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.215](http://teamcity.monogame.net/viewLog.html?buildId=7884&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


Buenas... digame si lo hice bien !!!


ok


Sorry,Waiting for update.Moment !


I didn't even know that!


Isn't the first "av.wad" redundant?


修复 #50


This should go below the `$pod = ...` at line 2726 in this patch


13行目と74行目のスペース除いてもういっちょこのブランチ(to_ascii)にコミットしてくださいますようおねがいもうしあげます


Ah, very true. It is below in our production site not this patch. I'll update it :)


Thanks - I saw that via Travis (man that thing is useful!!!) :)

https://github.com/zathras-crypto/omnicore/commit/bdfddb1c73e59ff20a2acb8618ddc698360eac31


So glad to see this is fixed.


yup, it works, but it's no really clean
```
Exec=doom -file av.wad av.wad -deh av.deh
```


Pretty close I'd say - I kind of segwayed into some performance related stuff with our recent discussions but I'd like to finish this up - I wanted to address the ```amountremainingtofill``` attribute too which also means extending the ```CMPMetaDEx``` class - that'll probably be next on my list


I think these lines will need some explanation?!


Indeed! The FPS counter in Crispy Doom now shows 35 fps instead of 70. Good job!


Remove this from general_test.go


Sounds fine. I'd like to get it in as soon as possible though, since it's pretty big change.


In general, syntax cannot do any matching (like in the `[_]` example), and pattern synonyms are restricted to constructors and cannot be overloaded. DISPLAY doesn't have these restrictions. My motivating example (I can put that in the release notes instead of `[_]`) is

    instance
      SemiringNat : Semiring Nat
      SemiringNat = record { ...; _+_ = natPlus }

    {-# DISPLAY natPlus a b = a + b #-}

A secondary motivation is that it makes it easier to debug out auto-generated display forms.


@magnumripper I think the usage of vswap32/64 is a bit ambiguous. Sometimes it's used for in-place updating, sometimes for returning a updated value. Using curly braces `#define vswap32 { ... }` won't fit in the latter use case.


Thanks Jiachen. Two major comments; let's go a further round.
(1) no #pragma once in .cpp files
(2) make sure the code can compile on Ubuntu (I see many PDH realted code)


you can move this javascript to custom JS file like links.js and apply it over here using javascript_include_tag form helper


Y'a des gens qui s'inscrivent à ça ? On le sait où ?


OK. The DLL reloading scenario probably did not cross my mind. What about this change instead? It seem to me this should fix the issue.

~~~~
diff --git a/src/global-init.cxx b/src/global-init.cxx
index e85d187..2d6d338 100644
--- a/src/global-init.cxx
+++ b/src/global-init.cxx
@@ -559,10 +559,8 @@ thread_callback (LPVOID /*hinstDLL*/, DWORD fdwReason, LPVOID /*lpReserved*/)

         // Do thread-specific cleanup.
         log4cplus::threadCleanup ();
-#if ! defined (LOG4CPLUS_THREAD_LOCAL_VAR)
         log4cplus::thread::impl::tls_cleanup (
             log4cplus::internal::tls_storage_key);
-#endif
         break;
     }
 ~~~~


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.215](http://teamcity.monogame.net/viewLog.html?buildId=7884&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 486, ignored: 6 Build time: 00:21:31


> This commit comes too late - the previous commit did not compile because the parts this commit changes were still using the removed long getLastActivity()

Please do never compile and run individual intermediate commits which I ship as a whole branch by sending a branch pull request.
They are not chosen with the intention of allowing people to do this - if development commits were supposed to be fully working units, they would be branches. They're instead small work steps such as building a car engine, a car door, etc. They're not a whole car yet which would be safe to drive at 150 mph :)
Thus, they might do stuff such as not compiling, permanently breaking fred/WOT/etc. databases, causing a zombie apocalypse, etc.

(This shall also be reflected by the fact that the WOT "next" branch will in the future never receive individual single commits, it will only ever see merge commits. All work shall be wrapped in a branch.)

> This mixes updating the existing column to function, renaming it, and adding a new one.

Sorry. I had thought adapting the whole QueueToadlet, i.e. the UI, would be a reasonable candidate for squashing multiple small fry commits.


No, not really, all lumps in this WAD will get processed twice. :/


woops incorrect message, added line to prevent console from appearing in application


Sure thing bud :)


You have added paginate method at the end of each logic, can you separate it out and add it at the end of case statement. This will remove code duplication.


Maybe Better switch back to support lib dialogfragment. What you described in your PR is also true for dialog fragments


@ChrisCoffey We can change from firebase if its becoming annoying


Can you brake this big line? As per the rails style guide the length of line should be around 78 to 80 characters and then break the line. This will increase the code readablity.


Here same issue, I think we can separate the paginate method from the logic and apply it only once.


Looks great and all the tests still run. Thanks so much, I'll merge this in right away.


TeamCity xBIM :: Xbim Geometry :: Release [Build 85](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1780&buildTypeId=XBIM_XbimGeometry_Release) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 03:28:32


TeamCity xBIM :: Xbim Geometry :: Release [Build 86](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1781&buildTypeId=XBIM_XbimGeometry_Release) is now running


Change indentation to 2 space instead of 4 space.


Move this script logic to a separate JS file and then include it.


Hi!
I'd actually rather do the opposite (remove /etc/cowrie/cowrie.cfg, /etc/cowrie.cfg). For the sake of easy containerization. So I'll probalby do that.

 I see you added 'data/cowrie.cfg' as a possible configuration path. Do you have a reason for this?


If this route is for searching of a link then I prefer search_links instead of this one. What do you think?


code duplication on paginate method. Separate it from the logic and call at the end.


Hater når dette skjer :'(


As this association is between Link and LearnTime and you have collection of LearnTime objects on which you map link then the name should be 'learn_time'  instead of 'link'. link { |learn_time| learn_time.link }


Can you please explain me the use of bypass: true in sign_in method?


このNSLongの書き方で正常に動くのかな？ isAnswered = NOが条件式になってるんだろうか。
知らないので教えて欲しいっす。


Awesome, thank you! (see
https://github.com/SeaRbSg/little-schemer/commit/25891921ba713b65503d30c5fb90e7af596431bb
)

On Sun, May 31, 2015 at 5:13 PM, Scott Windsor <notifications@github.com>
wrote:

> Protip: http://stevenharman.net/git-pull-with-automatic-rebase
>
> an alwaysrebase keeps a @zenspider <https://github.com/zenspider> happy [image:
> :+1:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SeaRbSg/little-schemer/commit/92fe9594be4b014b3946b2be49987dbbba07418e#commitcomment-11446260>
> .
>


@chicagogrooves you mean just like this package you published? https://atmospherejs.com/deanius/meteor-computation


#include<stdio.h>

main()
{
    printf("Hello World");
}


This block is supposed to display the gig description in each event. It's not working. I don't yet understand what's happening.  I will try in the browser console to see if the gig.description is properly feeding into json events.  Then I can try to understand what the eventRender function is doing.


No spaces after (, [ or before ], ). As per the ruby style guide https://github.com/styleguide/ruby


I have retrieved the code of boilerpipe-1.2 , did't find the code of ImageExtractor.class .
raise follow error:
        jpype._jexception.RuntimeExceptionPyRaisable: java.lang.RuntimeException: Class       de.l3s.boilerpipe.sax.ImageExtractor not found
Did you compile the code by yourself?


We should probably use a separate field then -- perhaps `last_run_time`.


Look for readyState with a capital S


 does not work.
maintenance mode ON and do not OFF


@mogoodrich fyi, I did this to fix the build, so I can commit my code from the plane ride home.


JSON.parse not JSON.encode


Thoughts? https://github.com/zathras-crypto/omnicore/commit/2fa8ad0172e752dfbda38951a024d9174107a631

I also need to get my thinking cap on and make a cohesive proposal about what to do about this idea of using "market" on the RPC layer - at the moment it's only used in one or two places...


网页版APP,这里面用到了融云作为即时通信、百度地图API等技术


test ob du schon wach bist :)


Algunos archivos dice : File mode changed , no se que es eso , pero a los archivos los veia igual , no me dejaba subir los otros archivos si no los subia tambien


Will fix it and send it back tonight.


"assert" is only used in debug, right?
When used wrong - in release - you'll get ther error message (on each call) and ...
the rest of that method is executed too.

So ... shouldn't there be a "return" after "assert".


Sign in the user by passing validation in case their password changed.


This line can, and probably should be removed.


Follwing the C++11 way:

using signal_func_t = std::function<void (int)>


You have checked `params[:search_string].empty?` thirce. Can you seperate it and add a method and use it over here. It will remove the duplication.


`FormatMP()` takes a property identifier, not a divisibility flag.


OK, this is going to produce extreme effects! My suggestion instead was to do things how you were doing them before, but weight the contribution of a group to the expected value by whether or not it is in the current list. That way you are still matching the same groups as in your previous simulations, but giving more weight to the groups in the most recent list.


Here once again you checked for empty params same as sort_links method.


Right you are - so up until now ```MetaDexObjectToJSON``` has been giving out bad info?  (check the std::string lines I removed)


Ach so ...
Jau, gerade aufgestanden. ;)


@MeyerJL


It requires http://plugins.jetbrains.com/plugin/7251


`"who likes the story"`


Looks good! :)

The market idea is really an interesting one, and I think it has the potential to create a significantly better user experience, because we may get around these strange "is it a buy or a sell? ..." questions. Nominating all prices in MSC for example was already a great improvement in my opinion, and this seems like an extension to it.

At the same time though, I think we should always keep in mind that it may change at some point, and it might not always remain to be just `token/one-other` trading pairs.


Why did you add this? It isn't used.


Commonly ... the dispatcher basically is designed to be a singleton BUT
you provide a public constructor?

Also ... since it should be a singleton ... you don't wanna allow to derive ... do you? "final"?


in release .... the assert will have been gone.
What's then with the next lines?

Don't you miss a "return" here?


@fuzzytern its displayed when app starts up. You can remove it if you want to. Its there for sites that have a lot of initialization to do before displaying the ux.


008c8c2 ㄴㄴ친구 squash하면서 이전 커밋이 남아있어서 그런데 여기서 수정되어있음


I think so, since recently at least.

This is one of many things not tested at the moment, because the tests focus primarily on logic, but not fields. This is something I'd like to change rather sooner than later though, once the RPC layer is mostly finalized.


It's ok. I still work on the client in my spare time. I moved all those builtin actions in a class of their own. I also wrote a test library that allows for easy reuse of todo.sh's tests. Now it is very easy to work on a feature. No need to figure out how to test.


No.


which means user get updated with new credentials and automatically logged in again. Right?


@fuzzytern the `loading.html` partial is a convenient way to show an overlay when the app is loading (global loading state). For example when the route is changing or data is being loaded. Feel free to refactor it for your app's needs


Please don't add whitespace in unrelated commits.


https://github.com/zathras-crypto/omnicore/commit/3bbe9658395fc236c2a15752a0ce4c4683d2616f

Since ```getorderbook_MP``` uses ```MetaDExObjectsToJSON``` which in turn uses ```MetaDExObjectToJSON``` do I'd imagine our order book has been broken up until now - do we have any tests for that call?


Thanks for doing this, @jroper and @pvlugter! Being able to bind against 127.0.0.1 only instead of 0.0.0.0 locally in "activator run" is way nicer.


We really need to clean this madness up.


1. I don't know the answer to this question. It is a research topic.
2. The time complexity is O(m*m*k) for each instances. m: number of fields, k: number of latent factors.
3. For each pair of fields, we learn a dedicated latent space.
4. Easy to overfitting. Training time is longer than LR/Poly2/FM. Model size is also larger.


Can you do this as if-else?


ALTER TABLEなりの
https://github.com/aipocom/aipo/blob/master/sql/postgres/
の中のorg001.sql及び8000to8020.sql
に書いておくといいかと思います。


These branches are kind of ugly. Can you make a helper function for this bit as well?

Something like:

```
def node_gtkid(node):
     if isinstance(node, ast.Function) and node.shadows:
         name = node.shadows
     else:
         name = node.name
     return '%s%s%s' % (...)
```


Don't hardcode this. Please squash @ptomato's commit in here.


Not a big fan of auto-pluralization like this. Too many heuristics that have the potential to go wrong.


The tests involving the orderbook currently use only it's size to determine, whether a trade was added/removed.


Please stop with the whitespace.


The closing of `<p>`s here at the start is a bit ugly. Are we guaranteed to be in a `<p>`?


std::cout << "Hello World!" <<std::endl;


a


Unrelated fix, please split out.


std::cout << "im bk" <<std::endl;


This is just code clean up.


@vangheem Could you link the failed Jenkins job?

On robot test error, we could execute some inspecting JS code and dump the results, but probably there isn't anything generic to call, which could help debugging these.


The next time I am providing a feature - I do it for wicket-7.x / master branch, so that there is no need for a cherry pick anymore.


The codebase is now GPL3. So, anything going in as GPL2 is now GPL3 (via the "any later version" clause). This is because we have added code in the past with licenses which is only compatible with GPL3 (I forgot details, but this was discussed at that time about 1-2 years ago).


No. Sections files as checked into GTK+, etc. are *not* proper XML and do not look like this. Not to mention that you are missing the doctype and `<?xml version="1.0" charset="utf8">`.

See https://git.gnome.org/browse/gtk+/tree/docs/reference/gtk/gtk3-sections.txt


The logic here is getting really complex. This is sort of why I don't like template languages: you're effectively writing code in them.


The Pandora (aka OpenPandora) is a Handheld Linux Computer. It uses an ARM CPU and use GLES|2 for graphics.


Meh. OK.


Since `input` can be `IO` as well, should probably check `isa(input, AbstractString)`?


Huh. Good catch. Looking back on it, I think that must be how I've always done it. But everytihng points to that being wrong. :smile:


@tisto I shortly looked into Selenium support for getting browser logs. Firefox supports it, but it returns only its global "Browser console" logs. Not window specific "Web console" and it's not possible to log into "Browser console" from untrusted JS code :(


So technically, rubygems is actually adding `$gem_dir/extensions/x86_64-linux/2.2.0-static/ruby-ll-2.1.1` to `$LOAD_PATH` once ruby-ll is activated by rubygems.

    \irb
    >> orig_load_path = $LOAD_PATH.dup
    => ["/opt/rubies/ruby-2.2.2/lib/ruby/site_ruby/2.2.0", "/opt/rubies/ruby-2.2.2/lib/ruby/site_ruby/2.2.0/x86_64-linux", "/opt/rubies/ruby-2.2.2/lib/ruby/site_ruby", "/opt/rubies/ruby-2.2.2/lib/ruby/vendor_ruby/2.2.0", "/opt/rubies/ruby-2.2.2/lib/ruby/vendor_ruby/2.2.0/x86_64-linux", "/opt/rubies/ruby-2.2.2/lib/ruby/vendor_ruby", "/opt/rubies/ruby-2.2.2/lib/ruby/2.2.0", "/opt/rubies/ruby-2.2.2/lib/ruby/2.2.0/x86_64-linux"]
    >> gem 'ruby-ll'
    => true
    >> $LOAD_PATH - orig_load_path
    => ["/home/hal/.gem/ruby/2.2.2/gems/ast-2.0.0/lib", "/home/hal/.gem/ruby/2.2.2/gems/ansi-1.5.0/lib", "/home/hal/.gem/ruby/2.2.2/extensions/x86_64-linux/2.2.0-static/ruby-ll-2.1.1", "/home/hal/.gem/ruby/2.2.2/gems/ruby-ll-2.1.1/lib"]

This seems to be the case for Fedora's rubygems and vanilla rubygems 2.4.5.


fixed...


Merged


Initial Commit


Correct, there is a 2MB buffer reserved for each client. 100 clients means 200MB memory. And yes we need to clean this after each client disconnects to prevent this huge memory leak.


What do you mean by race here?


Have you tried it?


`generator.return()` (WS unless this is clang-format)


@mrshelly what do you mean? Check [documentation wiki](https://github.com/odoo/odoo/wiki/Translations) for details on the process.


http://jenkins.plone.org/job/plone-5.0-python-2.7-robot/lastFailedBuild/robot/

I'm not sure browser logs would help in this case. I know what error is happening--I see it live when it's running. If you give it a try, you'll see the window.alert shown(from the modal pattern)--that is what causes the test to fail.

Thinking it over, I imagine how you'd need to go about reproducing this is building a special plone and plone-logged-in bundle with a debugger in javascript set where the window.alert is triggered. Then, put in a robot debug statement at the beginning of the robot test scenario. Finally, activate the debugger console on firefox  so when robot framework causes that error, you'll finally be able to capture, *maybe*, what is going on.

That's painful though.


rebase 할 때 a0ea388 여기에서 수정했어야 하지 않을까?


It is not very smart yet to put the options to user folder. Eventually it is, but the problem is the asset conversion. It truly is an application setting, not user setting. We have to solve this some day, but maybe not know.


Yes


I linked you up! I must have mistyped ric70x7 when testing it!

Neil

On Sun, May 31, 2015 at 9:50 PM, Ricardo Andrade <notifications@github.com>
wrote:

> The site has been up for almost a month, I should be linked by now :)
>
> 2015-05-29 12:46 GMT+01:00 Neil Lawrence <notifications@github.com>:
>
> > Stop messing with the colors and get the site up so I can link to you ;-)
> >
> > --
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/ric70x7/ric70x7.github.io/commit/df0d7016b3cbdf6cf7227e81de89186e06637573#commitcomment-11425539
> >
> > .
> >
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/ric70x7/ric70x7.github.io/commit/df0d7016b3cbdf6cf7227e81de89186e06637573#commitcomment-11445592>
> .
>


Was reading through and noticed a small typo. quote should be `"best results it is strongly suggested to"` instead of `"best results it is strongly suggest to"`.


Although the install_requires in setup.py does list the msgpack library.


Could you fix the indentation of the comment here please?
This won't run otherwise.


Please add this back.


A question: I have Anaconda in my path so that my python uses all of its packages. Installing *Biopython* in Anaconda should be sufficient for this warning, yeah?


Addressed in ddea2dd99d91a1a82dd0958ca82f3499fb76bebb :+1:


Keep state across invocations of ansi_up so that we can improve colouring across chunk boundaries.


Initial push for Issue 254.


შექმნილი გაქვთ ჰეროკუზე ამ სახელის app?


Added foreground and background colors for regular and bright(intensity) colors.


Yep - `Time` objects can be trivially converted to `datetime` objects (``t.datetime``), and then those can be time-zoned.  That seems the most natural approach to me whenever a local time is called for.  Whenever someone just asks for "the time" (and local or a string is not specifically requested), I'd say just give back `Time` objects, though - less conversion necessary then.


Sorry, I messed this up on the designer.


Any call for this to use subtle's method?


The matching check for now requires that the MP's value fulfills the matching against <b>all</b> of the MMP's current values. An example is for name: MP's name must fulfill the matching criteria against all of the MMP's current names. We may want to change this later on.


And matching with 1000 MP's takes about 1hr 30mins on my laptop...


Just 'master' is OK for me. I keep wicket-7.x up-to-date by regularly merging from master


Context가 안맞긴 하네 ㅠ 근데 이 commit의 일부만 squash할수는 없지 않나?


I vote not to use GitHub's automatic merging of pull requests - that clutters the repo history with redundant merge commits. For example this commit's diff overlaps with the diff of the previous commit.


interactive rebase로 하고 있지 않아? squash 말고 edit 을 해서 저 커밋에 추가하고 amend 하면 됨


[system variable-ების დაყენება შეგიძლიათ travis-ის სერვერზე](http://docs.travis-ci.com/user/environment-variables/#Using-Settings)

travis.yml-ში აღწერ `api_key: $herokuToken`
და travis-ის საიტზე შენი პროექტის setting-ებში ჩაწერ `herokuToken`    `hjbh2b-1jh32b-j1h23k-jn56k`


Don't know how to use automatic tuning. My testing suggests it is better higher.  I just tried 3 values: the patch value, 5 up, and 5 down. It is all within the margin of error. So my results could be in doubt, but it was very consistent throughout.  I am going to try some higher values as well.  This was at very fast time controls. Average depth was bit over 14.  If I get a value that looks best, I will run it head to head and hopefully get enough games to get a reasonable assurance that it is better.

5/31/2015 10:52:18 PM :

    Program                          Elo    +   -   Games   Score   Av.Op.  Draws

  1 SF mbp 25                      : 3402    5   5 11396    50.4 %   3399   43.4 %
  2 SF mbp 20                      : 3400    5   5 11397    49.9 %   3400   44.4 %
  3 SF mbp 15                      : 3398    5   5 11395    49.6 %   3401   44.6 %


Should we check regErr here?


It gets checked down on 689, but that's like... 8 lines away. That's like... forever.


@phallstrom Our team has reached integration limit (i guess it's free plan) so the simplest way for me was to use its api :)


Додав перший issue: https://github.com/greatehop/progit2/issues/6


Виправив.


Il faudrait trouver une méthode pour prendre l'année courante directement plutôt que  2015

Calendar. Get(calendar. Year)

Et on peut faire de même pour le mois et le jour du mois


Дякую, виправив.


Oh, how could I forgot. It's done, thanks!


Виправив, дякую.


the "(just kidding)" really kills it :disappointed:


Дійсно звучить краще, виправив.


Цікаво як я це зробив :) Подивився у вікіпедії, побачив "екран", та написав повсюди "фільтр" :)
Дякую, виправив.


Виправив на "накладати".


Two comments here: I may be missing something since `err` is orange here, but I am wondering why you used `=` instead of `<-`? Second, I can hear Bob saying "Everything is at least a vector." Thus, I think that you can simply assign `'fill'`to `radius_options` instead of `c('fill')`.


use block brand
```
{% block brand %}
<a class="navbar-brand" href="#">{{ admin_view.admin.name }}</a>
{% endblock %}
```


Sorry, I'm probably missing something. I tried to run `dpkg -L postgresql-9.4` and the `/etc/postgresql/9.4/main` directory wasn't mentioned among the installed files. But when PostgreSQL is installed, that directory is present and contains configuration files. What value did you set the `postgresql_conf_directory` to?

I had to run the following workaround:
`sudo pg_createcluster 9.4 main --start`


it's should be its.


Thank you for the clarifications.


http://sum.in.ua/s/literaljno
Розумію, "рідко", проте мені це слово подобається.
буква -> буквально, літера -> літерально. Я набагато частіше використовую "літера", тому мені здається логічним використовувати літерально. Якщо наполягаєш, зроблю "буквально".


The filesystem state can change between you checking if the file exists, and opening it. It's pretty benign in this case (you'll just throw inside ReplayMetadata.Read instead).


changed liek to like


I think a separate watch method in Kv class to which app would pass index
as paramater will do. It would be a responsibility of the app to remember
the previous index and pass it along to next watch call.

This is how watch could be implemented
https://github.com/SupersonicAds/consul-rb/blob/master/lib/consul/client/http.rb#L44

On Sun, May 31, 2015 at 8:23 PM, Kim Toms <notifications@github.com> wrote:

> I'd be glad to do this work, but want to make sure no-one else is doing
> it. And to ask how it should be done.
>
> Approach 1 would be to create a WaitableKv class as described above, and
> leave everything else pretty much the same. Except maybe replace the guts
> of the Kv class with a call to an WaitableKv object which would not wait in
> this case. Maybe to a method like current_value which doesn't block. This
> would also allow access to the flags, the lock index and session, etc. If
> we used a class name more like FullKv we could add locks, indexes, flags,
> and support the full range of Kv functions. I like the current Kv class,
> though, because it has a simple interface. Any suggestions for the name of
> this class are welcome.
>
> Approach 2 would be to modify the Kv class itself, incorporating this
> functionality but with new methods such as wait_get. The issue here is that
> it would be difficult to properly do this. The use of Kv#raw references the
> last key read, but we need a particular key's raw data. It seems like this
> would end up impacting what is currently a simple to use interface with a
> bunch of concerns.
>
> Please let me know what you'd like done. It will take some time.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/WeAreFarmGeek/diplomat/commit/5652b568da402f9cf712aade178012909c74e013#commitcomment-11444560>
> .
>


Do not worry, I have plans for restructure serializers to other location and put each serializer to separated files.


嗯


Broken markdown link syntax in the last sentence


Only question is why use a `tryCatch()` here instead of just throwing a custom error? I know that you modify some of this stuff in later commits, but I still do not know what advantage it gives.


I have removed the getImage() method , because I did't find any relevant source about ImageExtractor in biolerpipe-1.2.


This lists the same bug 3 times...


Hmm, most if not all of these changes shouldn't have happened...looks like something went wrong with my import process with the last PR. Will fix.


@keul adding this locale is wrong. This means that everybody who runs coredev tests will have to install an italian locale. coredev tests are already buggy and fail on different machines, we shouldn't add more of that. Please amend the test to mock the locales setting.

@gforcada could you please revert that commit and push? I'd rather have a failing test, or remove a test than having another locale in our setting.


Duplicate of the above 2 bugs


apache SOFTWARE license ...


this looks good!


Thank you!   I really appreciate the fix!


@vangheem I understood that the alert should be shown only when there's an error (modal form submission fails). So, isn't then real question about, why does it fail? Maybe somethng has failed all the time in tests, but because the previous impl. just console.logged it, we haven't seen it.


dd


Maybe fold this condition into the loop condition instead?


Exactly. It only fails when runs in robot framework full speed. That's what I'm trying to explain. If you put a Debug statement in, run the commands manually, or just put a debug statement and continue, or manually run completely in the browser, the error does NOT happen. That is the whole point of why this is incredibly difficult.


@shangguokan What about installing Composer I think you're right, so will better in contribute section. But composer.lock file should be in this repo for same work environment


That's a pretty common problem then. You need to make sure that every single element you are interacting with is visible before you are actually committing an action. Maybe the "patterns are loaded" keyword is just not working as expected.


events() is a method defined on the EventRepository interface to return index of all upcoming events. It is an object and all the events are in a value called "events" - which is what is returned rather than specifying that in the view every time.


Done.


Is ```+ 1``` removed by accident?


No need to update ```n``` in the dict entry?


Tried it. "patterns are loaded" were in place. I even tried spamming "Sleep  2s" in a bunch of places. Nothing worked. Maybe someone else will have better luck.


Should revert the bad changes in 3bd0fad.


That seems to be incorrect, as we use <h1> for top-level section titles.


@avsej I kindly ask for providing some kind of description behind this commit, especially as this is effectively revert of my commit e31bca55fae7e451a2da76fa0b64dec8562f9573 (this SHA ref should be given here as well!). I don't want to argue why are you reverting, but pushing revert without ANY description given is just very bad for other ppl involved in the project absolutely not knowing what is going on.

@araq I humbly ask you to not accept any PRs without good commit description. Even you knows what is going on, this doesn't apply however to anyone else.


It is not a memory leak, though. The garbage collector will collect it at the appropriate time... If you wish to have the memory back to reduce memory usage of the server, we should really put GC.Collect(); at the end of the server's disconnect function after disconnecting all those clients so we can save lots of CPU and get the same effect.


올 해보겠삼


@Araq Can you please tell me why you have switched `+x`? I am working on many projects that are checked out on Windows and `+x` doesn't do any charm there. So I am very curious about the reasons of this switch.


commit


No, we don't get the same effect. Try sending a reconnect command to all connected clients and see the memory usage (with and without the GC.Collect). It's not about the server's disconnect method.


You should be able to do this without six also, using regular `dict.items()`. It's not supported on Python < 2.7, but that's the only Python 2 version we support anyway.


LGTM!


The latest version has a downgrade option. It was not 100% tested however unlike the update option. I'm also not sure if new titles which don't exist on older fws will cause problems because i can't really detect these and uninstall them. Try it out i guess.


There are two things to consider:

1. I used previously the `inspect_image` API call output as metadata for the squashed image, but it seems it's **different** from the JSON prepared for the image. The difference is described in 2).
2. The correct metadata consists from a **mixed** camel case and underscore, this is why we need to do: `metadata['config']['Image']` instead `metadata['Config']['Image']`.


Logs from user failed HEAD request http://pastebin.com/NeXfTje2 and same without errors after changing request method to GET http://pastebin.com/tF9X7cwE


I don't think this is a good idea. The purpose of unittests like these is to be predictable and not be influenced by outside environment variables such as the configuration you normally use for the bot.

If you use a different `BOT_PREFIX` in your config, it will now break all the tests for example because they all use `!`.

If the goal is to test something with a different config setting, it should just be mocked in the test that needs it, I think.


There is a bug here because implementers of onNext are allows to queue items.  If they have a queue size of 10 they issue a request(10).  However because this loop is reusing the buffer, such a queue could end up with 10 entries, each pointing to the same wrapped byte array


Requiem
there is one "u" too much ;)


OK


Oh, right. I was just thinking about a situation of clients being told to disconnect (multiple) at once via the server's handling... I see.


/CC @kasper93 @Underground78 for the release highlights. Note that I have uploaded the binaries on SF already so the faster we have this the better.


Remove empty line :)


Daran hab' ich bisher gar nicht gedackt, danke. Done!

--
:www
   www.marioonline.at
   www.computercamp.at
:gpg
   0x1B7EFD90EC2CFFA1

On 2015-05-30 14:16, Andreas Hubel wrote:
> @MarHai [1] Update doch auch mal die Koordinaten in der Karte.
>
> --
> Reply to this email directly or view it on GitHub [2].
>
> 	*
>
> Links:
> ------
> [1] https://github.com/MarHai
> [2]
> https://github.com/MarHai/codefor.de/commit/59b79013e9460bd4ffddfded6108758c538cf368#commitcomment-11439394


ez mi he ??


Where is the source of this new risno/elasticsearch image ?


De jónak tűnik .


hi @gabriel-bezerra, I replaced it with version badge, which will be updated automatically as soon as release published, could you please check screenshots below ?
http://prntscr.com/7brlyt
http://prntscr.com/7brmg9
http://prntscr.com/7brmln


It's a copy from the previous test.


Script causes the following error:

cp: target ‘/home/nathan/.quark/sysroot/usr/include/quark’ is not a directory


While actually, there is even a screenshot of the poll results available online:
http://thefish.8m.com/cgi-bin/i/pics/cn_pwad.jpg


Causes an error:

cp: target ‘/home/nathan/.quark/sysroot/usr/include/quark’ is not a directory


lacika állj már le pls nem itt kéne ezt


Can scientificBuilder produce text outside of ascii range?


Magadban beszélsz?


test #545


Because of the way InternalAgentBehaviorImpl is instantiated in AgentBuilder.


Maybe this should be exported, to help users with custom type instances?
But maybe it's better to wait until this module stabilizer or someone asks for this function to be exported.


Uptream was trying to fix this bug :
https://github.com/hexameron/dl-fldigi/commit/9520568563d6c44ac4f548e103f8fe6b95652872

but what they should have done was this:

https://github.com/hexameron/dl-fldigi/commit/8d64fe84ddc4167679a7c8db86cd11329ceb4551


Can this be merged into the 1.22 branch? If not, which release will contain the fix? It would be nice to have for auto-detecting the right configuration of the OpenGLRaw package.


Are you sure this case is needed and is not covered by the case just above `((string-match "\\```-*\\'" (match-string 0))`?


OK


The original name of the status variable was originally z_minus, the member was not updated when the variable changed name, I will make the corresponding changes.


Ha elfogadja,meg ipythonban kipróbálod és mükszik,akkor nem lehet gond.


I see your point, this is terrible for those local unittests.

I was doing that for my plugin-tester as I need the same bot twice.


-MPMovieScalingModeAspectFill
-视频缩放到外框，4比3的视频上下会被削掉一部分

但是我只想按要求，一个方向会被削掉，而不是上下两个方向都会被削掉。


个人的Demo库,主要就是些常见的项目中的模块Demo,以及常用的工具库,后续会慢慢完善, 分为Ui库和工具库


You're right. Feel free to commit the change.


Yes, n is the number of new connections. I will add a suitable comment for it.


because otherwise the new null constraint would be violated when saving the first time as start and end date are not set. besides, the code was saving the instance two times.


causes error at the moment tema
frameworks/base/core/java/com/android/internal/os/BinderInternal.java:116: error: cannot find symbol
        VMRuntime.getRuntime().requestConcurrentGC();
                              ^
  symbol:   method requestConcurrentGC()
  location: class VMRuntime
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: Some input files use unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
1 error


I see. Removed.


Agree, I will add comprehensive comments to describe the growth curve class and its children.


В описании не нужно описывать, как реализована функция, нужно указывать, что она делает и для чего используется. Т.е. здесь достаточно сказать, что функция запускает игровой цикл.


This is for cases like `/--`.


OK.


Потому что для нормального их использования нужно подробнее рассказать о вложенных функциях, так как могут возникать ситуации, где они ведут себя не так, как ожидается.


Ingat field_images sudah dihapus ya! Jangan di add lagi..


@vangheem Would it help, if Robot would optionally pause when test fails?


E.g. by inserting "Debug on failure" -keyword.


Нужные вещи ты мог бы просто импортировать из другого файла, а не копировать сюда целиком


just note, don't add user.props to solution (as i wrote on forum). this will cause internal git client in visual studio to push it back. always modify user.props outside opened solution.


Видимо, это не вопрос, а самонапоминание =)


Wow, that's spooky. I did some work on this file in the train this morning (from the previous commit) and the archive classes are almost completely the same :P


itt egy tabbal visszább kéne tenni


It might help. :) sorry, I need to sleep. You Europeans are all just waking
up!
On Jun 1, 2015 2:08 AM, "Asko Soukka" <notifications@github.com> wrote:

> E.g. by inserting "Debug on failure" -keyword.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/plone/Products.CMFPlone/commit/6585d8f35ecdc509cb19440221bbc26c2abf4bee#commitcomment-11448630>
> .
>


Sure, it can be done.


PGM


Baru juga dibahas. Ingat field_images sudah dihapus ya! Jangan di add lagi..


I've just seen this commit to OpenGLRaw's fork, and I'm thinking about how to integrate it upstream. Let's break this apart a little bit:

* Current cabal versions don't detect Android, but there's already a patch on master which does it (haskell/cabal@058959a4218ac48a45b91191abb065db66a3d120). So the `flag android` is basically a temporary workaround, and it should default to `false` (not `true`, as above).

* On Android we need to link against some OpenGL ES library. You mention GLESv2, but IIRC newer Android versions ship with GLESv3, too, so we might want to make that configurable. What Android version are you using?

* The above patch mentions `-DUSE_DLSYM`, shouldn't we use `eglGetProcAddress`?

* What's the reason for `-DANDROID`?


dejó vagy


ts


ah I was originally going to use [lstatsync](https://nodejs.org/api/fs.html#fs_fs_lstatsync_path) to check if the `.npmignore` existed, then remembered I could use `this.src.isFile`. So that line can be removed.


This should have worked before, since `addNewFile` is the same as `addNewFileWithStatus(... "Done")`.


@fpietka I see that you used ENT_SUBSTITUTE as one of the flags, but that has only been introduced in PHP 5.4, while the 2.1 branch has minimum PHP requirement 5.3.3


Good :) The magic line is

    Register keyword to run on failure  Debug

We just need to document this...

http://rtomac.github.io/robotframework-selenium2library/doc/Selenium2Library.html#Register%20Keyword%20To%20Run%20On%20Failure


/cc @cvrebert @mdo for feedback about the wording.


Leftover ?


Not sure PHPDoc is right here. Storage gateway is not here.


Forgot to erase it


Yes, sorry for confusion. I mean lastlog (my previous comment updated).


Hello！ I used GCDAsyncSocket to connect with server in Tcp.  For some reason ，I need to receive  message from server without sending message to server and getting response data. How Can I listen to the server?Does GCDAsyncSocket  can do this？Please help me! Thank you !


Вероятно, использовать считывание клавиши из консоли будет не очень удобно: используй считывание в окне:


typo, param is ```$aliasNames```


Really needed ? :stuck_out_tongue:


That would be great. Is there already a schedule for the next cabal-1.22 release? I'm currently trying to figure out when to release what... :smile:

Just for reference: The reason I'm interested in this change is that I've just seen sseefried/OpenGLRaw@4a24f2dbbec674e018608d10852ce50259f14b91 on an OpenGLRaw fork, and I'd like to provide out-of-the-box support of OpenGLRaw on Android. Using a flag is OK as a workaround, but something automatic would be nicer.


This code cannot be here.
But anyway I will move it to correct position, thanks for fixing :-)


@mike-lang find the revert and change of order in this commit:
https://github.com/db-migrate/node-db-migrate/commit/03a494d81fb264a3005d5a862e81f31ae9b6db6b

was also applied to v0.9.x and published with v0.9.16


I get a [build error](http://universalmediaserver-ci.hopto.org:9999/job/ums%20trunk%20(nightly)/110/consoleFull) when building with 'mvn clean package site -P linux'. I haven't investigated which commit causes this error but suppose it is this one as it is related to cobertura.


maybe you should handle the basic_publish in similar way though this is not the current bug ....


looks ok


unneeded blank line? :)


Spelled canols oil wrong


Создавать окно при каждой отрисовке неправильно


There is actually no need at all to depend on $timeout since it isn't used any longer. See the whole file.


i'd add phpdoc for this method


Hmm...


Вместо этого создай функцию, которая по координатам в масштабе поля создаёт прямоугольник с нужным цветом и размером.


LGTM


:smile:


@xu-cheng, libtool provided by OS X doesn't work here, it needs glibtool instead, but I don't find a tool named "glibtool" with command "which glibtool". Neither does brew supply a  formula named "glibtool", but brew has "libtool".


鼓掌！


dev is free'd, this can never happen.


It is a great pleasure for issuing comment on your site. Very informative and educative at all. Thanks


Looks great, thanks! No need for the semi-colon here though.


@datakurre @vangheem I will schedule a test team hangout to discuss how we could further improve our acceptance test story.


@pecko - what does this next commented out block represent?


haha :D


Nice!


lol

@leofeyer I totally agree with this change, but according to our discussion for `ContaoAuthenticator`, this would be a BC break too?


@pecko  - I'd like to include the o.m.asset classes in the auto-test, please.  This will avoid a headache in the future.


I suppose it's because it's on it's own yes. Is that wrong? Where have you corrected the other two?


brew's libtool  is glibtool.


Uhm, do that null check at the start of the method.


Hi @slygent, thanks for this! Are you planning to add a screenshot, or do you want us to make one?


How about something like `O(|x|^6)`, instead of `O(x_1^6, ..., x_n^6)`?


@pecko  - I think it would be tidier if SensorWrapper specific code was in the SensorWrapper class.

Isn't it possible to generate these objects in SensorWrapper?


No, because the listener is `@internal` (which I forgot to add). Also, @Toflar said that event listeners are not part of the API in our discussion last Friday evening. Not sure about this though.


that was a last-minute copy-paste change... fixed


I would, but I cannot rebuild the playground successfully right now. I've described the issue in the Gophers #gopherjs channel. Maybe you could take a look, and fix this at the same time?

I'd rather not fix something if I can't test that my fix actually works correctly, and I can't do that now due to the aforementioned issue.


Oje :D Na ja, geht wohl nicht anders.
Das `.equals("")` könnte man noch optimieren durch `.isEmpty()` (→String-Vergleich entfällt und es muss kein neuer String (`""`) erzeugt werden).


I have a method of picking out the titles that need to be installed and have been doing it successfully on about 10+ units. It just takes forever with FBI and your way would save about an hour of time.

If you want to know how I can tell what files need to be on the sd let me know. For example to go from 9.7 to 9.0 you need like 47 files but to go from 9.5 to 9.0 you need 34 files


Done -> https://github.com/droolsjbpm/jbpm/commit/465c57b22924f3c8c70a5fb7668cb346cee07eab


Das scheint es anders zu lösen: http://stackoverflow.com/a/1739037/3157536
Hab gerade kein Zeit, das zu evaluieren.


It's fixed in what I pushed to Alioth


anyátok

2015-06-01 9:15 GMT+02:00 Jánosi Dávid <notifications@github.com>:

> dejó vagy
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Palfy-2016e/prim/commit/3e37c09d4e10aa8cb038705b83a20beca9f0d0e3#commitcomment-11448703>
> .
>


Why is it wrong to store binary data in std::string? It seems to me that it just makes our API easier to use for novices. I'll admit that function could be optimised a bit there. But in principle I see nothing wrong with std::string usage here.

When you read from a uart you have no guarrantee that your requested read length will actually be read. Would you rather a biger buffer with some garbage be returned?


> Also, @Toflar said that event listeners are not part of the API in our discussion last Friday evening. Not sure about this though.

I didn't say that. I said we'll have to check the Symfony BC promise on that. :)


yes we should be writing data.length() there to mraa_uart_write.


I'd quite like to refactor this part of the testing strategy.  FixWrapper.testMe.testMyParams includes code to generate the relevant type of object (line 1273).

Can you explain the benefits of creating the text subjects here in this class?

I haven't really considered the best logic flow for ensuring we test every class.  But, how about if the Editable interface included a "getTestSubject()" method?   We could then loop through all Editables, retrieve the test subject, and run the properties test.

We would need to ensure that sub-classes still provide their own test subject (not just the one from the parent).  So, we're check that the test object is of the same class as the subject class.


Ok, my bad.

BTW, did you guys install the Slack app on your computers? There is something to discuss :)


buat apa ditambah comment


This is so close to what the datapackage.json file contains - why not just use that directly? :) Good work on abstracting this out though.


Could you please explain why this was done? In my opinion this doesn't add any value.


Shouldn't line 461 be sizeof(char) * (strlen(string) + 1) and on my system I had to change the #include <string> to #include <string.h> ?


If I set debug: false in redwood.json RedwoodConfiguration will be true not false. Added Default value true to RedwoodConfiguration class fixed this.


To have simpler diffs (e.g. here the first array item is part of the diff whereas it shouldn't) could you always add a trailing `,` in arrays


meleg vagyok :DDD


Why is this one removed?


Changed RenderSelfClosingTag to RenderBeginTag. Added some newline and Indent into HTML code formating.


you dont necessarily have to do this here:

constants.hero_names[(npc_dota_hero_something}].id should lookup the id given a name


There is still no need for doing this - see earlier comments. You can access that via LockStatementSyntax properties more easily.


lgtm


why is this nested?  If an array of tuples, can't you just start with [] and push [x,y,z]?


Looks good to me, thanks Matt.


That's a bit too complicated - just use binaryExpression.DescentendNodes.OfType<MemberAccessExpressionSyntax>.Where ... etc.


First cut organisational design radar


should go on a new tab?  @albertcui ?


@xu-cheng ah, that's it! thank you


beware of case where match.parsed_data doesn't exist (a non parsed match).

I usually deal with this by placing parsed data within match.players.  Each player has a parsedPlayer property that contains their parsed data.


Consider adding more than one syntax node action - then you do not need to handle 2 different cases in TryGetDiagnostic


On second thought you may not need to do this, I am not doing it above with teamfights and a crash here will only 500 the page without bringing down the server.


Okay - then only for master. :-)


You may need to worry about the case of 1v1 matches.  The match.parsed_data always contains 10 players, if 1v1 I believe the populated slots are 0 and 5.  This may not be important enough to deal with, though.


CC @Underground78 because we need this documented clearly. I forgot to upload the drdump files, I'm doing it now.

And we should really look into automating the process at least for stable builds. I'll see if I can make @kasper93's patch work.


assert will be used in release builds and will always terminate the process.


Done, it was counting properly, but I've added elsif, so it only add game
to game master if he doesn't have any heroes. Now second condition is being
checked only if game master have no hero.

2015-05-31 13:39 GMT+02:00 Jakub Kubacki <notifications@github.com>:

> Nice! What about game master with hero in game?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fantasygame/campaigns/commit/3e472b93ded4666030b694d299b8f8fba3afc04f#commitcomment-11443128>
> .
>

--
Pozdrawiam,

Małgorzata Grobelska


Is that behaviour wise? That would probably confuse people.


I'm in hurry right now, maybe mention proper color conversion for subtitles and DVB HEVC broadcasts support. Also DTS-HD support with all extensions. I don't know if playlist changes are worth to include.


it's better to check width and height too. if size is changed, recreate pbuffer surface is required i think


Incoming.


So if I remove a buddy from my contact list then I shouldn't seem them on there anymore with this commit correct? Because I still see people I've removed on my contact list after restarting Pidgin.


I think we need to mention the playlist improvements too.


Interesting to append "just kidding" after the message that informs about formatting the c:\ drive, but not, for instance, after "Mailing wife about female contacts" :D


[Huber](http://en.wikipedia.org/wiki/Huber_loss), nicht zu verwechseln mit [Hubbard](http://en.wikipedia.org/wiki/L._Ron_Hubbard).


why does this module depend on `../ajax` but doesn't use a reference to the module in the callback a line below? Is this related to the buildsystem?


Maybe, perhaps :)

It's what the official client does when you remove someone, but I haven't
seen first-hand if it actually does what it says

On 1 June 2015 at 19:54, Gethyn ThomasQuail <notifications@github.com>
wrote:

> So if I remove a buddy from my contact list then I shouldn't seem them on
> there anymore with this commit correct? Because I still see people I've
> removed on my contact list after restarting Pidgin.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/EionRobb/skype4pidgin/commit/95d3f25b3625b00e9f519adbb439a59319a13bd5#commitcomment-11449173>
> .
>


Hi Thomas,

I haven't checked everything, but two things stands out:
- Attributes.sameValuesAs currently relies on communities to be sortable, so Community._cmp_ should
be defined.
- bagpipe code does sorting on RouteTargets, for easier output reading, so keeping _cmp_ there would be nice

Additionally, Bagpipe currently does not rely on it, but t seems to me that Attribute.eq should actually look at the packed attribute value (not only id and flags), no ?

Also, in a few place (at least localpref and med), implementing cmp would make sense.

-Thomas


We usually create a factory for all the REST API calls. So you can do something like:

RoboconfAPI.shutdown(appName);


Not really. If it's not there, then nothing works. It was commented out for testing :-)


There's not GameLib.GetUnitByName Only GetPlayerUnitByName


Indeed, it was refactored later.


Good idea. I will add an issue about it and implement it later.
And this will be indeed easier to mock it.


Awesome, thanks!


these truncates should really be in `@After` method. They are useless before the first test and no clean up is done after the last one.


Why do you put to noError when there is an error?


Same comment for factory


Same factory++


2015-05-31 13:49 GMT+02:00 Jakub Kubacki <notifications@github.com>:

> Indeed [image: :+1:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fantasygame/campaigns/commit/191f3df95c489d8303847ffa8607a9ad3ca21872#commitcomment-11443159>
> .
>

--
Pozdrawiam,

Małgorzata Grobelska


Now I'm also working for changing myisam seq tables to innodb for our projects which use spring-batch.

Is this enough? [MySQLMaxValueIncrementer.java](https://github.com/spring-projects/spring-framework/blob/master/spring-jdbc/src/main/java/org/springframework/jdbc/support/incrementer/MySQLMaxValueIncrementer.java) says **The table type of the sequence table should be MyISAM so the sequences are allocated without regard to any transactions that might be in progress**


I noticed you are using spaces instead of 'tabulation'. Why?

About whole meaning of this pull request. If there's someone already working on it, that's fine but it's not public. So I had to edit ti by myself and I don't mind sharing it right now, because it's without a doubt more useful boss mod then before.


Lines 65-74 should raise an issue.


This code has been deleted.
I don't understand why you are commenting it. :)


@nanoant sorry about that. @Araq and I discussed this in IRC. In general using submodules require additional step when you are making changes to external repository. It was hard to ignore the `csources` products properly with previous implementation, so it was like always dirty state. With current state it is simple to ignore everything in `csources/` directory, and also to get recent version, you only need to update https://github.com/nim-lang/csources


This "else" clause is useless. Nested conditional expression will be visited by visitConditionalExpression() as you have "super.visitConditionalExpression(tree);" inside.


My bad :(  I'll add a context to the mutator and also the relevant obsoletes!

I must have gotten carried away by the joy of removing all that shitty code :)


@RPCMoritz looks good. I will merge it once I am done with #131


http://irclogs.nim-lang.org/25-05-2015.html
http://irclogs.nim-lang.org/26-05-2015.html


This commit https://android.googlesource.com/platform/libcore/+/a57efa2083afd417b0850a0e2a904780c90e1e1d%5E!/#F0 is missing in CM sources..


couldn't you use containsOnly here?


couldn't you use isEmpty here?


OMG, sorry. Djerba syndrom

2015-06-01 10:05 GMT+02:00 Vincent Zurczak <notifications@github.com>:

> This code has been deleted.
> I don't understand why you are commenting it. :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/vincent-zurczak/roboconf-web-administration/commit/625b4d681566b1e5fc8667a9f88349100e558d0f#commitcomment-11449276>
> .
>

--
Christophe Hamerling


Does Junit test fail?


couldn't you factor `issueAuthorizationHits.get(0).sourceAsMap()`?


But I see how it ensures a known environment... I'm wondering...


Changed a bit after your suggestion, now I can load only if not ```nil``` and ```Bundle.ACTIVE```.
I think is the same as the little monitor below while you are waiting for the workbench to be running and the only state where you can actually do something...


чудно


So if I understand it correctly I can remove (BigDecimal,int) specializations in favor of (BigDecimal,long)?


Do you guys have a sparc machine to test, or is this just guessing?


this change in code does not have a corresponding change in unit test?

coverage was probably lacking, but you need to fix that now


Ok.


I don't see why you need such complex logic here. If I don't miss anything, to implement this ticket you need just
```java
  @Override
  public void visitConditionalExpression(ConditionalExpressionTree tree) {
    visitExpression(tree.condition());
    visitExpression(tree.trueExpression());
    visitExpression(tree.falseExpression());
    super.visitConditionalExpression(tree);
  }
```
Anyway you have a good catch with considering PARENTHESISED_EXPRESSION inside visitExpression()


that won't help when log4cplus is statically linked in a case like ours . there is something similar in global-init.cxx line 650 for static destructing but the problem is that TlsAlloc is always called and freed by
log4cplus::thread::impl::tls_cleanup (log4cplus::internal::tls_storage_key) -> TlsFree only when LOG4CPLUS_THREAD_LOCAL_VAR is defined.

e.g. when compiling for win 7 we have _declspec(thread) but we still allocate tls with tlsalloc then in the destruction we don't deallocate it because LOG4CPLUS_THREAD_LOCAL_VAR is defined.

another possible solution might be removing the #if ! defined altogether and just deallocate it even though we never used the tls slot.


Please normalize how key definitions look.


alpha 1.0 for #14


Oh, I not suggested to use it right now, I just asked to consider it, sorry if it annoyed you.
task_resource_vfs may be expressed like
```
struct task_vfs *task_resource_vfs(const struct task *task) {
     return task_resource(task, task_vfs);
}
```


I couldn't find the corresponding struct in hal-msg.h.
It seems this is only used for server_send_response cmd, but the pdu unpacking here doesn't match struct hal_cmd_gatt_server_send_response in hal-msg.h.


you can use `.find('.widget-header').text();`


Fixed.


So it can "toast" now? Nice! And why must it run on the UI thread?


Will that limit work correctly? We've seen cases where it doesn't.


done


`Arbeitssolln`, is that a typo?


is page number is invalid this would return a legit bad request response


QtCreator does so when I use "Toggle comment selection" on some lines.
Fixed.


can "if len(h.config.Password)" and "else" condition be merged to one reason as "invalid password"?


As you are working fulltime on this PR, I would expect all the small comments to be addressed. I guess that it's already the case in your code. Can you push it so we can be updated?


yes I can :)


Indeed


yes


TeamCity xBIM :: Xbim Geometry :: Release [Build 86](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1781&buildTypeId=XBIM_XbimGeometry_Release) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 74 Build time: 03:32:54
### Failed tests
```
VSTest: GeometryTests.XbimModelExtensionsTests.CanGetRawGeometry: <no details avaliable>

```


TeamCity xBIM :: Xbim Geometry :: Release [Build 87](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1782&buildTypeId=XBIM_XbimGeometry_Release) is now running


Any reasons to duplicate most of the codes in ANDROID_VERSION >=21 and else?
Why couldn't we write similar as
#if ANDROID_VERSION >= 21
  status = btgatt_interface->client->connect(client_if, &bdaddr, is_direct, transport);
#else
  status = btgatt_interface->client->connect(client_if, &bdaddr, is_direct);
#endif
here?


No changes in the solr port should be committed


Well, I prefer the new code to be better than existing =) However if you want, you may add javadocs to existing methods also


In fact I remove it because it should not be the responsability of this method to check existence of the component.


shouldn't it be named g_weakGLSL?


Did you not push to master branch by intention?


Do we really want this to be commented?


namespace(name) |> :hello_client.call({method, args, []})


haha, At this rate the deadline is not going to be met


:+1: I love consistency between browsers.


fff


Если этот метод не требует авторизации вам еще нужно добавить третий параметр в _vk.Call, вот так:
```csharp
_vk.Call("users.get", parameters, true); // <-- добавил true
```


I corrected these two in patch-11 (#87)
https://github.com/torchtrust/liblouis/commit/9a34f72db7d73ade03d229614d0ff90a9f19e6dc

I thought that's where I should correct it?
Can you correct the test harness, then?
Thanks
Paul

On 01/06/2015 08:27, Bert Frees wrote:
>
> I suppose it's because it's on it's own yes. Is that wrong? Where have
> you corrected the other two?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/liblouis/liblouis/commit/9a787fbc95a36c8f840096c5bb2d7b54fcc65009#commitcomment-11448862>.
>

--
Paulw.torchtrust signature

Paul Wood, Chief Technical Officer
*Torch Trust*
Torch House, Torch Way,
Market Harborough, Leics. LE16 9HL, UK
Direct Line: *+44(0)1858 438269*
Tel: *+44(0)1858 438260*, Fax: *+44(0)1858 438275*
Email: paulw@torchtrust.org <mailto:paulw@torchtrust.org>
Website: www.torchtrust.org <http://www.torchtrust.org/>

____________________________________________________

Chief Executive: Dr Gordon Temple
Charity No. 1095904

Privileged/Confidential Information may be contained in this message.
If you are not the intended recipient please destroy this message
and kindly notify the sender by reply email. The computer from which
this mail originates is equipped with virus screening software.
However Torch Trust cannot guarantee that the mail and its attachments
are free from virus infection.


Typiskt dåligt...


V1.0 release


I prefer put it in the @before because it's a good way to see the "prerequisite" of a test.


What is parameterize? I guest this has nothing to do with the "admin" change"


why does container belongs to image? you probably meant "has one"


Screenshot:
![tdash](https://cloud.githubusercontent.com/assets/10080039/7909226/f89a0380-084a-11e5-8725-2aa68497e410.jpg)


There is a security issue of this lib, see https://nodesecurity.io/advisories/marked_redos


As we agreed last week, the change to $3 to $8 needs to be discussed first.


Hm... still issues with white spaces and merging - please try to fix this on local instance


@tnarihi Hi, why to change this if channel_shared? (new to caffe, thanks)


par**s**ing test surely?


Isn't 'hash' undefined at this point?
```
/…/bittorrent-dht/node_modules/k-bucket/index.js:225
    if ((id.length <= bytesDescribedByBitIndex)
           ^
TypeError: Cannot read property 'length' of undefined
    at KBucket.determineBucket (/…/bittorrent-dht/node_modules/k-bucket/index.js:225:12)
    at KBucket.get (/…/bittorrent-dht/node_modules/k-bucket/index.js:258:18)
    at DHT._onPut (/…/bittorrent-dht/client.js:485:27)
```


It seems the read_pdu_at here doesn't match the struct hal_cmd_gatt_server_send_response in hal-msg.h.


This formatting is kind of funky. What IDE are you even using? :P


@myselfjivan : we use Snake case in xml.
Always check the naming scheme used in the file and continue the same naming scheme for your variables.
http://en.wikipedia.org/wiki/Snake_case


Any reason to not use `exportrecipe`?


Yep, stupid typo


This seems to miss clientversion.h.


no, redis distinguishes these


Ragazzi ma allora quando la mandiamo la mail con la consegna? oggi io controllerei gli stili e li sistemerei ancora e poi devo verificare se servono dei pattern da qualche parte


Copy that.


looks good to me


HelloWorlt


shoule we call AUTH() before call doSyncFromMater? other than set c.authenticated = true directly?


These used to be related to the use of a high precision timer, probably can be removed now.


The "Points" variable breaks Java naming convention. Please change it to lowercase.


ok, got it


This is correct. It is rough with no guarantees of accuracy.


What is this commit for?


Yeah, this was the simplest implementation I could come up with to detect spikes (single frames with high values). Probably could have a better implementation showing highest value in the last X frames.


This also fixes the security problems in production with password hashed and adding salt


Seems legit


fix this


Yep, you're right. I encountered this while I was testing the component and added cross domain headers on the remote server to fix it. I was considering adding an endpoint to the host server that could act as a proxy, and return a JSON object to the component. This would be similar to how dashing uses Ruby to collect data and format it for JavaScript presenters.


When I do 'git pull' I get the bootstrap.sh file in my windows directory and by this action the access rights are modified already. So then bootstrap.sh is marked as modified by git and of course 'git checkout bootstrap.sh' cannot undo my "modifications". So ... I cannot commit/push anything until I committed my "changes" to bootstrap.sh. No idea why that is no problem for you on Windows.


kodi :+1:


General comment - please use tabs instead of spaces to indent lines.


panc 胖次
百度云网盘搜索引擎


Will need bump if new package required after adding new shaders.


What's the error on jenkins? Do we activate the virtualenv there?


give some name to 1 and 2, instead of using the figures.


Removed.


> Any reason to not use `exportrecipe`?

This script is standalone and `exportrecipe` is not available here.


- (void)presentViewController:(UIViewController *)viewControllerToPresent
                    foldStyle:(MPFoldStyle)style
                   completion:(void (^)(BOOL finished))completion;

- (void)dismissViewControllerWithFoldStyle:(MPFoldStyle)style
                                completion:(void (^)(BOOL finished))completion;


fix this.


no, here we will receive commands from master, we can not send any command to master from now on.


Ezeket nem kellett volna. Hiába tettem be utána gitignore-ba, ignorálja az ignorálást. :(


Add directories of Researches & Docs


nit: btgatt_gatt_id_t char_id, desc_id;


Here we get an unused warning for `currentPhotoIndexPath`. Can we just remove this line?


Prefieren haga un ticket con referencia a todos mis comentarios de este
commit así no se pierden?

No se que es un WIP, work in process? Algunos comentarios igual son válidos
y prefiero que los vean ahora y no con el trabajo listo :)

Saludos
Mariano
Am 31.05.2015 23:42 schrieb "FrancoSalinas" <notifications@github.com>:

> Hola Mariano, es un WIP. Lo subí porque hay un nuevo integrante en el
> equipo y él se va a encargar de este archivo. Es muy difícil entender la
> función de este archivo porque falta todo el resto del código en el que
> venimos trabajando. Espero subirlo durante esta semana. Perdón por el
> desorden. Saludos.
>
> 2015-05-30 15:07 GMT-03:00 Mariano Cerdeiro <notifications@github.com>:
>
> > mejor seria hacer 2 asserts?
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/ciaa/Firmware/commit/d349ab31d26c50ba1914fd1a1425b77d891cf432#commitcomment-11440524
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ciaa/Firmware/commit/d349ab31d26c50ba1914fd1a1425b77d891cf432#commitcomment-11445794>
> .
>


PC edit and upload.


@avsej @Araq but did you anyone of you read why the submodule was actually introduced? quoting my commit e31bca55fae7e451a2da76fa0b64dec8562f9573 desc:

> 1. Solves a problem that updating Nim sources we have no indication we need to update csources as well.
> 2. Using submodule we can explicitly request that some Nim version requires some csources version to bootstrap.

Does 1. and 2. somehow became irrelevant?

1. Did you invent any workaround for point 1. !? Because I see none, since we basically came back to square 1.

2. How do you know you can build any (earlier) version of Nim with latest `csources`!?

   Violà, you just removed safety feature allowing you build any previous version of Nim from Git repo, just when Nim is heading first stable release. Example, you want build version 1.0, but latest 1.2 `csources` pulled, then you get errors on phase 1 build since 1.2 compiler no longer accepts some internal stuff (eg. syntax) removed later. Good for anyone? Doubt it.

Provided IRC logs only shows that @Araq took the decision, but there is no explanation on the decision itself. But some arguments found on these IRC logs are:

1. `csources` shown in dirty state is fault of submodules

  Untrue. Actually I did PR and it hangs there for over 2 weeks, look: https://github.com/nim-lang/csources/pull/16 It is `csources` repo responsibility to name its temporary files, not `Nim` main repo.

  If I checkout `csources` and trigger `build.sh` then my repo will get dirty too, when it shouldn't. But hey, it is no longer submodule, so who to blame?

2. Submodules require manual update of ref SHA when `csources` are updated.

  This is good for anybody, since the workflow is that you update SHA ref once you make sure that this SHA work for you project. But I guess some devs prefer something will blow up than making just one extra check & commit once for a while, especially when `csources` are meant to be updated once for stable release.

  Yeah, lets save some 2 minutes, and then waste 2 hours in the future figuring out why someone fails to compile some Nim version from pristine Git clone.

So wrapping everything up, if you end up with your submodule screwed, showing dirty state, then it is in 99% your fault not submodules.


@tm1990, can you please add small example in ```src/main/example``` demonstrating how to use this new multiple-points feature?


I don't think this is necessary anymore, given that the prototype has been split back out from the main codebase, and so Jenkins no longer needs to build the prototype.


fixed


It's very important changes!


When testing an hybrid mobile app using Appium, one need to be able to control the context to work with.
By using the context actions added to protocol.js, it is now possible to retrieve available contexts, switch context and check the current one.


LGTM.


__API Blueprint syntax error on line 184__: the use of tab(s) '\t' in source data isn't currently supported, please contact makers (Autogenerated message from apiary.io)


OpemMVS => OpenMVS ;-)


Yes, you are correct. I checked in a fix. Thanks for catching this.

> Am 31.05.2015 um 21:07 schrieb Leho Kraav <notifications@github.com>:
>
> there seem to be some fonclifct markers left in here.
>
> —
> Reply to this email directly or view it on GitHub.
>

<green/>bytes GmbH
Hafenweg 16, 48155 Münster, Germany
Phone: +49 251 2807760. Amtsgericht Münster: HRB5782


Addressed in https://github.com/brawer/fonttools/commit/6841bdb9572022cc638338d4ec29cbcbfb541049


Renamed the functions. I hope that matches what you want


I would leave host empty when default


It can still contain items, no?


cool. thanks. let me know when u r done


Thanks guys :D


Directed Acyclic Graph


You are actually working around *msysGit* bug, by removing useful flag for non-flawed Git versions, see:

https://www.google.com/search?q=git+config+core.fileMode+false+msysgit

Just do `git config core.fileMode false` or `git config --global core.fileMode false`, which actually should be default for msysGit, but it isn't. Even they have issue open for that.

Moreover I can't reproduce this problem on my Windows VM machine, so it was likely fixed in latest version.

Quoting `git config --help`:

    core.fileMode
        If false, the executable bit differences between the index and the
        working copy are ignored; useful on broken filesystems like FAT.
        See git-update-index(1). True by default.


Not related with the code:
I am a bit worried that I am able to see this user and password form this machine logged out from my account.

I would suggest to put this information in an external file not tracked by git (or make a private repo)


It also relies on the wav_proc.sh script being run first.


cool


semantical


duplicate space


Ah, even better. Cheers!


Just a small English [kink](http://www.wordreference.com/enfr/kink): *log something* ***to*** *the console*.


 FOTA: Create /cache/recovery in init.rc for FOTA

* FOTA depends on the existence of /cache/recovery and will fail
to reboot otherwise

Change-Id: I0d4d2150d54f36a0927aee3b66e64f9b81672d86


Well that is annoying as fuck. :(


We must remove these things from our .pri.
Since we have no source files directly in src, I think it's safe here. I added that it was possible to use "beautiful" paths of core includes (for example, ```#include "core/bittorrent/session.h"```). Otherwise we have to use relative paths like ```"../../core/bittorrent/session.h"```.
P.S. I interpret the Core part as some "library" which is used by other application parts.


abbstmpf不同的调用输出是不一样的。。。而且每次调用会创建一个文件。。。所以必须是$TMPF而不是重新调用


1. CSources is for building 'git devel', it's not really versionized, we frequently *erase* the history for it to keep it small. So referencing some old commit to the CSources is a horrible idea.
2. The proper releases ship with the C code that can be used to build them.


1: M100 needs more reviewers
1: M101 signature is missing
1: M110 missing component name


24:1: I100 Imports statements are in the wrong order. from invenio.modules.search.api should be before from six
43:32: E711 comparison to None should be 'if cond is not None:'


37: D103 Missing docstring in public function
47:13: E121 continuation line under-indented for hanging indent


thanks!


52:1: I101 Imported names are in the wrong order. Should be current_app, jsonify, request
52:1: I100 Imports statements are in the wrong order. from flask should be before import requests
53:1: I201 Missing newline before sections or imports.
54:1: I201 Missing newline before sections or imports.
56:1: I100 Imports statements are in the wrong order. from invenio.utils.xmlhelpers should be before from lxml.etree