Если вызвать этот метод до `compare_sequence()`, то будет обращение за границы массива к `compare_matrix`. Как минимум стоит поставить assert'ы на то что `compare_matrix` ожидаемого размера. Возможно стоит перенести инициализацию размера `compare_matrix` в конструктор, но это не спасёт от некорректности этого метода до завершения `compare_sequence()`. Возможно стоит убрать оба метода в `private` и сделать одну `public` функцию, которая возвращает общую подпоследовательность, а внутри себя нужным образом вызывает эти две. Или сделать только `compare_sequence()` приватным и вызвать его прямо из конструктора. Но мне вообще не нравится идея городить класс где можно сделать простую функцию `string find_lcs(const string& a, const string& b);`.


obfuscated email


could use a little whitespace in this line
```
self.submit(lock_metric_type.get(lock_stat), 'lock.Global.%s.%s' % (lock_stat, lock_type.get(k)), v)
```


Лишний отступ.


This is awesome. I like the idea!


Never mind I will have a look at the code later and see if we can add a custom converter


@maxiloc I have tested this improvement and it's working fine. Thanks.


@corbt The PR #3238 had to be split into two commits because of a technical detail of how the sync with the internal fb codebase works: Network.md lives on github only, all the other files also live in the fb codebase. Currently we have to commit these two types of changes separately.


`k_row`? множественное число – s – лишнее?


should be private


This commit appears to break GIS visualization, please see https://github.com/phpmyadmin/phpmyadmin/issues/11546. Could you take a look?


Use `flatMap` here instead of `optFilter` and `map`.


`if` не нужен, ведь в `while` то же условие и проверяет он его до запуска тела.


another comment


This would be the equivalent of start_ts being None because we know for sure that "timestamp" entity cannot be negative. Though, mongo does not know this so it would try to use index in case we supply "timestamp >= 0". I'm sure this will not generate any profit for us. So I consider checking for zero value unnecessary.


Imho the namespaces and file structure should be somewhat similar (e.g. `/consensus` -> `Consensus::`), so +1. In the long run it would be nice to have consistency here.

In this particular case the `using namespace` isn't necessary at all though, and could safely be removed.


Invert this as a `guard` — the fast path of missing either side of the key sets can early return.


Should arrays start at 0 or 1 ? ;-)


hey foo


С `numeric_limits<size_t>::max()` не взлетело?


Framing should be handled with CSS, not `<hr>`. Preferably the same way as with `dita2html:get-default-fig-class` mode. Same change should be made to `<pre>`


this can be defined once at a broader scope


mistakenly removed. added again with https://github.com/lahiru823/carbon-appmgt/commit/d087077f1ed639c19243ea28b542aa58ded13e35


mistakenly removed when fixing conflicts. added again with commit# https://github.com/lahiru823/carbon-appmgt/commit/62efecee604def961a8eb0271ec7ce5b744a1535


mistakenly removed when fixing conflicts. added again with commit# https://github.com/lahiru823/carbon-appmgt/commit/d4a283b11fd99c3431a950d2f0adf833ed8285cb


i'd rather not have spearmint need yet another configuration parameter..the location of the lockfiles. this works.

(btw, i've used pylockfile for my (other) project. it's nice.)


mistakenly removed when merging. added again with commit# https://github.com/lahiru823/carbon-appmgt/commit/b8820d130b0aea81c0f99ff5e81d723b4466ec3a


Please check file: file instanceof PerlFile.
Otherwise somebody can receive https://github.com/ignatov/intellij-erlang/issues/683
Thanks.


Mejora issue https://github.com/InspectorIncognito/androidApp/issues/28


I'm trying to use QMutex when NextLine() is calculated. It should help, but I'm not sure.


why is this suddenly necessary?


```lock_metric_type.get(lock_stat)``` can just be ```lock_metric_type[lock_stat]``` since you've already tested for ```lock_stat in lock_metric_type```


Schöne Idee. Ich hoffe die Umsetzung gelingt gut! Viel Erfolg.


cool!


It would be nice to add some echo statements before/after the sudos to explain that we're starting/finishing tests that require root

echo "Running tests that require root ..."
sudo su - root -c "source /home/jenkins/.gvm/scripts/gvm; gvm use go1.4.2; cd $PWD/volume; GOPATH=$GOPATH godep go test -tags=\"${BUILD_TAGS}\" ./..."
sudo su - root -c "source /home/jenkins/.gvm/scripts/gvm; gvm use go1.4.2; cd $PWD/web; GOPATH=$GOPATH godep go test -tags=\"${BUILD_TAGS}\" ./..."echo "Finished tests that require root"


You haven't tested the `except` flow.


great use of sharex!


what was changed here?


@Ulduar Последний релиз оказался битым. MD5 не зря выкладывал, не совпал он. На выходных поищу последнюю сборку. Сейчас в другом городе. Предпоследний релиз здесь, на гитхабе, выложен.


We have had a chance to talk about this in person and I am particularly pumped about this. My background in education and data science would be of great help I think :)


ok I tested but still no luck, only google logo


Are you already accepting client certificates?
or this is just importing the module?


Благодарю вас! Я пробовал скомпилить - не получилось.


remember that in hist, you can change optional parameters like number of bins and such!


set api key for themoviedb.org in string resource file app\src\main\res\values\strings.xml


nice


Se cambia el backend biber por bibtex ya que el primero viene de MIKTEX que no está disponible para Ubuntu, por lo que no se puede utilizar el formato APA 6, se pretende prontamente compilar el latex en windows para así poder usar este formato.


Echt schöne kreative idee. Macht bestimm richtig Spaß wenns fertig ist. Viel Spaß bei der Gestaltung.


Also added why-do-we-use-the-web.html and stylesheet.


Shit but THX


I have some problems with this one.I don't know if it's written well but for a spiral with a side length of 3 or 5 or 7 it worked.The problem is that the side length becomes really large and the ratio is almost stuck at 11%.


Also, since I commented on the commit, Github doesn't give good context. :disappointed:


as for the local scheduler this is something to think about generally for the program to be strictly correct:

given a (os) pid, you would not know, certainly, if it's associated with spearmint. unlike the other schedulers, the local scheduler is 1. not persistent 2. does not keep track of every process that it launches with a unique id.


Note - the list of extensions shouldn't change from call to call to call. This will introduce a severe performance degredation doing all that work every call.


I have been removing those line by lines things as much as I can... it's impossible to keep them updated... the user can just read the code...

we should remove these IMO.


should we make this part of .env as well?


you can also just do `killings.isnull().sum()`


cool


great!


Another thing I have been doing as much as I can.. is to remove JNDI from examples...

This can be simply cf = new ActiveMQConnectionFactory();

Can you do that to this example?


here we have changes for nexus 7 https://android.googlesource.com/device/asus/flo/+log/marshmallow-release


The '\n' at the end of the file was removed for some reason.


figured out how to use my actual account on the school computers, required a bit of config changing.


I think that is more clear.  I'll make the change.


these get passed when you build `ember build --prod` and defaults to `development`


Certificate auth is not yet supported. This is just importing the module.


@hulpke Hmm, I am worried about this change. For me, a generalized mapping is a possible non-total relation in R x S, and its generalized inverse is a relation in S x R. This change breaks that assumption.

Granted, the documentation of `InverseGeneralMapping` does not promise that, but I wonder if it maybe should, and instead of this change, we instead should fix any code that relies on a general mapping being total?


included badAuth param to test bad authorities


This needs to go.


@sarahmcalear Thanks for going over this. I'll have to do alot of cleaning up and organization.
I'll have some questions tonight for you, on how to read these files on Github.


My bad: https://github.com/yandex/mastermind/commit/3c242c48697903d289e34451a3b0c256a1d954a1


I merged the ones I found important as u can see in my hostory. Flo specific ones I haven been deleting.


@paramaggarwal Hmm... I see. But you'll only face issues when you update the app right? May be have some versioning system for the bundles and avoid downloading a bundle if app version has changed?


This should be reinstated.


added statementID in body for cors post/put


yes, correct, you have spotted very well that you need to remove the multiples of both 3 and 5, however this can be done a lot more elegantly. Check [this post](http://math.stackexchange.com/questions/9259/find-the-sum-of-all-the-multiples-of-3-or-5-below-1000)


If you want to remove it, you'll need to annotate FindBugs as well. Not having it originally triggered a FindBugs failure.


Not true; reset the local engine.


Great !! thank it works like a charm .
But there is one more issue  when the keybord is displayed to start typing , it covers all the recent message so u still have to scroll to the bottom to see the last message  . is there a way to make the keyboard to be displayed below the chat instead of over the chat . also preferably make the keyboard stay on after u send the message to replay easy to next message


This should check definedness not truthfulness.


I used the button group only to keep them together when resize.
I changed grid sizes to hold the left panel of the designer for lower resolution.


everything else looks good!


@web2solutions, did you mean to check in the `node_modules` sub-directory?


Why this change? It doesn't conform to the JS Mixpanel SDK


damn .. wrong time again ... should not have refurbished a used time machine ...


Sehr interessante Idee, vor allem die Multiplayer - Idee klingt sehr vielvesprechend, wenn umgesetzt.
Viel Glück :)


If you make a deep reference to a Vega source file, I believe browserify will package Vega up as part of your project. In that case, you'll want to follow [these instructions](https://github.com/vega/vega/wiki/Vega-and-Browserify) to resolve the errors you're experiencing.


All in all, great work. I made a few coding comments on where you can make your code a bit more efficient :)


This is traditionally done like this:
```
let locals: any = $scope.resolveData || {};
```


Okay, so you want to see

    if (limit != NULL && session_id + len > limit) {

and just let the control flow continue through tls1_process_ticket() as usual, relying on tls1_process_ticket()'s check on the SSL version?  I'll go ahead and generate that patch.


@Ulduar если захотите попробовать собрать снова, напишите мне на имейл, указанный на гитхабе, я вышлю вам ссылки на плагины и инструкции по установке.


This was a pull from the clean repository state, before the massive explosion. I guess this contains the fast forward up to what happened after the explosion.


I actually thought about it and removed my comment because passing NO_PROXY will ignore jvm settings as well. This will introduce a breaking change. Lets not do this. Sorry for the back and forth.


something must be messed up because noone portet it yet so far:D


This needs to go away, correct?


@sipa consensus/chain.h gets used in a later commit. I suppose it could get added then, though it doesn't really hurt to have it here anyway.


comment comment


Okay, I wasn't sure if we preferred to not do it that way for clarity, or if that's becoming the JS/TS equivalent of ??


Or:
```
options = options || {};
```


Next commit adds `.DisableEngine` (and `.EnableEngine`), so I think we're okay.


I think the reddit idea is awesome, I've worked with that data before so I can definitely help there. It is a great dataset for clustering and natural language processing. To see which subreddits are negative in general, positive, etc..

As for the gaming, does Steam have an API to pull gaming data, or maybe Twitch?

I think you are on the right track for project ideas. I would try to narrow it down to a single idea within a week so that you can start working on it full force!


Good catch!  I just sent a PR to fix it.


Coole Idee, und sollte auch machbar sein. Sowas in der Richtung hatte ich ja auch überlegt.


I think the above is pretty standard in JS and will be commonly understood by most developers.


@sipa Agreed about Consensus::SoftForks
@dexX7 The namespace is not used in this commit...but it does get used in later commits. As with the consensus/params.h include, probably doesn't hurt to go ahead and add that line here anyhow.


Плагины в readme указаны! Я когда пытался вчера собрать, прочёл.


I mean including consensus/params.h instead of params.h. Consensus code
shouldn't depend on the non-consensus parameters.


Could get pretty messy, if more contributors start to add similar extra notes..


It's like the best ever!
Let it be for a while, this will be the part of orm library and will vanish from here.


With this change, is the expensive resource ever cleaned up? If so, where?


Beautiful reorganization!


Tamagotchiiii! Mehr ist nicht hinzuzufügen.


Maybe we need to update some blobs (adreno) but it shouldn't hang at Google logoz let's wait for mako guys.


So child controllers have to individually resolve things that they want injected from angular? I'm a little confused by this...


```
extension Prefs {
func setEngineConfiguration(e: EngineConfiguration, forKey: String) {
…
}}
```
?


comment comment comment


Agree.


I think I'm going to rename the field from `MPBalanceEntry` to `BalanceEntry` and deprecate (and later remove) the address field. It's probably better to call it BalanceEntry than Balance since it has two numeric fields and is not a simple balance.


Es incorrecto:
```
self.scheduled_to_date = (
             self.qty_available + self.move_incoming_to_date +
             self.qty_available + self.move_incoming_to_date +
             self.procurement_incoming_to_date + self.incoming_in_po -
             self.procurement_incoming_to_date + self.incoming_in_po -
             self.outgoing_to_date + self.incoming_in_mo -
             self.outgoing_to_date - self.outgoing_to_date_reserve_destination +
             self.procurement_plan_incoming_to_date)
             self.incoming_in_mo -
             self.procurement_plan_incoming_to_date_levelgreater0)
```
Debería ser:
Quantity On Hand
+ Incoming up to date from procurements
+ Incoming up to date from procurements level > 0
+ Incoming in PO
+ Incoming in MO
+ Incoming up to date from moves
- Outgoing to date
- Outgoing to date reserve destination

```
self.scheduled_to_date = ( self.qty_available +

             self.procurement_incoming_to_date O self.procurement_plan_incoming_to_date (Uno de los dos, el que se haya dejado. El otro ya NO EXISTE) +
             self.procurement_plan_incoming_to_date_levelgreater0 ¿ES ESTE?
             self.incoming_in_po +
             self.incoming_in_mo +
             self.move_incoming_to_date -
             self.outgoing_to_date -
             self.outgoing_to_date_reserve_destination)
``'


I think you need to gitignore IDE files (.idea).
They has polluted this commit very much.


Methods chaining is a common ORM way of applying several filters and conditions to the query. Take a look at sqlalchemy docs: e.g. http://docs.sqlalchemy.org/en/rel_0_8/orm/query.html#sqlalchemy.orm.query.Query.having


mako guys have same problem


if it implemented it to my code, the inline editing isn't working (after when i removed it, it worked fine)


Fair enough...I'll restructure.


I think it's because boringssl


Available here: https://www.nuget.org/packages/canopy/0.9.35

Also updated to the latest Selenium that came out today.

Let me know if you have any issues!


Removed  few of the tests, didn't think there was a way to test the internal workings of an LRS


ty Aya


Enabling and disabling imply resetting, no?


On "list" naming: I will also rename in in orm library. As for now, this is already used in other ORM classes in mastermind, so it should be a part of overall refactoring or migrating to that great ORM library that I've been working on. Let's wait a little bit till it's implemented.


Looks like copypasta got the best of you here. Also in the following two tests.


You didn't just "slightly modify" my commit. :) But I hope it helped.


`meow` does this for you.


hey now


@dadi11 naah.. I don't think so:)


Great.Thank you.


Added a docstring to the "list" method: https://github.com/yandex/mastermind/commit/c06bdd2026739de6ff66eee16cdc343c4f6ba131


Only apparently does it get stuck at 11% !
I ran your program and changed it a little bit so I can see the progress (basically, I made made `ratio` into a float variable)
After ~15-20 minutes, your program finished and the answer was indeed correct. The side length is `26.241`.
Of course, you may want to optimize your code but it is nevertheless, correct.


hey foo


I considered making this so, but keeping this fine-grained allows the consumer to easily choose what to do.


Whoops! My bad. Should be Mojo::UserAgent::Mockable::Serializer. I'll correct it.


Looks good - thanks for catching that!


You are selecting all the posts from thread in show_page, consider using LIMIT. Also try getting rid of DateTime module, it may improve load time by around 105ms.


@psychonic Did the gamedata file get put on the gamedata updater?


Woops, @danny-andrews - care to make a PR? I'm swamped at the moment.


good point!


So, beim suchen des Fehlers, wieso das Modul nicht mehr angesprochen wird habe ich herausgefunden dass bereits ein Mechanismus in FHEM integriert ist um doppelte Meldungen zu filtern.

Es wird über die sub FingerprintFn realisiert. Denn diese wird von dispatch aus aufgerufen und soll prüfen ob die Meldung angenommen / verworfen wird.

Das könnte dann das Attribut und die Implementierung von minsecs komplett ersetzen oder?


comment on line 57 deletion


Shit, hope mako guys can sort it, lolz :D


we've got same problem on g3 with adreno libs which aren't working because boringssl changed too much in how they work


idk. use the dev2 branch for the latest fixes. Its going to be release soon (I hope).
I wont fix anything on this branch anymore. But thx for reporting.


git pull


done!


Quick fix after the previous commit had index.js in the wrong folder.


Números de Bernoulli obtenidos de https://oeis.org/A027641 (numeradores) y https://oeis.org/A027642 (denominadores)


Misspelling of "mock".


For consistency, have `tm-save-tiddler` handle all of this?


This might not work like you intended... :disappointed:

|| is a boolean operator. So the first expression would resolve to false if useDate is null, undefined OR false. So `var hasDate` will always be true here... :neutral_face:


@dadi11, probably we can kang from flo (n7 2013). Need to ask them guys :)


so we should try blobs from nexus 7 2013


Note it was not possible to get here (LOCALE_CUSTOM_DEFAULT) previously as Linux doesn't support custom locales like Windows


Consider using a mocking framework like "Moq" (which we use in PUMP) for mocks that are both simple and uniquely used like this one.


Are you sure the container should be named suffixed with '-agent'?


WTD did you do here? Why did you remove those methods?


@paramaggarwal we need to figure out a good solution for backward compatibility. Right now we're not ready yet to put this constraint upon us.


@tobibeer the trouble is that then the `tm-save-tiddler` handling in the navigator widget would need to be aware of the special tiddlers `$:/temp/NewTagName` and `$:/temp/newfieldname`. That's an unacceptable breach of the layering of the system.


2. & 3. Done
4.  Changed to:
 On Oct 4, 2015, at 8:29 PM, Brady Miller <notifications@github.com> wrote:
>
> interface/forms/eye_mag/php/eye_mag_functions.php review:
> 1. Agree to use the "openEMR global user preference for date formatting" in priors_select() function.
> 2. line 84, wrap text() around $priors[$i]['exam_date']
> 3. line 94, do not translate the actual date, and instead wrap it with text()
> 4. line 115 will not be possible to translate. Note the translation constants are placed in a spreadsheet and translated, so this will never be translated. Probably better to do something like(note ideally have the constants all in one line, but can't figure out how to do that; sometimes it take a bit of creativity and compromise :) ):
>
>  To see, what a translator sees, check out the following:
> https://docs.google.com/spreadsheets/d/1i2_WsjBX9cwa9mx0gIv3psMzQ28VsUZ-MqlAyZcmbX0/edit?hl=en <https://docs.google.com/spreadsheets/d/1i2_WsjBX9cwa9mx0gIv3psMzQ28VsUZ-MqlAyZcmbX0/edit?hl=en>
> 5. What are example of the $zone parameter(trying to figure out if should be translated above and in the titles).
> 6. Noted extract in line 189 and line 194. Not an awful thing, especially if populating hundreds of variables it actually makes sense; but noted it.
> 7. line 551, what is example of what’s in the $output variable (seeing if need to html escape it)
>
$output is a whole section of the exam from  prior visit and is handled in the priors_select function.
> 8. For translated constants like lines 613-616, 624, 631, consider comments in the the constant (via the {{ and }}) to clarify them for translators.(for example ‘L{{left}}' )
>
OK
> 9. line 926 and similar lines below, does $PRIOR_MOTILITY_RS_? need to be html escaped?
>
They represent a fontAwesome icon, defined as a $hash_tag above in the same script and do not need to be escaped (lines 832-890).  Right now they are either horizontal or vertical (“|” or “-“) but I also need to do one at 45 degrees.  Some pediatric ophthalmologist or a neuro-ophthalmologist doing strabismus use these obliques - depends where they trained.

> 10. Don't place include/require_once() stuff in functions. Can end up with horrid out of scope issues that suck up debugging time. Instead place all of these at the top of the library outside of the functions.
>
OK.  Seems a waste though since only a very small portion of the code ever needs to reference these files/includes.  I moved them to the top of the file in eye_mag_functions.php.
> 11. line 1348 translate POH
>
ok
> 12. line 1351 translate New
>
Yup
> 13. lines 1361,1362 html escape (via attr() and text() )
> 14. line 1367 is not within html attribute, so xlt() is better
> 15. Repeated blocks of code with above issues in 13 and 14 until line 1596
> 16. Will need some thought at later date on how to integrate the Issue Types and the issues screens code. Also need to look at the SH stuff and FH stuff closely. A FUTURE TODO.
> 17. line 1654, translate 'Missing Title'
> 18. Line 1662, don’t translate the code or code description.
>
OK. 1662=1685 now.
> 19. Some missed translation and html escaping from line 1918 to 2165.
> 20. There is an issue with how you are collecting document in display_draw_section() function. TODO will come back to this when I have a better idea of how the code is storing docs etc.
> 21. HTML escaping in the display() function.
> 22. Translation and HTML escaping in the menu_overhaul_top(), menu_overhaul_left(), function.
> 23. Where is the row_deleter function used? That mechanism may be impossible to prevent sql-injection on.
>
it is not.  Deleted.
> 24. In Menu_myGetRegistered() function use sql binding and use the escape_limit() function around the $limit and $offset variables.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/bradymiller/openemr/commit/87c95fa272aa1b4169a78009b727c1fc777d662b#commitcomment-13583273>.
>


Sure, coming right up!


Looking at the code and usage, this is not a change I think I should make. allPartitions is of type Pool[(String, Int), Partition] and in this case the Partition class is a logical abstraction over both the partitions and replicas. I could chance the code to allPartitions.values.flatMap(_.getReplica).size but that will actually result in the same value.


In that case, we can find the upper bound of binary search, that is, find a element larger than target without reaching the end of array, for example, 1,2,3,4,5,6,7,8,9,10,,.........10,000,000,-1,-1,-1,-1,.....-1, and target is 5, we only need 3 iterations to get 8 in order to do the binary search, while without that condition we need to do more. My logic for the   terminal condition in this statement is to find a value larger than the target, not necessarily reach the end of array.


Thanks for naming all these constants. The quantity, however, is telling. Pretend I'm repeating my earlier feedback about this class having way too many responsibilities.


hmm. It was throwing errors on my local build. I'll take another look.


@imcgames Can you change this to "One-handed Spear Sub", please? I forgot to add "Spear". Thank you.


REGISTER_SERVER_PACKET_NAME ? ``` CMSG_ ```


Please adhere to the indentation of the file. Its always 4 spaces.


Makes sense - thanks @vjeux.


Note that a lot of these can be lightly refactored to become integration tests (or at the very least, sociable unit tests), if you extract functionality from the Game class. This is the power of red-green-refactor.


Didn't see this. Does this fix https://github.com/facebook/react-native/issues/3265 ?


Ah, ok. It does depend on chain.h (for CBlockIndex) so I add that correction.


@imcgames Can you change this to "Equip One-handed Spear Sub", please? I forgot to add "Spear". Thanks.


In that case could I pass another parameter in the function : the length of valid elements in array a?


This commit it fine.


@Oscar-Rodrigues here


Do they need to be final if you have a `sealed trait`?


With the current slider, I felt that keeping e.g. "Years" plural made more sense ("Years: 0 to 1"). However, since Dan's mockup shows period UI similarly to step size UI, this change makes sense, so let's keep it.


Should we make a common class for itc and seqexec?


Two backslashes missing, should be
```
\\\"p1\\\"
```


Fixed in current version, not yet released. Thanks anyway :)


yep, when it doesn't find the ticket count, it fails and does nothing, as if by design ;)
I have an improved code, in case you are curious or ever need it: http://codepen.io/anapsix/pen/epWwyr


How does this not trigger a FindBugs issue now? Did you annotate it somewhere?


Where does the actual date come from?


What is it used for?


I did think about that one but it seemed to match the error string.


I update the user account properties. Sorry I hit the commit button to early (pressed enter :/)


why is require foundation/foundation moved to line no 9. this is re-introducing bug fixed by https://github.com/zurb/foundation-rails/commit/2e380a3578f


This looks extremely confusing to me, `http_client_for` looks like it should already be returning an HTTPClient. Perhaps `Web#http_client_for` should return a `Gemstash::HTTPClient` directly, and `Gemstash::HTTPClient` could construct it's own Faraday client, optionally passed in for use in tests?

Then this would become `@gem_fetcher ||= Gemstash::GemFetcher.new(http_client_for(upstream.to_s))`


Ok. So this is not exactly solved. The cache entries may not always be there, say for instance a the application restores state where the view is scrolled far down or if NSCache removes entries because of memory pressure.


Yeah... that wouldn't be good. :smirk:


Am I reading it right that this commit removed EMS-Keys.plist?


Yup, so long as we remember, we're all good! :)


Typo


Fixed, https://github.com/owncloud/documentation/commit/076d67c0077fedbdec5a11eb0b655a1cf4565a2c


Typo !


TYPO !


One small thing here: can you remove the print statement you added? (I assume for debugging reasons)


Console errors:

<img width="472" alt="screen shot 2015-10-07 at 2 23 58 pm" src="https://cloud.githubusercontent.com/assets/338642/10346830/19a5bd8a-6cff-11e5-9537-bea6b0b0f3ac.png">


Excellent.


Some people wanted it. @_@


Hope it's more clear this time :D


Magnifique.


Not sure what these were supposed to mean..LRS will not reject get requests b/c of this


Intéressant.


Hey @posita .. I'm sorry, I'm not following which mention I said and you assumed it ... But I think no .. there is no reason to check `node_modules` sub-directory


Bien pensé.


Bravo.


This change breaks Hotn. Maybe keep the old function as an overloaded alternative the next time you do a similar change.


Super.


no, it is not being cleaned up


Thanks a lot!


It fixes part of it, the other part is related to https://github.com/facebook/react-native/issues/2852


It looks like you're inspecting the grandchildren here... Is that right?

Oh, there's some other nesting in between e.g. the SequenceType and the TaggedType, is there?

I think it would make more sense to start from `self.components` here and work your way downwards, something like:

    component_types = [c.type_decl for c in self.components]
    already_tagged = any(isinstance(c, TaggedType) for c in component_types)

I could be missing a level here, but I think that's roughly it.


what do you mean by hang?

As I've seen with trying to go through the various timeouts in Grid... these need to be very clearly defined. Is this the timeout from sending the command before receiving a response?


Use `enumerate` to generate tag numbers:

    for tag_number, type_decl in enumerate(component_types):
        ...


Yes, it was just to quickly delete the binaries x)


Oops, I do that all the time as it is impossible to manage commits of the project file. What I do since the EMS-Keys.plist is not part of the is to delete that entry in the copy-resources phase. I could probably do it in some other way, but thats just the way I´ve been doing it. In addition I always have to change the certificate stuff, since we are not part of the same team. So you always have to fix those two things before release I assume. Would be cool if we could find a better solution for both of those problems.

For problem 1: Include a dummy Ems-Keys.plist for test builds.
For problem 2: Get a JavaBin developer account and transfer the ownership of the app to Javelin.

Not sure how you feel about that?

> 7. okt. 2015 kl. 20.22 skrev Chris Searle <notifications@github.com>:
>
> Am I reading it right that this commit removed EMS-Keys.plist?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/javaBin/ems-ios/commit/a0e5b0ad0c9be2376dbbe74e9a708ee0394cee30#commitcomment-13645645>.
>

Med vennlig hilsen
Ronnie Nessa


102867486-FixRectangleShapeTest-5173 (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1901 of 1910 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.53%25-yellow.svg)


you can simplify this by writing
`return number == reverse;`


Looks good


So much seal. Hard to handle such amount of seal. Perhaps think to consider less seal? If not, that is good also.


again something to add much more clarity to. client idle is the time between selenium commands received from the client (automation script)?


@bberenz another item I forgot to mention is the suppression of the `Client connected` and `Client disconnected` messages.  Can we make that a configured parameter?


I wanted to follow the patches homebrew applied here https://github.com/Homebrew/homebrew/commit/7d66582

specifically: https://gist.githubusercontent.com/UniqMartin/baf089e326f572150971/raw/1de52d53929bc3472cc7f345c16f068c37c75263/qtbug-47641.patch

I believe the other changes in the upstream QT commit were just macro updates. This only applies the runtime error fix


extracted qcom blobs for n7 2013 https://hubdroid.com/~DevSwift/vendor


Yes.


Let's include the reversed operators (`public func * (lhs: Double, rhs: [Double])`) and also division by a scalar


Great


Let's make this a separate PR and include `-`


registerSnap -> addSnap


This method exactly duplicates `WebHelper#get`... either this class should extend `WebHelper` (or vice versa), or `WebHelper` should not serve that purpose anymore.


then put that in the commit message


can we add a comment about what is a snapshotBuffer and what is its responsibility?


Can you send me a pull request to https://github.com/lollipop-og/proprietary_vendor_lge_gee
with updated adreno blobs?


can we add a comment about snapshotProcessor and what is its responsibility?


Done, I created #15 and assigned to myself. I'll give a look into this tonight or tomorrow.


whoops not meant to be here


requestSnap -> getSnapFrom. Also can we add a comment to this method?


I guess this means we don't need the suppression below.
I'm away from fancy tools right now. How does the empty `a` render?


Also need to remove the trailing =
```
| sed 's/=//g'
```


gauge.globalLock.currentQueue.total
gauge.globalLock.currentQueue.readers
gauge.globalLock.currentQueue.writers

Did you mean something like this ?
lock_type = {'R': 'read', 'W': 'write', 'r': 'intentShared', 'w': 'intentExclusive',
                     'total':'total', 'readers':'readers','writers':'writers'}
lock_metric_type = {'deadlockCount':'counter', 'acquireCount':'counter',
                            'timeAcquiringMicros':'gauge', 'acquireWaitCount':'gauge',
                            'timeLockedMicros':'counter', 'currentQueue':'gauge',
                            'activeClients':'gauge'}
Have the above dict to use for every lock metrics?


That being said - if you think it would be more consistent to apply the whole upstream QT patch, I can do that.


Haha I just caught this same thing.  It's always the little stuff...


I just made your test to pass .. and that is why I said I don't trust this code :+1:

Shall the `NAMESPACE._all_ids[_id]` to be cleaned on `NAMESPACE.resource` level all the times we close it?

For example, supposing it is a database connection (as you did before), shouldn't it to be handled on  `NAMESPACE` and shared through all created `NAMESPACE.resource` ?

Do you have any requirement/proposal that makes is more clear?


Whoops -- You're right, mine isn't recursive, like x.a.b.

You're also quite right that it might not belong in Lumen.  It depends
entirely on who the audience is.  If the audience is strictly compiler
writers, then it will get in the way more than it will help.  But if people
will be using Lumen the same way they'd e.g. use Racket, then it seems like
being able to write "reader.read-all" might be good.  The downside is, it
makes Lumen bigger, and therefore there's more that can go wrong.

I propose ignoring this for now and revisiting it later.  (Or rather, I'll
leave it up to you.)  I don't actually need it right now, and I can get by
without it.

The PR contains some other stuff that you might want to check out when you
get time, but nothing super important.  Looks like you recompiled lumen --
the only other thing was a fix to an error illustrated by running
LUMEN_HOST=luajit
bin/lumen -e "(require 'ffi)"  or LUMEN_HOST=luajit bin/lumen -e "((get
(require 'ffi) 'new) 'char)"

On Wed, Oct 7, 2015 at 8:00 AM, Scott Bell <notifications@github.com> wrote:

> I'm on the fence about having this functionality in Lumen—I'll think about
> it. If I were going to restore it, the previous implementation was more
> robust: these forms worked as expected x.0, x.a.b, x.a.0.b.2, for
> example. I would probably restore it from the git history.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/laarc/lumen/commit/994acfeec9383a199fff13fe5bb4d7ea73cf1f7f#commitcomment-13640829>
> .
>


Hi David,

Why? the value is encapsulated with quotes:
"s#@APP_REMOTE_ADDR_HEADER_SALT@#\"$APP_REMOTE_ADDR_HEADER_SALT\"#g"
so '=' will not trigger an INI parse error


I think the better solution here is store the location from where the user gets redirected to login in the redux store. Then on successful login, in `Application.js` we can redirect appropriately.

There is also the possibility of adding [redux-router](https://github.com/rackt/redux-router) (started [on this branch](https://github.com/codefordenver/fresh-food-connect/tree/redux-router)) at some point in the future which allows us more visibility and control over the application routing.


Could you just use the case class copy constructors?


Sample Input file for Supply route algorithm


One of the reasons I noticed is that it took quite a long time to download the updates to your repository. I was surprised since I only expected a couple text files. Upon further examination, I noticed this revision has over 5,000 new files. I also noticed that `.DS_Store` got checked in at some point as well. This is probably not what you want, and can be difficult to reverse (see [here](http://stackoverflow.com/questions/1904860/how-to-remove-unreferenced-blobs-from-my-git-repo), [here](http://naleid.com/blog/2012/01/17/finding-and-purging-big-files-from-git-history), and [here](https://help.github.com/articles/remove-sensitive-data/)). It also imposes significant, and unnecessary overhead on other developers who may be working on the same project by bloating the Git's [object store](https://git-scm.com/book/en/v2/Git-Internals-Git-Objects).

Besides being more careful, is there a way to discourage checking in unwanted files in future commits?


@posita ... I got you ..

I would mean you check the commit https://github.com/web2solutions/Veritaseum-proof-of-concept/commit/fa8ea9233501441f06278659c9bc94ae60169b55 for the changes which makes your test pass


> That being said - if you think it would be more consistent to apply the whole upstream QT patch, I can do that.

I just hope it's done for purpose but not because of mistake :) Will return here is there any errors occurs.


yep, already pulled


Yep ^^


Gute Idee, Textadventures brauchen Zuwachs ;) Schau dir mal meine Idee an, hab sowas ähnliches.


If we are going to do this, why didn't you just add a option to remove any room rank? IE /deroommodall


Ah, we've introduced a new behavior here (returning. Should we include tests to validate that the new behavior works as expected?


wew


Thanks for teh head up. I have addedd gcc for osx....


dcsdc


Ah, so I do.  Thanks, the problem was another set of crossed wires in my mind. :)


Don't commit as root. Never.
</rant>


I am cleaning this crap up now.  All of this will be in jumbo.h and that will be all that is needed.

I did leave the code in blf_plug.c alone, however.


You should probably factor this `len(a.cur.buf)-a.off <= 0` into a method and call it something like `isEmpty` - then you can use it below.


yeah, i moved the List justWords inside the Try {} block because when it was outside wordsFromText was not reachable.


D'oh, there's actually a bug here: simulations can't be run because on line 98 `GraphStore` doesn't exist. (When merging my branch, the new simulation tests caught that, which is nice).

I'm not sure quite why GraphStore doesn't exist (we're `require`-ing it), but I'm guessing the solution would be to keep track of the nodes from 57: `onGraphChanged`. And while we're at it should we get rid of the `require graph-store` at the top, as we don't seem to be using it?


I didn't. I deprecated the method, because it is almost never what you need.


Shoot. Dom does a typo in the commit message. :mans_shoe: :tired_face:


Wenn's so gut wie Locoroco wird ne super Idee ;)


thx, only 9 changed files though?


:grimace:
I guess we had better revisit this one more time. Integers should probably use `? :` as well. Otherwise 0 will default to the optional value.


I was confused by this line.  I think it is because you initially assign `a.cur` to be an empty buffer.  It might read better as `if a.cur.buf != nil`? Otherwise it is making me think you are leaking buffers (which you aren't of course).


I pushed only adreno blobs, you want all??


Yes, OpenSSL is being bundled as a part of libgit2, that is correct.


youre doing great work pls see me in my office for a promotion


This format is weird. Why not put these on their own lines?


nah, Last timeI updated adreno blobs it had been ike 16 chanegd files


I forgot about rest of libs:D


> One of the reasons I noticed is that it took quite a long time to download the updates to your repository. I was surprised since I only expected a couple text files. Upon further examination, I noticed this revision has over 5,000 new files.

Yes. I follow you.

You asked me to modify the documentation and provide more clear informations. Then I decided to write a 'step by step' test manual. The proposal at this point is to let the end user to be able to easily run and test it.

> I also noticed that .DS_Store got checked in at some point as well. This is probably not what you want, and can be difficult to reverse (see here, here, and here). It also imposes significant, and unnecessary overhead on other developers who may be working on the same project by bloating the Git's object store.

Yes, you are very right. Right.

> Besides being more careful, is there a way to discourage checking in unwanted files in future commits?

sure, adding it to `.gitignore`


 int min, max;
int maxVal = 0;
System.out.println("Enter a Value: ");
    int val = s.nextInt();
    min = max = val;
while (val != 0) {
      System.out.print("Enter a Value: ");
      val = s.nextInt();
      if ( val < 0){
     System.exit(0);
      }
      else{
          if (val < maxVal) {
          min = val;
      }
      if (val > max) {
         max = val;

      }

      }
    }

    System.out.println("Min: " + min);
    System.out.println("Max: " + max);
    }
}


Yes, my intent is to remove the web helper and split the responsibilities
of both building URLs and fetching resources into objects that are
dedicated to that. Just didn't do the cleanup yet.

Agree on the structure, was the idea, just didn't get there yet.

On Wed, Oct 7, 2015, 19:30 Mike Virata-Stone <notifications@github.com>
wrote:

> This method exactly duplicates WebHelper#get... either this class should
> extend WebHelper (or vice versa), or WebHelper should not serve that
> purpose anymore.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bundler/gemstash/commit/7de9dcfa1ddb857c10023f72e7a096d61e17137d#commitcomment-13645826>
> .
>


Paranoid comment: make this test >= so if you end up shoving more items on somehow the block still gets triggered. No I'm not sure how that would happen with the code as is, but paranoia.


like libCB.so and so...


I integrated it with Travis too, then we can test every build


Did you check it? I tried it and it does not work.


Seeing how much different opinions there are here, I think I'll try to turn it into a template where anyone could set it up in whatever way which suites him. It will take some thinking though, and the GSdx part might still stay for now as its own "unit". But I like this idea.


My proposed changes seem to work fine, and I'm happy just adding them to my PR, so we don't have to change this one after it's already been merged in, and so I don't have to rebase simulation-speed onto it a second time.


jkgjj


saw the new fie "Data2.R"


@posita
Did you read the new README ?


Unnecessary comments


super minor nit but you dropped this onto a new line as well :)


More unnecessary comments


I'd prefer starting new things in a "org.seleniumhq" package instead... but meh.


Хорошо я это понял сразу. Хотел узнать как ты это понимаешь. Смысл делать dao сериализуемым возможно есть,но мало вероятен. Скорее классы из core к этому больше подойдут. А до этого было установлено ограничение на тип generic в dao. Так что если не возражаешь верни назад.


This is an array. It doesn't "return" anything.


ok


Yes, I checked and it worked. On which version of Sublime did you tested?


Doesn't return anything


This is my favorite commit


I don't think this function returns the base address. I think it returns the mask for a given IO pin.


Done!


Is there only one callback for all the GPIO pins and banks?

Also we should not use compiler extensions in our code (__weak), and instead make a method for registering an interrupt.


will add missing ones


This is really the only problem I have with this update, so I might get you to clarify. Why are you returning a set of values here?

My design with this particular library was to keep a Bag's state encapsulated within the class. I would support storing these things as instance variables so that this info is available to other methods like `show_bag_info`.


food=


В целом ты прав и это не криминал. Но в женерик передаеться integer только потому что в женерик нельзя указывать примитивы. Если я правильно понял. Поэтому скорее всего можно было оставить. Но для единообразия лучше сделать так как ты сделал. Так что я за твоё решение.


I think you should probably change the function interface to take an `io.ReadCloser` to avoid these type assertions and make it more obvious that the input may be closed.


Hi Mike you seem to have lost the Ophis entry, replacing it with the new UniAsm one.


<img src="http://thelapine.ca/wp-content/uploads/Penguins.jpg">


Anjali


legal


This had me stymied for a while, but I think I see a better way to do this now --

1) Make `TaggedType`'s `__init__` accept either a token list `elements` or the individual things (type_decl, class_name, class_number, implicity). I'm not sure if there's an elegant Python way to do this, but the obvious is to make all arguments default to None and then assert that the right combinations are set. Or, create a static factory method that does the "normal" parsing of tokens into members, and make the initializer just take the values.

2) Extract the details here (I think you have all of them already) and just create TaggedType object and assign it to child.

3) Maybe even better, build a new `components` list containing the tagged types and reassign `self.components`.


how about make it toplevel private?


hadn't merged and committed fully. fixes #233


+1. I have faced the same problem.


Could you separate this into go imports and external imports. (see other files in the distribution project)


how do we edit in here or add things


Done!
https://github.com/apmuthu/opendocman/commit/97963492a6baab3a4e0c4fbcf09595a8d21c38c9#diff-5b847b1d73abef135cc6af9d9c8be15b


If you do the above, you'll replace this by some kind of "overloaded constructor" for TaggedType


This function has just enough duplicated code in it to make me think it could be re-organised!


Do we need the build badge of the reveal.js framework here?


yep I added it
https://github.com/ProjectG/proprietary_vendor_lge_gee
you can pull it


Done!
https://github.com/apmuthu/opendocman/commit/97963492a6baab3a4e0c4fbcf09595a8d21c38c9


Find ich gut. Gibt zu wenige solcher Spiele.


Great, I was worried about this case, but I guess your use of descendants digs all the way through to these, right?


That's fair, I can't say I'm actually using it. It just seemed like useful information to have as it will be gone after the update step. Storing it internally seems a like the wrong solution as it's very temporary data, if someone writes a file to the directory it will be out of date already.

Either way, I'm not using it anyhow so I have no problem with simply dropping the feature :)


that should be in your global gitignore :dancer:


I'm removing the `node_modules` from the build


This hadn't even occurred to me, what happens here?


Ok cool. In the future should I do something differently when a PR touches
multiple places, or is this just something someone internal to FB has to
take care of?

On Wed, Oct 7, 2015, 6:35 PM Martin Konicek <notifications@github.com>
wrote:

> @corbt <https://github.com/corbt> The PR #3238
> <https://github.com/facebook/react-native/pull/3238> had to be split into
> two commits because of a technical detail of how the sync with the internal
> fb codebase works: Network.md lives on github only, all the other files
> also live in the fb codebase. Currently we have to commit these two types
> of changes separately.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/facebook/react-native/commit/2d7747ecb7a21caccd30c15cd254cb2325e94b3c#commitcomment-13644578>
> .
>


Visual Studio recognises this format as a hyperlink and thus the suppression isn't required.  Tools such as Sandcastle will correctly render this as a hyperlink when creating MSDN style documentation.


Thanks Ed, fixed in f0a43a63387a645b96cef8eaff61151ff4362032.


I'm not sure it's "useless" info -- the current validation state is also stored in the class, and that's "temporary" info.


https://github.com/lollipop-og/proprietary_vendor_lge_gee/commit/21eebbcc79384be7583347118aaaf5a979b70d1d
lolz


> @posita
> Did you read the new README ?

I did. I am able to run tests locally. Thank you.


I just fixed it myself locally and then found this commit. :) Do you plan to make a PR for this? If so I can review it.


I conjecture using `io.ReadFull` here would increase throughput on readers which return lots of small chunks (perhaps the networking code - who knows!)


Looks good to me. :+1:


If this status is only "enabled" or "disabled", consider renaming it to stm32_gpio_clock_enabled


so much garbage from DW


Find ich beides cool. Auch wenn die Sculpture-Idee ein bisschen cooler ist. Außerdem soll es glaube ich eine Anwendung werden, die auch lokal ohne Netzwerkverbindung Sinn macht.


@posita there is a new build available without the `node_modules` sub-directory


:D


rebuilding soon :dancer:


Are there really only 16 pins per port?


me too:D 45 minutes to get build done


Thanks for the corrections!


cddc004

This cleans up the crap, and no longer do we have to NOT force single include in jumbo.h for MSC builds.


Zur Not würde ich auch mein Hackeradventure aufgeben, um an diesem Kuhdrama zu arbeiten :D :+1:


u use my tree or u did other changes?


Looks like you're looking at an old version of the code. The latest release was 6.11.


nope, I took only build repo from cm


New tests should `use strict; use warnings; use Test::More;`


and my mm release branches? device, kernel or lp ones?
Here is my build but takes about 70mins: http://jenkins.olivierk.eu/job/Marshmallow%20(extrem%20wip%20and%20faulty)/10/console


Indeed it was an oversight to include that.


I actually think the biggest 'problem' was what was in misc.h  Defining inline to static would cause this to happen.

```C
static inline test() {foobar();}

// turns into

static static test() {foobar();}
```

I was getting a double decorated something error.  Well, static static  could probably do that, lol


device modified from 5.1.1 with sepolicy deleted, kernel_msm, and vendor repo from you


Where are the definitions for the various functions the pin can do?

My understanding was that pin_data is a pointer to a stm32 specific struct that defines how the pin should be configured.


Sure


Reading the input file is a separate responsibility from the logic of a `Grid`. What if the input format changes? What if you want to run this without any user input (e.g. a random initial state)?


I keep forgetting to make one ....

You need to show me next time


kernel_msm, not sure if this is good to go, I updated bricked with flo commits:https://github.com/lollipop-og/bricked-geehrc/commits/marshamallow-release


Any reason for counting down?


Richtig gut. Für so eine App gibt es bestimmt einen gewissen... Drang.


ok we will see:D here is my build https://jenkins.hubdroid.com/job/Vanilla%20E975/200/console


Also, the `File.ReadAllLines` method can throw a variety of [Exceptions](https://msdn.microsoft.com/en-us/library/s2tte0y1(v=vs.110).aspx#Anchor_1) which wouldn't typically be expected from constructing a grid.


Can surely have an empty EMS-Keys.plist there - just need to make sure we don't commit the real keys.

Ref signing - can't we do the "set the code signing identity to iPhone Developer (Automatic selection) instead of iPhone Developer: Joe McDoe, then it will automatically find an appropriate signing identity in the local keychain, and it should work fine for each developer on your team." ?


Is there one interrupt per bank?


hopefully something will boot soon xD


ok


Is there way to make extension function with two receivers?


Why has the annotation cache been switched to ``Doctrine\Common\Cache\PhpFileCache``? This leads to an issue (#16168) with ``Sensio\Bundle\FrameworkExtraBundle\Configuration\Route`` annotations (and most likely other annotations as well).


it should be 1.10


Generally looks pretty good.  A few minor comment issues, a couple questions.

I'm a bit concerned about the interrupt installation and how that plays with the GPIO system.


`topo.add` is a fairly expensive operation because it re-sorts the graph. Do you have any thoughts on how to optimize this or maybe the ability to disable the feature?

For some of our larger objects, it can take upwards of `50ms` to perform the validation. Reverting this change results in a 3x improvement.


thanks


Looks like `memberTypes` shouldn't be parameter of constructor


A hyperlink where the anchor text is also the URL?


Hi Brian, the spec file has moved forward but you should be able to drop in an old spec and build fine on rhel6.


TeamCity Dirigo Edge :: Pull Request Smoke Tests [Build 89](http://teamcity/viewLog.html?buildId=1259&buildTypeId=DirigoEdge_PullRequestFullChain) outcome was **FAILURE**
Summary: Compilation error: SmokeTests\SmokeTests.csproj (new) Build time: 00:00:12


What is 2500+ lines?


Also, update the comment then?


From <https://github.com/web2solutions/Veritaseum-proof-of-concept/commit/ad1838a239895f06881957d79d03807be4db480b#commitcomment-13632411>:

> @posita ... your test is passing, look: https://travis-ci.org/web2solutions/Veritaseum-proof-of-concept
>
> I just dont trust on this code :+1:

From <https://github.com/web2solutions/Veritaseum-proof-of-concept/commit/2f9fc969d81436bce10467a2a2082a1661a19b62#commitcomment-13645911>:

> I just made your test to pass .. and that is why I said I don't trust this code :+1:
>
> Shall the `NAMESPACE._all_ids[_id]` to be cleaned on `NAMESPACE.resource` level all the times we close it?
>
> For example, supposing it is a database connection (as you did before), shouldn't it to be handled on `NAMESPACE` and shared through all created `NAMESPACE.resource` ?
>
> Do you have any requirement/proposal that makes is more clear?

I think you're right not to trust it. :wink: And you bring up a good point. It's necessary to understand what is actually required to make sure the tests are sound. There is no guarantee that I wrote proper tests. And, as in this case, getting poorly-written tests to pass can give a false sense of security as to whether or not the code is functioning properly.

In answering the question about requirements, let's consider the [test I proposed before](https://github.com/web2solutions/Veritaseum-proof-of-concept/commit/ad1838a239895f06881957d79d03807be4db480b#commitcomment-13621943) in more detail:

```javascript
QUnit.test( "resource is closed at the appropriate time", function( assert ) {
    var rsrc1 = NAMESPACE.resource(42);
    var rsrc2 = NAMESPACE.resource(42);
```

So far, we've got two handles to resources with the same ID. Hopefully, they reference the same underlying resource, because it is expensive to create and we don't want to allocate a new one unnecessarily.

```javascript
    rsrc1.close();
```

Now we've "closed" one of the references (`rsrc1`). What should be the result here? For API consistency, `rsrc1` should probably no longer be usable, and should behave just as it would if the expensive resource was destroyed. In other words, further calls to `rsrc1.close()` it should either be harmless or indicate some kind of error. Also, `rsrc1.getExpensiveResource()` should probably have a behavior appropriate for a "closed" state. My guess is that this is what you were [trying to do here](https://github.com/web2solutions/Veritaseum-proof-of-concept/commit/fa8ea9233501441f06278659c9bc94ae60169b55#diff-d209e42617a0912c33a856b2f2857482R49).

But, as far as `rsrc2` is aware, the resource should still be available, and `rsrc2.getExpensiveResource()` should work as expected.

```javascript
    var rsrc3 = NAMESPACE.resource(42);
    assert.ok( rsrc2.getExpensiveResource() === rsrc3.getExpensiveResource(), "unnecessarily reallocating resource(42)" );
```

Now we create a third handle to the same underlying resource. What the above test should tell you is that we should not create another copy of the expensive resource. Instead, we should use the one we already have (i.e., the one `rsrc2` is still using).

```javascript
    rsrc2.close();
    rsrc3.close();
```

Now we've closed all handles to the expensive resource. The expensive resource should no longer be being used by anything, and should safely be cleaned up.

```javascript
    var rsrc4 = NAMESPACE.resource(42);
    assert.ok( rsrc4.getExpensiveResource() !== rsrc1.getExpensiveResource(), "unnecessarily keeping resource(42) alive" );
});
```

Now we create a new handle to a resource with the same ID as those above. What the above test should tell you is that we expect that a new expensive resource should be allocated, not just because the test says so, but because any existing expensive resource associated with that ID should have been cleaned up in the previous phase.

Your code happens to pass the tests, not because it honors the spirit of what is needed, but because it allocates and returns new "null" objects directly from the handles after they've been closed. Unfortunately, the expensive resources are [still hanging around](https://github.com/web2solutions/Veritaseum-proof-of-concept/commit/2f9fc969d81436bce10467a2a2082a1661a19b62#diff-d209e42617a0912c33a856b2f2857482R58). So now we have a leak and our current tests are not well-written enough to detect that.

Does that give you a better idea of what is expected? If so, can you write better tests to detect the flaws, and then get them to pass by making corrections to the code?


When I've used Sandcastle in the past it would default to hyperlinking the entire URL, rather than a keyword.  However, that was a few years ago so it might be worthwhile double-checking, do you guys publish your documentation anywhere?


Looks good


Try adding them all at the same time (in an array) so that the list is only sorted once.


OK, done, see 88f087d6e16acac8df34aba6329d0cb13c560fe1


You are right. If Read(upstream) returns n=0, err=nil this could actually give "missing" buffers.


ok


I'm not talking about `stopIndex`, but `getIdentifyingStartOffset`, `getIdentifyingEndOffset` etc.


What is the use of the above comment?


icy cool


Haha, it's no big deal [GitHub has a manual where they briefly mention this](https://help.github.com/articles/ignoring-files/#create-a-global-gitignore)


ah yes - I added the "buffer" struct late, so `off` can be contained in that, and a method could be added.


Cut this off


Almost.


happy to help :+1:


Replace this line with:
```
$currencyManager->getCurrency($code)
```


Uncommenting these lines would make reformatting a standard part of the build.


I didn't include the actual code changes from calling ```gradle formatScala```. If you would like me to just let me know.


Like this:
```
func (a *asyncReader) close() (err error) {
	a.closed = true
	if cl, ok := a.in.(io.Closer); ok {
		return cl.Close()
	}
	return nil
}
```


How important is this commit to add into our project? I have been using release 2.1.5, which does not include this commit, and I have had various strange CSRF errors cropping up in the last few months.


ok, but maybe it should be default implementation of `render`?


Can you move this to the "org.openmrs.util.databasechange" package, just like the rest of the changesets?


This fails to link on the CI's (debian) workers, but it works fine on ArchLinux... Looking into it.

```
make[4]: Entering directory `/home/jenkins/workspace/lttng-tools-master-staging/arch/x86-64/conf/std/tests/regression/ust/baddr-statedump'
  CC       prog.o
  CC       tp.o
  CCLD     prog
/usr/bin/ld: prog.o: undefined reference to symbol 'dlclose@@GLIBC_2.2.5'
//lib/x86_64-linux-gnu/libdl.so.2: error adding symbols: DSO missing from command line
collect2: error: ld returned 1 exit status
make[4]: *** [prog] Error 1
```


`<!-- maximum file length 2000 lines -->`
That's not what it does.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.989.07eed44a6b25d9145e4b2343ee8d4057d07ca5a4](http://build.palaso.org/viewLog.html?buildId=64094&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 541, ignored: 6 Build time: 00:04:49


It looks like this breaks the webservice. We need to make sure that when `@language` is set in the webservice we automatically set the output language to ES5.


like:
```
fun render() = name + if (nullable) "?" else ""
```


You are correct. The original changes where made to be compatible with the Datasync version of the API and PUBNUB creation was slightly different from what I recall. I would leave the PUBNUB side in tac.


Why is this class called HasInterceptors?  Looks like an operation.


xq hiciste esto


esto funciona? eu intentei chamar o onSaveInstanceState e petame todo.


This is the only piece of code which uses the JSON API (imported as `storage`).

I think you can get the bucket info from the `gcs` API with this object:
https://godoc.org/google.golang.org/cloud/storage#Bucket


Merged to 9.8.8.x


[-] MO : fix features not being generated correctly in prepareTitle()


Typo on this line, should be session_start();


Not as such. NuDoq picks up the API docs, but appears to be stuck on 1.17.0 for some reason…


an operation that makes sense only on descriptors which supports interceptors. Take into account that this if for the declaration fluent API... It's the same as HasParameters


Thanks Fred.


TeamCity LabkeyTrunk :: Git sqlserver 2014 [Build 7](https://teamcity.labkey.org/viewLog.html?buildId=276321&buildTypeId=LabkeyTrunk_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:01:29


I think this should be a warning instead


syntax error fix


I don't understand why ZeroReader is necessary here.  Could you explain to me :)


wrong Hugo Bessa :P
never commited on the repository


You can remove these allergy-api comments.


Don't tell me what to do!


This behaviour is a bug in the Doctrine cache library (see doctrine/cache#94).


Can we add it some lava damages and some fall damages ?


@abadams Wouldn't something like

```
stream << op->value << 'h';
```

be more consistent?


fixed


In general, it shouldn't even be closing it. That should be the job of the caller. However it made the change in "operation.go" very ugly. That, and when I saw that "Account" also does this, is why I did it anyway ;)

I think testing for an interface and acting on it is ok, and makes it possible to use it elsewhere without big concerns.


Erroneously pushed to master. Sorry for that.


TeamCity Dirigo Edge :: Pull Request Smoke Tests [Build 90](http://teamcity/viewLog.html?buildId=1264&buildTypeId=DirigoEdge_PullRequestFullChain) outcome was **FAILURE**
Summary: Compilation error: DirigoEdge.sln Build time: 00:00:06


It should be safe to hoist this out to the top-level, avoiding needless closures.


Is this type assertion always safe?


Nevermind :smile:


wacky linebreaks?


Hehe wrong terminal window.

---
Mobile

> On 07.10.2015, at 20:39, Sebastian <notifications@github.com> wrote:
>
> Don't commit as root. Never.
> —
> Reply to this email directly or view it on GitHub.
>


Sorry for the reply. I misinterpreted it as the description of another bug.


Also perhaps also an assert

```
halide_assert( op->type.bits ==16) << "expected float16_t\n";
```

I'm also leaning towards this code being a switch statement rather than lots of ``else if``s.


This should be start, not stop, right?


@matthandlersux Latest few drops from my brain dump


will add


This should probably be "before the first non-comment line" not after, right?


TeamCity Dirigo Edge :: Pull Request Smoke Tests [Build 91](http://teamcity/viewLog.html?buildId=1270&buildTypeId=DirigoEdge_PullRequestFullChain) outcome was **SUCCESS**
Summary: Tests passed: 4 Build time: 00:00:57


Bumblebee..... Giant Bug... LOL


Same comments as I made for ``IRPrinter``.


ok. That project required VS15. BUT, VS15 has bugs with constexpr implementation, so if you undef code above, compiler will show error with message like 'not all class fields initialized in constexpr ctor'


I see now


TeamCity Dirigo Edge :: Pull Request Smoke Tests [Build 92](http://teamcity/viewLog.html?buildId=1277&buildTypeId=DirigoEdge_PullRequestFullChain) outcome was **SUCCESS**
Summary: Tests passed: 4 Build time: 00:00:45


@sgehrig because the previous class was been deprecated.


this is suuuper weird


Yeah I'm planning to update the videos soon.


UCSC doesn't do enough to publicize this, but although most of our URLs include `hgsid=...`, there are problems with sharing URLs that contain hgsid.  Instead of `hgsid=...` I suggest `db=hg19` for hgTrackUi links.  FWIW we have some guidelines here: https://genome.ucsc.edu/FAQ/FAQlink.html


ok. вернул в прежнее состояние.


Note: the diff here contains spurious non-semantic differences. The actual change to the ontology involved only adding two new ObjectProperties and associated logical and non-logical axioms. The diff reorders property chains, as well as axiom annotations

```
<!-- Generated by the OWL API (version 3.5.3.20150903-2211) http://owlapi.sourceforge.net -->
```


so meta :+1:


Ah, that's how you do it :-)


*I missed a link_to. will add


You should use the min, max values to mt_rand() instead, as this might favor certain characters (the ones defined first in the string will have a slight advantage over the ones defined later).

We might also want to wrap this with a check if random_int() is available (PHP7) and use that instead, as it will use a proper random generator instead of a pseudo random generator - although it really shouldn't matter, unless someone attempts to create a sequence of collisions by using an initially known state of the mt_rand() sequence from a previous request.


Interesting, I've never integrated with that before.  You can edit the community content by forking this repo, I wonder if you can trigger an update from there somehow?

https://github.com/NuDoq/Community/new/master/Projects?filename=FakeItEasy.md


Hahahaha sorry dude!

Let me introduce your namesake developer..

cc @hugobessaa


Use strlen() instead, so we don't introduce a bug if we extend the number of valid chars later or you've counted wrong. :-)


lose this space.


looks good!


I'm not seeing it on either Linux or Mac, even after doing a "git pull".  When I run "./build.sh native", I don't see any failures or anything indicating that it tried to build System.Globalization.Native.

In corefx/src/Native, I don't see any System.Globalization files.  Where are they supposed to come from?


awesome I am learning a lot from them. You da man


Here docker doesn't return because of the `tail -F` thing. And using `&` isn't enough for ssh to return. That's why `nohup` is required here. Yvan seems acquainted with this kind of ssh tricks and told me to use `nohup`.


Heroku would not allow the following to be added to the <head> of the application layout. There was an error stating that HTTPS was required for external sources.
```ruby
<%= stylesheet_link_tag 'http://dev.gnshield.com/assets/styles/styles.css' %>
```

A quick fix was to shove all the contents of the styles.css into application.scss.


can we push this into the `executor` package? I don't think it belongs in `executor/service`


That's certainly an option, a bit like MSI Afterburner and such where you can have what sensors you want on :)


How about passing those tiddlers as arguments to `tm-save-tiddler` rather than having them hard-coded?


webcomponent.js is not in lib/polymer folder.


This seriously limits the annotation cache or better: it adds a strict requirement to classes used as annotations causing BC breaks all over the place. See linked issue above.


Nope. Consider dictionaries:

    sysconf = {
        'foo': 'bar',
        'subdict': {
            'foo1': 'bar1',
            'foo2': 'bar2',
        }}

    userconf = {
        'foo': 'foo',
        'subdict': {
            'foo2': 'another',
        }}

    sysconf.update(userconf)

    >>> sysconf
    {'foo': 'foo', 'subdict': {'foo2': 'another'}}

Thus I wrote the loop. But it needs to check if the type of the key is dict.


Can we push this into the `executor` package? Perhaps into a `NewMesosCadvisor()` func?


would rather that these fields were package-private. see related comments about pushing things into the `executor` package in order to achieve this.


It builds out of CoreCLR repo.


I think you mean `!D`
Also, the comment one line above is incorrectly mentionning blood splatter instead of dirt.


Hello Richard,

Thank you for heading up development of this valuable parsing script. Unfortunately, I haven't been able to get gvproc.py to work. When I open Terminal (running os x Yosemite), and type "python gvproc.py [path of folder containing boatload of useless HTML files] output.sqlite3", I am met with the following error:

Traceback (most recent call last):
  File "gvproc.py", line 239, in <module>
    WriteContactRecords(args.contactcsv,numbers_to_names,number_notes)
NameError: name 'number_notes' is not defined

I was wondering if you could help me figure out where I'm being derailed. Thanks again.

Alex

P.S. Is .sqlite3 is the appropriate extension for the output file?


It's nice to prevent "undefined... for nil" when moving deep in a hash. Also for setting default values if key does not exists. Or raise.

I will try to put something together soon. Right now I'm patching for the most part, no nice contributions. :(


`get_preferred_phone_number_for_recipient` will either return a `VerifiedNumber` or the phone number as a string, so you'll want to check `isinstance(phone_number, VerifiedNumber)` and use `send_sms_to_verified_number` if that's the case


102867486-FixRectangleShapeTest-5173 (mjwitte) - Win64-Windows-7-VisualStudio-12: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Yes, that's one approach, but it ends up overloading the save-tiddler handler; I like that the current implementation makes use of six separate action widgets for each part of the process.

There might be other ways of thinking about it, anyhow:

* Rather than the current message-based keyboard widget, we could somehow wire the shortcut to the save button, so that the action widgets above would all get executed as usual
* We could extend the keyboard widget so that it could trigger the same sequence of action widgets in response to the keypress


Yep.  I don't get it at all.


I'm pushing a new commit now that addresses this.


It looks like this used to return only mobile workers and now only returns web users - is that intentional?


@DustinCampbell @Pilchie : tagging for any early feedback and also as FYI. I'm still trying this out and adding tests..


Could you use WithContext here to retain the context which is passed into this package?


Don't we need to put in this somewhere that OUR specific project code is actually under this license? From the page link I posted it says, "The copying permission statement should come right after the copyright notices" ...? Now I'm not sure if I did it right for my assignment 1.


(same comment as above)


@janesma, is this what you were thinking?


thanx, Guy! :beers: :)


Merci,


Doesn't this also make unknown ioctls now return -EINVAL not -ENOTTY - I think you need two return paths - one for default: and one for "I know that ioctl but you are feeding me garbage"


covers and completes Issues: #13 #12 #11

adds options for #14


yo para q no pete uso:
this.onCreate(estado);
pero estoy intentando hacer que guarde desde el onActivityResult y no hay manera


Its necessary for all `POST` `PUT` and `UPDATE` calls.


crashes by leotrubach-7 cores, inherited by fastgm and JoJoM


by 9/7 I mean 10/7...


sorry for the delay - I just released ammeter 1.1.3 to rubygems.


I will just remove both commits then.


adds options for #14


sorry for the delay - I just released ammeter 1.1.3 to rubygems


@rorymckinley , this is helpful! it's an NPM package that allows users to populate DHIS2 tracker using a CSV file. It shows us what information we need in that payload!


Because you can now just use e.g. `getIdentifyingNode(node).getDistance()`, etc.


Вынес переменные в глобальную область


Скобка перед Max - лишняя. Max должен быть с маленькой буквы


Thank you, I'm not able to push to superpackage. I always get *nothing to commit, working directory clean*


Скобки вокруг array[i+1] не надо ставить


@paramaggarwal you should add that patch to your own code: `NativeModules.WebSocketModule = NativeModules.WebSocketModule || NativeModules.WebSocketManager`


Good idea! I also think that the most common use cases of batch request involve either all gets or all deletes or all updates. Making this use case simpler could also help the adoption of batch requests. I'll open an issue for this!


well technically defined %anything is wrong. Didn't we spend 8+ hours discussing that a year or 2 ago? :P


These files are necessary for d3d12 builds.


Забыл скобку после else


Please give this a nicer name. `getIds` is too general.


Integration example in Class Admin  :


Скобка после else


лалала


Do people normally import this qualified? I think I normally do an unqualified.


`parent == ""`


Probably because you added the commit only to master and not to superpackage branch. Use git cherry-pick while within the superpackage branch, to repick your commit into it.


Mine.


Why do you have `mText` here?


@youssefbiaz All of the variables in this class should be instance variables.

http://stackoverflow.com/questions/2714573/instance-variables-vs-class-variables-in-python

Check that out and let me know if you have any questions.


Remove the `debug` param, it's not part of our Lua API


I think there is a simpler way to check this:

```
        // if line is outside loc bounds
        if (line < start.line || line > end.line) {
            return false;
        }

        // if it's on start line before the start
        if (line === start.line && column < start.column) {
            return false;
        }

        // if it's on end line after the end
        if (line === end.line && column > end.column) {
            return false;
        }

        return true;
```


@tagliala Thanks! You could just put 1 note in the page footer instead of these inline notes, but this works too.


same comment as above - `phone_number` can either be a `VerifiedNumber` or a `basestring`


I would do something like `bufferIds <- getIds . buffers <$> withEditor get` and get it all done at once, but thats style.


I run ums standalone in linux,  and was getting (unjustified :-) isAdmin() refusals when testing plugin installation.  I looked into it a bit further by trying a local install in Windows, and found similar erroneous refusals from the auto-updater, so this is my attempt to properly honor the "platform-independent" java ethos by not hobbling standalone installation for no good reason.

If there is any good reason that I've missed, feel free to modify/revert :).


Lol thank you.


oh my god I see bootanimation


Yes (although actually tls1_process_ticket has an explicit NULL check too)


Ouch, rewriting imports is heinous indeed :disappointed:

This makes life hard extra hard for packagers too, but they're going to have to package all of etcd for this dep anyhow (which as I understand it is in progress for newer versions of etcd than the 2.0 I see in the Debian archive specifically, but newer versions can be a lot tricker, like how with Docker 1.8 we added a huge boatload of shady little new deps that are _still_ not quite all packaged).

Can we at least vendor version tags where possible?  I know etcd has explicit releases, and that helps quite a bit (even though it's only one small part).


`phone_number` can be `VerifiedNumber` or `basestring`


`runProguard` was changed to `minifyEnabled` in more-recent versions of the android studio gradle plugin.


Ok, this bug should probably be filled so they'll fix it in the next update.


Suggest something other than returning negative numbers.  Throw an error maybe?


This is called a set. Use the correct data structures, please!


Ah, it got disabled at some point. Fixed in 8a763e9d838fcb571d12c4175952cd4e15e95e00


actually finished the whole project, just made a mistake making the commit


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 745](https://teamcity.labkey.org/viewLog.html?buildId=276323&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 13, passed: 229, muted: 23 Build time: 00:34:46
### Failed tests
```
org.labkey.test.tests.CDSTest.verifyGroups: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutStudyProducts: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyFilters: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutAssays: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyHomePage: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyFilterPane: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.ScrumtimeTest: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyAntigenScatterPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyBoxPlots: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyScatterPlotColorAxis: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyAntigenBoxPlot: <no details avaliable>

##### there are 3 more failed tests, see build details
```


Could you describe the behavior of the change? I'd rather see that over that over sussing it out from reading the code.


Really? If yes, o shit


Don't need to return a tuple. A None return value can be used to indicate False, [] and a valid vcf.model._Record(...) can be used to indicate True, vcf.model._Record . Simplifies the code a bit. Later on, instead of checking if the first part of the tuple was True or False, we can simply check if the merged vcf record was None or not.


Nope, it's not the same type.


Thank you so much. I'm really a git noob. Is it possible to delete the empty commit I made?


Making this public doesn't seem harmful. It is used by some test classes and users could also reuse it.


Small tweak so that the returned codec is guaranteed to implement the extra interface.


Hrm, I think it should be safe to use the pointer, but I'm all about doing a defensive copy.


`AndroidManifest.xml` is not generated.


Bad algorithm, this is slow and memory inefficient. Also you forgot to clear the map afterwards. Use something like this:
```
Iterator<...> it = mMap.iterator();
while (it.hasNext()) {
   UIWrapper obj = it.next();
   obj.dispose();
   it.remove();
}
```


Yes, that is true, but on the other hand, it isn't very likely to happen for the two elements of this class, since they are suppose to be like simple values (or references it T is a class). Properties are gate to the interior of a structure/class, they can regulate the transit. If you don't want the regulate anything (leave the gate open, so to speak) you don't need them.

In regards to interfaces, on the other hand, where you can't define values for the structure/class to have, properties are a very good reason to be used. And you can even override properties, or to turn the reasoning around: you could use values, when you don't want them to get overriden or hidden.


It probably is -- I'll test.


A conditional here would be good instead. So, you can have `return pybedtools.Interval ... if sv_type == "INS" else feature` instead of the `if not sv_type == "INS": .... `


No, we already have this helper: strlen_flags()


wrong typehint


@SubJunk presumably this removes any obstacles to supporting "platform-lists" in your php database.  Single multi-platform jumpy install list is [here](http://skeptical.github.io/jumpy/jumpy-inst).


Originally it was to prevent the horrible diffing by indenting everything (it would treat the renames as deletes and creates instead), and I just stuck with it to be consistent. In C++17 we will be able to write `namespace chesspp::server` instead.


Rota desnecessária.


There's currently no way to do it automatically, but another options would be to use [vtables](https://en.wikipedia.org/wiki/Virtual_method_table). If you're familiar with C syntax, http://eelcovisser.org/post/11/virtual-method-tables is a good resource.


Yes, use ```git rebase -i HEAD~2``` and delete the line with the empty commit.
Then use ```git push --force``` to push it to github.


O certo seria um post para /users ou /usuarios


can you be a bit more explicit with this message? Boost does build with gcc 4.9 so maybe cite the relevant Jira ticket here to avoid downstream confusion?


post para /tarefas


post para /categorias


Hi guys,

I've noted your discussions here and wanted to tell you about my own work on this direction. Likely, everybody tried to build their own wrappers ;)

I analyzed the way with the `dynamic` keyword and liked it (even added to todo), however decided to proceed with a "static" direction. I defined the `IPtr` interface that provides address, type and heap and build a set of different extension methods around that. One kind of extensions allows to read fields from the object.
Below are examples of code that is used to achieve the different tasks.

Read fields:

    IPtr ptr; //here we somehow get the object.
    int intValue = ptr.GetInt("intField", /*interior true/false*/)
    string strValue = ptr.GetString("stringField");
    IPtr refValue = ptr.GetRef("ptrField");
    IPtr structPtr = ptr.GetStruct("structField");

Read chains:

    int intValue = ptr.GetSimpleChained<int>("refFld.someStruct.intFld");
    IPtr ref = ptr.GetRefChained("refFld.innerRef");

To read collections (example of supported: `Array`, `List<T>`, `Dictionary<TKey,TValue>`, `ConcurrentDictionary<TKey, TValue>`) I created the specialized wrappers. API usage example:

    List<int> listOfInts = ptr.AsCollection().List()/*Wrapper is returned. I can get elements, size, etc.*/.Read<int>();
    List<KeyValuePair<int, IPtr>> dictionary = ptr.AsCollection().Dictionary().Read<int, IPtr>(); //For Dictionary<int, object>

To read complex values:

    DateTime dateTime = ptr.GetThisComplexValue().DateTime();
    Guid guid = ptr.GetThisComplexValue().GuidValue();

Almost everything is implemented as extension methods, so could be easily expanded.

Unfortunately, currently source code is located in the private repository, so I cannot provide you with a link. However, everything is documented and most of the stuff is covered by the xUnit tests.

Reason why I'm posting all this here is that I could share you my code or even create a pull request (previously adapted it or renamed entities). While the "dynamic" model looks cool, I believe that "static" model is clearer and allows to start more quickly. Also it provides a little better performance :) So, ideally, we could have 2 models and later consumers will be capable of selecting the model they like.

What do you thing about this? Does it make sense to create a pull request and you will merge it (or modify if you need) as well? I spent a lot of time developing and testing all these extensions, so would be very happy to contribute them to this great project, instead of using them in my local projects only :)


is there a reason we switched back to 0.6.3 here?


delete para:
* /usuario/:id
* /tarefa/:id
* /categoria/:id


Главная страница (голый макет).


sure, in cases of pressing shift enter after typing open parenthesis to open a block, assuming brace completion is on, we produced code like this:

initial:
```C#
try { |}
```
final:
```C#
try { }
|
```
basically, in such a case, we didn't have anything to complete the line (like a semicolon that we need to insert), yet we handled it anyway and inserted a Newline after the block and positioned caret on the next line.

a better output here could be what we produce on enter:

```C#
try
{
    |
}
```

the implementation is simple: detect cases like this and delegate the command to the editor whose default implementation is exactly what we need.


also see: http://stackoverflow.com/questions/32259287/visual-studio-2015-brackets-not-automatically-formatting-when-pressing-shift


Good idea, done.


LGTM.  Is the case that "cookie.parse(req.http.Cookie)" blows up if req.http.Cookie is not set?


yep but now we have bootloop http://pastebin.com/GrwSkdW2


Sounds great!


Nope. Not sure why it was rolled back.


Benches should be identical...


You made a typo. You wrote "canols" instead of "canola".


It looks like `TEST_BACKEND` is set to 'test-backend' which is not an actual backend in the system so this will cause an error.  You can set this value to 'MOBILE_BACKEND_TEST', however


In these instructions, the only files copied from CoreCLR to the coreclr-demo directory are corerun, libcoreclr.dylib, and mscorlib.  Those files are copied explicitly; no wildcards are used to copy other files from CoreCLR.


k


will flash mine when it finishes, which is very soon


Awesome map, it's nice to see winter in flare. If we could make possible to enter houses, it would be even more awesome, but we need new tileset for interior.


Thats good!


It looks like a lot of these tests were switched from running on mobile workers to running on web users.  Is that because the recipients of these alerts are only web users?


I wonder how this association was established in TW2.

Other than being less redundant, this sounds safer to me:
> Rather than the current message-based keyboard widget, we could somehow wire the shortcut to the save button, so that the action widgets above would all get executed as usual

than this:
> We could extend the keyboard widget so that it could trigger the same sequence of action widgets in response to the keypress

If the latter, perhaps introduce a way to transclude action-templates, e.g.:

```
<$fieldmangler>
<$button tooltip={{$:/language/Buttons/Save/Hint}} aria-label={{$:/language/Buttons/Save/Caption}} class=<<tv-config-toolbar-class>>>
{{$:/core/actions/save-tiddler}}
<$list filter="[<tv-config-toolbar-icons>prefix[yes]]">
{{$:/core/images/done-button}}
</$list>
<$list filter="[<tv-config-toolbar-text>prefix[yes]]">
<span class="tc-btn-text"><$text text={{$:/language/Buttons/Save/Caption}}/></span>
</$list>
</$button>
</$fieldmangler>
```


Mocks injected in the unit we want to test. Very good!


This sentence seems weird: "a small number of observations from the information"?
I would say it oppositely, "a small number of information from the observations".


Thank you again!


Also right. We check that both collaborators will be called if we execute carService.export()


“formule” is French for “formula” :wink:


##### Application to Image Compression
This sentence seems weird: "a small number of *observations* from the *information*"?
I would say it oppositely, "a small number of information from the observations".


@RandomAltThingy Awesme!


Thanks all! A few hacky regexes don't hold up too long, do they? :)

I have moved this into a dedicated script: https://github.com/paulirish/dotfiles/blob/master/bin/diff-so-fancy (https://github.com/paulirish/dotfiles/commit/30a6296f37c8f5d2d0aa3cb5520c32f6fda12102)

I've incorporated much of @devinivy's edits. And documented that gnu-sed is mandatory and all that.

Give it a whirl and PR me if it isn't good enough. Happy for this to graduate to its own repo too, if anyone is interested in helping maintain.


We enumerate all values, exception never occurs here. "UNKNOWN" is to just shut up false compiler warning.


Don't mind, I was just wandering what did it require such old version of scipy, and I failed to install that.

I'm excited to see you updated the requirement just after my trial. But I did not notice the branch and failed again…My fault :)


Nice job linking the Google fonts!


Hi Pieter.

I like the SaveFile change here. This is a brief heads-up that I'll do something similar in my repository (to be clear, not the UTF-8 specific stuff).

kind regards,

Linds.


Nice work with semantic markup!


@eneim Do you have any idea how to solve it?


When we conducted imports of several layouts at the same time I received this error, I do not know if this related to modification.

SQLSTATE[40001]: Serialization failure: 1213 Deadlock found when trying to get lock; try restarting transaction


Not quite the right use of clearfix, but good try!


Type and Status don't conflict. If they'd do, the program wouldn't compile. So no need for additional keywords.


Dear All,

I have checked online, so the JUnit test classes work now.

On the menu bar of Eclipse, go to Properties.

Then, click on Java Build Path on the left of the window which appears.

Then, expand the "JARS and class folders on the build path" by clicking on them.

If you find JUnit3, it is recommended that you delete it.

Most importantly, for JUnit testing to work, the test classes must use JUnit 4.

Therefore, the next step is to click on Add Library on the right of the window,

select JUnit,

then click Next,

then select JUnit4 in the drop-down menu,

and click on Finish.

Thank you.


@confile Hm I'm investigating. It doesn't crash on my Nexus 5 (5.1.1). But it seems that loading a bunch of Photo in your layout at once will cause the OOM in some ways. I will try a more critic situation (increase the visible cards number, for example) to reproduce the OOM.


Also, note that [restview](https://mg.pov.lt/restview/) is nice for previewing restructuredtext.  Or we could switch to markdown if you're more familiar with it.


Be careful using `<br />` tags in paragraphs to break lines. While this may work well on desktop/laptop, it can cause big problems in responsive.


Dont forget your image `alt` attributes.


I originally set the deck equal to shuffle. Shuffle holds a full deck. Then I remove cards from deck as I assign them to players. When the hand is over I set the deck back to shuffle, basically resetting the deck.


While using a button makes sense and can work, using a link (`<a>`) to create the "button" would be the best practice. Button tags cant link you anywhere.


corrected @ziad-saab


Do I allow to edit phonebook_orig.c for compatibility with this main.c ?
Or, I can only edit main.c ...  there might be a little trouble for me.


Originally i did not want to ship the ic2 classic api. Oops.


Nice job adding the image to the header! This is the best practice :) Good work linking the correct file path.


LOLOLOLOL/


Positioning! Now that's an advanced tag. Nice work experimenting.


My preference is avoiding heavy constructors, especially ones throwing exceptions.


teset


AWESOME JOB trying floats! Well done!


I believe line 58 needs guestId added as the first parameter?  I was testing this with a web app that opened new windows and posted messages between them and it didn't work because the parameters in the new window were off by one.  When I edited atom.asar to add guestId to that line everything started working properly.


You're doing a great job using a combination of selectors - elements and classes. Whether you realize it or not, you are starting to really use the power of the CSS cascade.


おっ travis 仕事してますね


Good job adding the background image.


助かりました。


I know, because I needed to release 0.4.0 before I could. I basically based that entire release on what I needed to move over SatanicBot to MediaWiki::Butt.


Tried with no success. I think merge() is the best name. What's wrong with out parameters?


Nice job using complex selectors! Another way to write this, because you are not using the benefits class on multiple elements, is `.benefits ul`


I still dont know how to do


我还是不懂


removed the files from previous exercise @ziad-saab


Esto directamente en vez de tenerlo en el constructor y en una variable privada podrias talves tenerlo como una constante ? #define regex ",(?=([^\"]*\"[^\"]*\")*[^\"]*$)"; (No esta chequeado igual, pero me parece que se podia)


It may be LibkedHashMap for jdk7


What similarities do you see between this button and the button in the header? Can you leverage two classes - one with shared elements for both of the buttons and then another to override or apply custom style?

The more advanced you get with CSS, the more you will want to push to not repeat yourself.


@wkitty42, I know that is probably a good way to do things but what I do is way easier and I only need one repo instead of two, (just the local). Every thing else as far as flow is basically the same. Pull from master make local changes, create a new local with a different name and push it back. Then ask for a pull request.

 I really don't like that one extra step of cloning on the web interface. For me it is a waist of time and space. Nut I can see the necessity if you don't have access.

@thevirtualfer, I think that is what you basically did except you need access rights. I'm sure we can get you set up that way.


I need to update this once cp-validations updates ver.


I would have expected System.Globalization.Native.dylib to also be in this list, did it not get built?


#Week 1 task

print "We want... a shrubbery!"


Much more familiar with Markdown if you don't mind that syntax instead.


Created a new issue for this here: #3


gdgdgdgdg


dg


gd


d


ggd


g


Great work, Amanda! You're doing a great job with your semantic markup and overall page structure. Your CSS syntax is solid and you're doing well with your selectors. It will be personal taste if you want to leverage classes or element selectors, and ofter you will do both. Look to see what elements on your page have shared properties and see where you can share these in either a selector or a class.

###Description

A hot, new NY Tech Startup, Relaxr, has approached you to help them develop a new landing page for their company. They've handed you the design file for the site, along with the copy and assets. You need to take these files and turn it into a landing page.

Student: ____Amanda Gaines______

|                                                                                |                                |                        |                          |
|-------------------------------------------------------------------------------------------------------------------------------|--------------------------------|------------------------|--------------------------|
| Technical Requirements                                                                                | Does Not Meet Expectations (0) | Meets Expectations (1) | Exceeds Expectations (2) |
| Use HTML5 structural elements (nav, header, footer)                                                                           |                                |                        |            X              |
| Use a CSS Reset file in addition to your style.css file to style the page                                                     |                                |                        |           X               |
| Use IDs and Classes to to select and style elements on the page                                                               |                                |                        |          X                |
| Style your text with the Google Fonts provided by your style guide                                                            |                                |                        |          X                |
| Follow naming conventions, maintain consistency across .html and .css files and use best practices for naming IDs and Classes |                                |                        |              X            |
| Indent nested elements to increase your code's readability                                                                    |                                |                        |           X               |
| Use width, padding and margin on at least two block elements on your page                                             |                                |                        |             X             |
| TOTAL: __14__/14                                                                                |                                |                        |                          |


I think a simple replacement of `recover` with `recoverWith` will make this correct.


This fix1 might work


Sure, let's plan to switch.


Don't think fix2 will work


想问下，activity的拦截通过PluginCallback 的hook来实现？


This patch for Cyanogen recovery and battery charger screen.


Hah, good point. Life has forced some chunks of Swift out of my head.


Implement.


I am Python Developer and this held so much promise and works on Scala, BUT NOT IN Python!!!
1. The compiling of the source does not work!  Not on Mac, Not on Ubuntu..  What's the deal
2. Pyspark with Data Frame -- Experimental!!! Are you kidding?

What is the plan to make this usable for Python?


@mstembera In the past, I have not run tests against other branches other than master. I have rescheduled with base tag set to:
1e238387b93324fd5b75d112ff7f11f79971506c
Is this correct?


Though Swift 2.0 can't express this, because it can't hoist the optionals correctly. Back to the way I wrote it, I guess!


I thought about that, and decided to do it this way, since it keeps the "preferred" version here, and the fallback relegated to a separate function. Also, the name for this version factored out would be weird, `caml_toplevel_readline_editline` or something, and it would still have to be behind an ifdef.

But happy to factor it otherwise if you disagree?